[Bug 2235079] Review Request: python-pygmars - Craft simple regex-based small language lexers and parser

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=2235079

Robert-André Mauchin 🐧 <zebob.m@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |needinfo?(benson_muite@emai
                   |                            |lplus.org)



--- Comment #2 from Robert-André Mauchin 🐧 <zebob.m@xxxxxxxxx> ---
> c) Documentation has a number of bundled javascript libraries such as jquery,
can the spec file indicate these are bundled?

There's only jquery so I added

Provides: bundled(js-jquery)

> Alternatively, consider generating man pages or another documentation format that does not bundle extra content.

We´ŕe not gonna stop providing html doc because sphinx bundles jquery I
think.This has always be the case and PDF doc is way less pratical.

> d) Is sphinx-rtd-theme required?

Why? This is packaged for Fedora and this is how the author wants, why would we
get rid of it, I don't understand. 


Thank you the review.

Spec URL: https://eclipseo.fedorapeople.org/for-review/python-pygmars.spec
SRPM URL:
https://eclipseo.fedorapeople.org/for-review/python-pygmars-0.8.0-1.fc39.src.rpm


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
https://bugzilla.redhat.com/show_bug.cgi?id=2235079

Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202235079%23c2
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux