[Bug 2099902] Review Request: python-sphinx-design - Sphinx extension for responsive web components

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

 



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



--- Comment #3 from Jerry James <loganjerry@xxxxxxxxx> ---
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.

> 7) Why not use tox to run the tests?
> https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_tox

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.spec
SRPM URL:
https://jjames.fedorapeople.org/python-sphinx-design/python-sphinx-design-0.2.0-2.fc37.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=2099902
_______________________________________________
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 on the list, report it: https://pagure.io/fedora-infrastructure




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

  Powered by Linux