[Bug 1919349] Review Request: crash-trace-command - Trace extension module for the crash utility

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

 



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

Vladislav Kazakov <vpackager@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |vpackager@xxxxxxxxx



--- Comment #2 from Vladislav Kazakov <vpackager@xxxxxxxxx> ---
Hello again! 
I will duplicate the remarks I made in BZ#1919347.

It's better to start versioning your libraries. As the upstream maintainer you
can do that. Please, see:
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_downstream_so_name_versioning

> Release: 0%{?dist}
Release: tag starting with 1 (never 0).

> ExclusiveArch: x86_64 aarch64
What about other arches? Crash supports other arches too.
As stated in Packaging Guidlines, "Fedora packagers should make every effort to
support all primary architectures".

> BuildRequires: crash-devel >= 7.2.0-2
All build dependencies must be listed in BuildRequires, except for any that are
listed in the exceptions section of the Packaging Guidelines.
You need at least gcc.

> make
Whenever possible, invocations of make should be done as %make_build.

> %defattr(-,root,root)
The %defattr directive in the %files list SHOULD ONLY be used when setting a
non-default value, or to reset to the default value after having set a
non-default value.

> %doc COPYING
It's more license than doc, so you can use %license here. 
You can set REAMDE as %doc for example.

And some new:
> %clean
> rm -rf %{buildroot}
%clean section SHOULD NOT be used.


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




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

  Powered by Linux