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



--- Comment #7 from fedepell <[email protected]> ---

Hello Jakub!
Once more: thanks again for your great help and time spent on the review! :)

(In reply to Jakub Kadlčík from comment #6)
> > These are then not delivered in the RPM
> This IMHO isn't true. Check this out:
> /usr/lib/python3.10/site-packages/robot/htmldata/lib/jquery.highlight.min.js

You are of course right :) I was confusing two things: robot HTML
documentation, which the package is not delivering (and would keep not to as
per other comments), and the minimal libraries it delivers to display the
generated HTML reports (when HTML reports are generated). This jquery is indeed
in the second one as well! 

I therefore added now licensing + bundled as suggested before! Hope it looks
fine! I'm a bit unsure, although I've tried to reread a couple of times the
bundling naming indications, if the naming I choose sound reasonable, as there
seem to be some freedom left for those.


> > Spec file as before: 
> > https://github.com/fedepell/rpms-specs/blob/master/python/robotframework/python-robotframework.spec
> The raw file ... ;-)

Hope I can get it right this time! :D (sorry!)

Spec:
https://raw.githubusercontent.com/fedepell/rpms-specs/master/python/robotframework/python-robotframework.spec
SRPM:
https://github.com/fedepell/rpms-specs/raw/master/python/robotframework/python-robotframework-5.0.1-6.fc36.src.rpm

Cheers,
Federico


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2087143
_______________________________________________
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