Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 202004 - Review Request: brandy
Summary: Review Request: brandy
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-08-10 10:33 UTC by Paul F. Johnson
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-08-17 22:32:44 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Paul F. Johnson 2006-08-10 10:33:47 UTC
Spec URL: http://www.knox.net.nz/~nodoid/brandy.spec
SRPM URL: http://www.knox.net.nz/~nodoid/brandy-1.0.19-1.src.rpm
Description: 

Brandy is an open source, GPL implementation of BBC BASIC V. It comes with documentation and example in %{_docdir}.

I've not split the package into main binary and doc package as I'm not sure it warrants it!

Comment 1 Jason Tibbitts 2006-08-13 04:24:18 UTC
This package looks good and builds and runs fine, but you seem to leave out the
documentation and the examples, save for COPYING and READ.ME.  It looks liks
stuff  manually copied into _docdir doesn't get added to the package, yet you
can't manually list _docdir in %files.

Comment 2 Paul F. Johnson 2006-08-13 21:12:51 UTC
Spec URL: http://www.knox.net.nz/~nodoid/brandy.spec
SRPM URL: http://www.knox.net.nz/~nodoid/brandy-1.0.19-2.src.rpm

Fixes docdir problem and examples problem


Comment 3 Jason Tibbitts 2006-08-15 17:58:57 UTC
Finally my mirror is updated and I can build again.

This still builds fine and indeed all of the documentation and examples are
there.  However, there are a couple of issues:

Proper flags are not pased to the compiler.  I use this hack at the end of %prep
to get them passed properly; the resulting package still seems to work correctly:

perl -pi -e "s/^(CFLAGS.*=.*)/\1 %{optflags}/" makefile

I wonder if the examples would be more proper as documentation.  This would be
more in line with what I've seen in the past, but I don't think it's a blocker.

* source files match upstream:
   0aedef51e76cf07533d82fe4dcd89efa  brandy_119.tgz
* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible.  License text included in package.
* latest version is being packaged.
* BuildRequires are proper (none!)
X compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, x86_64).
* debuginfo package looks complete (even though the compiler flags are wrong, -g
is still passed)
* rpmlint is silent.
* final provides and requires are sane:
   brandy = 1.0.19-2.fc6
  =
   (no non-glibc or rpm dependencies)
* %check is not present; no test suite upstream.  Manual testing shows that
things at least install and run.  (My BASIC is not the best after a couple of
decades of disuse.)
* no shared libraries are present.
* package is not relocatable.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no libtool .la droppings.

Comment 4 Paul F. Johnson 2006-08-16 21:17:31 UTC
Spec URL: http://www.knox.net.nz/~nodoid/brandy.spec

Updated the spec file (the src.rpm is the same)

Adds in the perl hack.

Comment 5 Jason Tibbitts 2006-08-17 03:52:14 UTC
Looks good to me.  The CFLAGS hack is kind of nasty but to my eyes it produces a
proper set of flags and the resulting package is fine.

I'm going to go ahead and approve this; I didn't get much guidance on the
subject of whether the examples should be included as %doc (instead of living in
%_datadir) so I'm not going to block on it.

APPROVED

Comment 6 Toshio Kuratomi 2006-08-17 19:16:59 UTC
Is there a justification for examples in %{_datadir}?  I see that you're even
copying them yourself....

Comment 7 Paul F. Johnson 2006-08-17 22:32:26 UTC
Where would be a better place that in %{_datadir}/%{name}-%{version}/examples? 

Comment 8 Toshio Kuratomi 2006-08-17 22:53:23 UTC
Why not::
%doc examples



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