https://bugzilla.redhat.com/show_bug.cgi?id=2006590
Ben Beasley <code(a)musicinmybrain.net> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags|needinfo?(code@musicinmybra |needinfo?(denis(a)fateyev.com
|in.net) |)
--- Comment #2 from Ben Beasley <code(a)musicinmybrain.net> ---
Thanks for the review!
- Dist tag is present.
Fedora-review reports a dist tag presence, but it seems a false positive and can be
ignored.
It’s actually confusingly trying to complain that the dist tag is not present,
but it’s because it does not understand rpmautospec. It’s just looking for the
%{?dist} macro in Release, and doesn’t understand that %autorelease includes
the dist tag.
- Architecture-specific "devel" packages are actually
identical: they provide the same "earcut.hpp" source file version.
The difference is in the testing suite flow, but for an end-user they all provide the
same content.
Please consider changing devel package to "noarch".
This seems reasonable—especially leaving the base package arched to run the
tests on all architectures, while marking the -devel subpackage noarch—and I
have used this approach on Python packages that have no compiled code but are
prone to arch-dependent test failures.
However, for header-only library packages like this one, the guidelines
actually forbid using noarch:
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_do_not_use_no...
--
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=2006590