[Bug 428138] Review Request: system-summary - A quick summary of system hardware

bugzilla at redhat.com bugzilla at redhat.com
Wed Jan 9 18:35:42 UTC 2008


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: system-summary - A quick summary of system hardware


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





------- Additional Comments From jbowes at redhat.com  2008-01-09 13:35 EST -------
Hi:

Updated spec and srpm:
Spec URL: http://jbowes.fedorapeople.org/tmp/system-summary.spec
SRPM URL: http://jbowes.fedorapeople.org/tmp/system-summary-0.0.3-1.fc8.src.rpm

(In reply to comment #1)
> 1)If direct download link of tarball is not available then add some text in
> comments on how tarball can be created?
> 
> 2)for desktop file installation you should use desktop-file-install command.
> see
>
http://fedoraproject.org/wiki/Packaging/Guidelines#head-d559ee7363418a5840ce63090c608c991cd39ce6
> here, you may like to add --vendor="fedora"
>     
> 3) rpmlint output on srpm gave me
> system-summary.src: W: strange-permission system-summary.spec 0600
> A file that you listed to include in your package has strange
> permissions. Usually, a file should have 0644 permissions.

I've fixed 1. (created a download location for the tarball) and 2., 3. is an
artifact of how I made the srpm (rpmbuild -ts), and will go away once everything
is in distcvs.

Thanks for taking a look!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.




More information about the Fedora-package-review mailing list