https://bugzilla.redhat.com/show_bug.cgi?id=1997994
Ben Beasley <code(a)musicinmybrain.net> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |code(a)musicinmybrain.net
--- Comment #3 from Ben Beasley <code(a)musicinmybrain.net> ---
A few comments, based on a quick look at the spec file. I didn’t try to build
it or run fedora-review.
----
Runtime dependencies due to shared library linking are automatically generated.
You should not need lines like:
Requires: libsodium >= 1.0.18
You might still need those that provide command-line tools or other resources.
----
Versioned dependencies are discouraged when the previous three Fedora releases
would satisfy the dependency, i.e.:
BuildRequires: libcurl-devel >= 7.29
should be
BuildRequires: libcurl-devel
after consulting
https://src.fedoraproject.org/rpms/curl. See
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_package_depen....
----
You can more cleanly write
%package -n oidc-agent-cli
as
%package cli
and so on.
----
You should use the %make_build and %make_install macros.
----
You should remove
%defattr(-,root,root,-)
per
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_file_permissions.
----
You shouldn’t package static libraries without a strong justification:
%{_libdir}/liboidc-agent.a
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_packaging_sta...
----
You mustn’t assume gzip compression for man pages
(
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_manpages).
----
Do you have a reason for using %config instead of %config(noreplace)?
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_configuration...
--
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=1997994