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

Robert-André Mauchin 🐧 <[email protected]> changed:

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



--- Comment #2 from Robert-André Mauchin 🐧 <[email protected]> ---
> 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 -- [email protected]
To unsubscribe send an email to [email protected]
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/[email protected]
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to