Red Hat Bugzilla – Bug 622630
Review Request: pcalc - A program for calculating and converting between different bases
Last modified: 2013-01-10 01:08:19 EST
pcalc is a program for calculating and converting between
There is an updated spec file and SRPM to fix some rpmlint errors.
* A more concise "Summary" can be found directly in the README file:
Summary: Programmer's calculator, command line utility
* Have you noticed the test suite?
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.
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:
> are you referring to rpmlint?
No, "make check" (or "make test") look suitable for a %check section.
New SPEC file:
New SRPM file:
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:
New SRPM file:
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:
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.