Spec URL: http://xiashing.fedorapeople.org/SPECS/pcalc.spec SRPM URL: http://xiashing.fedorapeople.org/SRPMS/pcalc-2-1.fc12.src.rpm pcalc is a program for calculating and converting between different bases.
There is an updated spec file and SRPM to fix some rpmlint errors. Spec URL: http://xiashing.fedorapeople.org/SPECS/pcalc.spec SRPM URL: http://xiashing.fedorapeople.org/SRPMS/pcalc-2-2.fc12.src.rpm
* A more concise "Summary" can be found directly in the README file: Summary: Programmer's calculator, command line utility * https://fedoraproject.org/wiki/Packaging:Guidelines#Compiler_flags * Have you noticed the test suite? * https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License_Clarification Not directly related to the review, but none of the source or documentation files confirms that the GPLv2 (as in file "COPYING") shall be applied. Only the two bison generated files pcalc.c and pcalc.h refer to GPLv3+. $ rpmlint pcalc-2-2.fc14.x86_64.rpm pcalc.x86_64: W: no-manual-page-for-binary pcalc 1 packages and 0 specfiles checked; 0 errors, 1 warnings. Including the "EXAMPLES" file would make sense. And why not include the "AUTHORS" file, too?
Not sure what to do about the licensing and the lack of manpage in the RPM, but I've fixed up the summary and description for the spec file. New SPEC: http://xiashing.fedorapeople.org/SPECS/pcalc.spec New SRPM: http://xiashing.fedorapeople.org/SRPMS/pcalc-2-3.fc12.src.rpm
What about the other issues? The compiler flags, the test suite, and the missing %doc files?
I fixed up the %doc files in the new spec file. What sort of compiler flags should I be adding? When you speak about the test suite, are you referring to rpmlint?
> What sort of compiler flags should I be adding? I've specifically linked to: https://fedoraproject.org/wiki/Packaging:Guidelines#Compiler_flags > are you referring to rpmlint? No, "make check" (or "make test") look suitable for a %check section.
New SPEC file: http://xiashing.fedorapeople.org/SPECS/pcalc.spec New SRPM file: http://xiashing.fedorapeople.org/SRPMS/pcalc-2-4.fc12.src.rpm I've added the new %check directive to incorporate the test suite. I didn't realize that the Makefile for pcalc had testing enabled. The $RPM_OPT_FLAGS compiler flags are located under the %prep directive.
> The $RPM_OPT_FLAGS compiler flags are located under the %prep directive. That's the wrong place. Hint: Test your changes, and a take look at the build output.
New SPEC file: http://xiashing.fedorapeople.org/SPECS/pcalc.spec New SRPM file: http://xiashing.fedorapeople.org/SRPMS/pcalc-2-5.fc12.src.rpm Ok, I've moved the compiler flags to the %build section now.
This message is a reminder that Fedora 12 is nearing its end of life. Approximately 30 (thirty) days from now Fedora will stop maintaining and issuing updates for Fedora 12. It is Fedora's policy to close all bug reports from releases that are no longer maintained. At that time this bug will be closed as WONTFIX if it remains open with a Fedora 'version' of '12'. Package Maintainer: If you wish for this bug to remain open because you plan to fix it in a currently maintained version, simply change the 'version' to a later Fedora version prior to Fedora 12's end of life. Bug Reporter: Thank you for reporting this issue and we are sorry that we may not be able to fix it before Fedora 12 is end of life. If you would still like to see this bug fixed and are able to reproduce it against a later version of Fedora please change the 'version' of this bug to the applicable version. If you are unable to change the version, please add a comment here and someone will do it for you. Although we aim to fix as many bugs as possible during every release's lifetime, sometimes those efforts are overtaken by events. Often a more recent Fedora release includes newer upstream software that fixes bugs or makes them obsolete. The process we are following is described here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping
You need to escape "%build" into Changelog section (add one more "%").
Sorry, read as "in the Cnagelog section".
Somehow this was filed with Version set to F12, while review tickets should always be set to rawhide. That caused the bugzapper script to comment on the ticket in error. Review tickets don't expire unless the submitter fails to respond. Fixing the version field so that this doesn't happen again.
In addition to Dmitrij's comment above which still needs to be addressed, could you comment on where you see that this software is under GPLv2? I can't see anything execpt the COPYING file (which does not imply that the software is GPLv2) and the pcalc.c and pcalc.h files, which are GPLv3+ with the special exception which doesn't impose GPLv3+ on the whole work.
Hi guys, I'd like to see this package in Fedora repos. What do i need to do? Regards, Cornel
> I'd like to see this package in Fedora repos. What do i need to do? https://docs.fedoraproject.org/en-US/package-maintainers/Package_Review_Process/