https://bugzilla.redhat.com/show_bug.cgi?id=2099902
--- Comment #3 from Jerry James <loganjerry(a)gmail.com> ---
Thank you for the unofficial review, Benson. Here are my (not completely
satisfactory) answers to your questions.
(In reply to Benson Muite from comment #2)
- If (and only if) the source package includes the text of the
license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %license.
Note: License file LICENSE is not marked as %license
See:
https://docs.fedoraproject.org/en-US/packaging-
guidelines/LicensingGuidelines/#_license_text
When the python dist-info is assembled, it includes the LICENSE file. You can
see it in the binary RPM under this path:
/usr/lib/python3.10/site-packages/sphinx_design-0.2.0.dist-info/LICENSE
Miro has been telling people that adding the license file under %license as
well is redundant, so I left it out of this package. However, it should be
included in the docs subpackage. I will fix that.
- Large documentation must go in a -doc subpackage. Large could be
size
(~1MB) or number of files.
Note: Documentation size is 2088960 bytes in 61 files.
See:
https://docs.fedoraproject.org/en-US/packaging-
guidelines/#_documentation
??? There is a python-sphinx-design-docs package, exactly the one that contains
61 files, in fact.
[!]: License file installed when any subpackage combination is
installed.
This will be fixed in the next version.
[!]: If the package is under multiple licenses, the licensing
breakdown
must be documented in the spec.
It is documented, in the comment just above the License field.
[!]: Package contains no bundled libraries without FPC exception.
I am unaware of any bundled libraries. Which ones did you find?
1) Material icons seem to be available as a package
https://src.fedoraproject.org/rpms/material-icons-fonts but are bundled with
this
installation. Can the material-icons-fonts package be updated so that
bundling is not needed?
Yes, and I'm even the maintainer of that package. :-) Here's the thing: the
JSON versions of the icons are not in the source package. There are SVG icons
in the source package that appear to be the source of the data in the JSON
icons in this package. I assume that there is some SVG-to-JSON converter, but
I know very little about fonts and certainly do not know anything about such a
tool. The font guidelines
(
https://docs.fedoraproject.org/en-US/packaging-guidelines/FontsPolicy/) say
that the SVG icons are not to be installed, but do give latitude for installing
JSON files. If anybody reading this knows what to do, we can modify the
material-icons-fonts package to include the JSON files and unbundle them from
this package.
In that case, I would also be willing to package octicons and do the same
unbundling. Even better would be if somebody who knows what they are doing
submits octicons for review. :-)
2) Material icons are under Apache Software License
Yes, the comment above the License field says so.
3) Documentation contains javascript files which may also have
different
licenses
Good point. I will add a comment above the License field in the docs
subpackage.
4) License file seems not to be packaged
See explanation above.
5) Can octicons
https://github.com/primer/octicons be packaged
separately
instead of using a bundled library?
If the SVG-to-JSON hurdle can be overcome, then yes.
6) Timestamps are preserved for documentation, but not for the python
files
The spec file uses the approved python macro, %pyproject_install, to do the
installation. If it doesn't preserve timestamps when it should, that is out of
my hands.
Because it doesn't seem to do anything. This runs with no noticeable delay:
```
+ /usr/bin/python3 -m tox --current-env -q --recreate -e py311
___________________________________ summary
____________________________________
py311: commands succeeded
congratulations :)
```
as opposed to using %pytest, which takes several seconds and prints meaningful
output.
New URLs (although that doesn't matter, since this bug is closed):
Spec URL:
https://jjames.fedorapeople.org/python-sphinx-design/python-sphinx-design...
SRPM URL:
https://jjames.fedorapeople.org/python-sphinx-design/python-sphinx-design...
--
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=2099902