Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 622630 - Review Request: pcalc - A program for calculating and converting between different bases
Summary: Review Request: pcalc - A program for calculating and converting between diff...
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2010-08-10 00:22 UTC by Christopher X.S. Zee
Modified: 2013-01-10 06:08 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2012-06-29 22:28:33 UTC

Attachments (Terms of Use)

Description Christopher X.S. Zee 2010-08-10 00:22:20 UTC
Spec URL:

pcalc is a program for calculating and converting between
different bases.

Comment 1 Christopher X.S. Zee 2010-08-10 08:51:19 UTC
There is an updated spec file and SRPM to fix some rpmlint errors.

Spec URL:

Comment 2 Michael Schwendt 2010-08-16 20:57:38 UTC
* 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?

Comment 3 Christopher X.S. Zee 2010-08-17 03:02:19 UTC
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.



Comment 4 Michael Schwendt 2010-08-17 07:55:00 UTC
What about the other issues? The compiler flags, the test suite, and the missing %doc files?

Comment 5 Christopher X.S. Zee 2010-08-18 08:04:27 UTC
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?

Comment 6 Michael Schwendt 2010-08-18 10:17:37 UTC
> 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.

Comment 7 Christopher X.S. Zee 2010-08-18 11:16:38 UTC
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.

Comment 8 Michael Schwendt 2010-08-18 11:24:39 UTC
> 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.

Comment 9 Christopher X.S. Zee 2010-08-18 11:33:33 UTC
New SPEC file:

New SRPM file:

Ok, I've moved the compiler flags to the %build section now.

Comment 10 Bug Zapper 2010-11-03 11:06:10 UTC
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:

Comment 11 Dmitrij S. Kryzhevich 2010-11-03 11:45:06 UTC
You need to escape "%build" into Changelog section (add one more "%").

Comment 12 Dmitrij S. Kryzhevich 2010-11-03 11:46:37 UTC
Sorry, read as "in the Cnagelog section".

Comment 13 Jason Tibbitts 2010-11-03 12:34:08 UTC
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.

Comment 14 Jason Tibbitts 2010-12-02 04:37:38 UTC
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.

Note You need to log in before you can comment on or make changes to this bug.