[Bug 2156237] Review Request: rapidfuzz-cpp - A fast string matching header-only library for C++

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

 



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



--- Comment #16 from Benson Muite <benson_muite@xxxxxxxxxxxxx> ---
Thanks. 

Javascript in HTML documentation is contentious and different approaches are in
packages available in Fedora.  It seems ok for me, at present with man pages,
though it maybe good to make some progress resolving this issue.  Is it
reasonable to remove jquery after the install step and place a symlink to
/usr/share/web-assets/jquery/latest/jquery.js ?

There seems to be one other bundled library in doxygen
https://github.com/doxygen/doxygen/blob/master/templates/html/svgpan.js
though this is not used in the generated documentation for rapidfuzz-cpp

There are similar issues for other html documentation generation systems. Maybe
one could come up with reasonable guidelines to add to Fedora packaging
documentation for doxygen?

It seems fine for me at this point, but if you want to add html documentation
with symlinked jquery.js this would be fine as well. Personally, would advocate
for html docs to be in separate docs packages since this allows easy opt out of
other software that is often bundled, but upto you on this if you want to ship
html documentation.


-- 
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=2156237
_______________________________________________
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