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
Bug 187622 - Review Request: cowbell
Summary: Review Request: cowbell
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Brian Pepple
QA Contact: Fedora Package Reviews List
Depends On:
TreeView+ depends on / blocked
Reported: 2006-04-01 23:35 UTC by Sindre Pedersen Bjørdal
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2006-05-16 13:32:06 UTC
Type: ---

Attachments (Terms of Use)
spec patch (deleted)
2006-05-03 00:17 UTC, John Mahowald
no flags Details | Diff

Description Sindre Pedersen Bjørdal 2006-04-01 23:35:33 UTC
Spec Name or Url:
SRPM Name or Url:


Cowbell is an elegant, album-based, music organizer
written by Brad Taylor in C# for Gtk+. Using TagLib,
it supports many audio formats including: Mp3, Ogg
Vorbis and MusePack

Comment 1 John Mahowald 2006-04-02 18:32:04 UTC
Build failed on x86_64. 

File not found: /var/tmp/cowbell-

Apparently mono apps install to %{_prefix}/lib.

Need more cowbell!!!

Comment 2 Sindre Pedersen Bjørdal 2006-04-03 00:25:39 UTC
replaced %{_libdir} with %{_prefix}/lib/

hope this will fix x86_64 build issue.

Updated spec:

Comment 3 John Mahowald 2006-04-19 02:50:53 UTC
Builds now, but for some reason I cannot add anything to the database. Possibly
the latest 0.2.7 release fixes it.

Comment 4 John Mahowald 2006-05-03 00:14:16 UTC
0.2.7 builds but crashes when music is imported. Traced the problem to cowbell
looking for but on my system it is  Symlinking
it works, perhaps cowbell can be made to look there or glib2 made to provide a link.

Comment 5 John Mahowald 2006-05-03 00:17:42 UTC
Created attachment 128528 [details]
spec patch

Fairly trivial, bumps version to 0.2.7. Doesn't split of a devel subpackage,
which may be necessary.

Comment 6 Sindre Pedersen Bjørdal 2006-05-03 09:18:45 UTC
Updated spec:
Updated SRPM:

Does this build and run cleanly on x86_64 now?

Comment 7 Michael Schwendt 2006-05-03 13:55:51 UTC
It should look for (not which belongs
into the glib2-devel package).

Comment 8 Sindre Pedersen Bjørdal 2006-05-03 17:13:55 UTC
This has been pushed upstream, new release imminent. I'll update the spec
accordingly as soon as the new release is out.

Comment 9 Sindre Pedersen Bjørdal 2006-05-09 09:45:22 UTC
New release is out, this should fix the glib dependency. 

Updated spec:

Comment 10 Brian Pepple 2006-05-11 15:31:25 UTC
fcf32bcf60be2ca204519192eb26bacc  cowbell-

* Source URL is canonical
* Upstream source tarball verified
* Group Tag is from the official list
* Buildroot has all required elements
* All paths begin with macros
* All directories are owned by this or other packages
* All necessary BuildRequires listed.
* All desired features are enabled
* Make succeeds even when %{_smp_mflags} is defined
* Files have appropriate permissions and owners
* Package installs and uninstalls cleanly on FC5

* Missing scriptlets for GTK+ icon cache.
* Missing Requires for mono-core & gtk-sharp2

* Why package the static lib? Pass the --disable-static flag to configure, and
remove '%{_libdir}/cowbell/libtaglibsharpglue.a' from your files.
* Unnecessary BR on gettext.  This is included in the minimum build environment
in Mock.
* I would drop the ChangeLog from the docs, since the NEWS file basically
contains the same info.

Comment 11 Sindre Pedersen Bjørdal 2006-05-11 15:48:20 UTC
Updated spec:

- Add missing mono-core and gtk-sharp2 Requires
- Add scriplets for gtk+ icon cache
- Removed ChangeLog from %doc
- Don't include static library 
- Remove gettext BuildRequires

Comment 12 Brian Pepple 2006-05-11 22:05:04 UTC
Changes in response to the review in comment #10 look good.

* rpmlint is clean.


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