[Bug 524107] Review Request: qbrew - A Brewing Recipe Calculator

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

 



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


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





--- Comment #2 from Martin Gieseking <martin.gieseking@xxxxxx>  2009-09-18 08:31:02 EDT ---
I'm not a sponsor but nonetheless some quick comments:

- the license tag must be BSD, as file LICENSE and the source headers mention
the BSD license (two clause variant)

- in %Source0, give the full URL to the tarball

- Since with Fedora 10 it's not necessary to define a BuildRoot as rpm does
this automatically. However, if you want to support older Fedora releases or
EPEL <= 5, you have to define it this way:
http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

- clear the execute flags of the source files, e.g. by adding 
    find src -type f -exec chmod 0644 {} \;
  to the %prep section

- in %files, replace %defattr(-,root,root) by %defattr(-,root,root,-)

- add the files AUTHORS, ChangeLog, and TODO to %doc

- add the version and revision number to the changelog entry:
  ... <chris.a.st.pierre@xxxxxxxxx> - 0.4.1-1


$ rpmlint /var/lib/mock/fedora-11-i386/result/qbrew-*
qbrew.i586: W: incoherent-version-in-changelog chris.a.st.pierre@xxxxxxxxx
['0.4.1-1.fc11', '0.4.1-1']
qbrew.i586: W: invalid-license Any permissive
qbrew.src: W: invalid-license Any permissive
qbrew-debuginfo.i586: W: invalid-license Any permissive
qbrew-debuginfo.i586: W: spurious-executable-perm
/usr/src/debug/qbrew-0.4.1/src/textprinter.h
qbrew-debuginfo.i586: W: spurious-executable-perm
/usr/src/debug/qbrew-0.4.1/src/textprinter.cpp
qbrew-debuginfo.i586: W: spurious-executable-perm
/usr/src/debug/qbrew-0.4.1/src/configure.cpp
qbrew-debuginfo.i586: W: spurious-executable-perm
/usr/src/debug/qbrew-0.4.1/src/configure.h
qbrew-debuginfo.i586: W: spurious-executable-perm
/usr/src/debug/qbrew-0.4.1/src/qbrew.cpp
3 packages and 0 specfiles checked; 0 errors, 9 warnings.

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

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]