systeminfo - review package
David Roheim
david.roheim at gmail.com
Tue Feb 18 23:13:49 UTC 2014
Hi Pavol,
I can not approve your package, but I would suggest some things.
* Bump your rev version. Incl. changelog.
* Define your BuildRoot
* Use %{_bindir} instead of /usr/bin
* Source-format your spec-file
Why do you have:
%if 0%{?rhel} && 0%{?rhel} <= 5
rm -rf %{buildroot}
%endif
Why not just:
rm -rf $RPM_BUILD_ROOT
Nice page: https://fedoraproject.org/wiki/Packaging:RPMMacros
Anyways :)
David
*David Roheim*
david.roheim at gmail.com - http://roheim.org/
On Tue, Feb 18, 2014 at 9:41 PM, Pavol Ipoth <pavol.ipoth at gmail.com> wrote:
> Hi,
>
> can please somebody approve my package, it was already reviewed but
> reviewer doesn't have permissions to approve new package. Can somebody look
> at it. Thanks.
>
> https://bugzilla.redhat.com/show_bug.cgi?id=1058038
>
> --
> devel mailing list
> devel at lists.fedoraproject.org
> https://admin.fedoraproject.org/mailman/listinfo/devel
> Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.fedoraproject.org/pipermail/devel/attachments/20140219/cf4cb7e2/attachment.html>
More information about the devel
mailing list