Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
Summary: Review Request: hydrogen - Advanced drum machine Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: bugzilla-sink@leemhuis.info ReportedBy: green@redhat.com QAContact: fedora-package-review@redhat.com
Spec URL: http://people.redhat.com/green/FE/FC5/hydrogen.spec SRPM URL: http://people.redhat.com/green/FE/FC5/hydrogen-0.9.3-1.src.rpm Description: Hydrogen is an advanced drum machine for GNU/Linux. It's main goal is to bring professional yet simple and intuitive pattern-based drum programming.
This package depends on jack-audio-connection-kit and liblrdf, both of which have been submitted to Extras recently.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
green@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |177841 nThis| |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
green@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO| |183912, 189313 nThis| |
------- Additional Comments From green@redhat.com 2006-05-13 12:33 EST ------- I've fixed an x86-64 install issue. Updated bits are here...
Spec URL: http://people.redhat.com/green/FE/FC5/hydrogen.spec SRPM URL: http://people.redhat.com/green/FE/FC5/hydrogen-0.9.3-2.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
green@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- BugsThisDependsOn| |183912, 189313 OtherBugsDependingO|183912, 189313 | nThis| |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From green@redhat.com 2006-05-13 17:17 EST ------- On x86-64, read ladspa plugins from lib64 dirs...
Spec URL: http://people.redhat.com/green/FE/FC5/hydrogen.spec SRPM URL: http://people.redhat.com/green/FE/FC5/hydrogen-0.9.3-3.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
bugs.michael@gmx.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |bugs.michael@gmx.net OtherBugsDependingO|177841 | nThis| |
------- Additional Comments From bugs.michael@gmx.net 2006-05-13 17:41 EST ------- [wrt 0.9.3-2]
* FE-NEEDSPONSOR blocker bug is for new contributors who don't have CVS access yet. You don't belong to that group anymore.
* Fails to build (FC5), due to a hardcoded -lxml2 in configure{.in}.
/usr/bin/ld: cannot find -lxml2 collect2: ld returned 1 exit status
Missing: BuildRequires: libxml2-devel
* From the build log:
| --- Checking for PortAudio ------------------------------------- | checking whether PORTAUDIOPATH environment variable is set... | PORTAUDIOPATH is not set. No PortAudio support. | -----------------------------------------------------------------
PortAudio is available in Extras. Should it be enabled?
* rpmlint hydrogen-0.9.3-2.i386.rpm E: hydrogen shared-lib-without-dependency-information /usr/lib/hydrogen/plugins/wasp_noisifier.so E: hydrogen shared-lib-without-dependency-information /usr/lib/hydrogen/plugins/wasp_booster.so E: hydrogen shared-lib-without-dependency-information /usr/lib/hydrogen/plugins/wasp_xshaper.so E: hydrogen script-without-shellbang /usr/share/applications/hydrogen.desktop
The first three are safe to ignore. The latter is not. .desktop file is executable. The review guidelines say it must be installed with desktop-file-install.
* The explicit Requires on package names are not good. Kill them and rely on rpmbuild's automatic dependencies on SONAMES.
* The hardcoded Qt version is not good. Source /etc/profile.d/qt.sh which sets QTDIR correctly. Further, since many Qt configure checks fail due to multilib locations, set QTLIB/QTINC like this:
--- hydrogen.spec.orig 2006-05-13 18:31:34.000000000 +0200 +++ hydrogen.spec 2006-05-13 22:58:46.000000000 +0200 @@ -11,8 +11,6 @@
BuildRequires: flac-devel jack-audio-connection-kit-devel liblrdf-devel BuildRequires: qt-devel libsndfile-devel alsa-lib-devel -Requires: flac jack-audio-connection-kit liblrdf -Requires: qt libsndfile alsa-lib
%description Hydrogen is an advanced drum machine for GNU/Linux. It's main goal is @@ -25,8 +23,10 @@ sed --in-place -e 's:$(prefix)/lib:%{_libdir}:g' plugins/wasp/Makefile.in
%build -%configure QTDIR=%{_libdir}/qt-3.3 -%{__make} %{?_smp_mflags} QTDIR=%{_libdir}/qt-3.3 +unset QTDIR || : ; . /etc/profile.d/qt.sh +export QTLIB=${QTDIR}/lib QTINC=${QTDIR}/include +%configure +%{__make} %{?_smp_mflags}
%install %{__rm} -rf $RPM_BUILD_ROOT
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From green@redhat.com 2006-05-13 19:50 EST ------- Thanks for the review. Updated bits here...
Spec URL: http://people.redhat.com/green/FE/FC5/hydrogen.spec SRPM URL: http://people.redhat.com/green/FE/FC5/hydrogen-0.9.3-4.src.rpm
(In reply to comment #3)
[wrt 0.9.3-2]
- FE-NEEDSPONSOR blocker bug is for new contributors who don't have CVS
access yet. You don't belong to that group anymore.
Ah.. I thought sponsorship applied to packages, not people. Thanks for clearing this up.
- Fails to build (FC5), due to a hardcoded -lxml2 in configure{.in}.
/usr/bin/ld: cannot find -lxml2 collect2: ld returned 1 exit status
Missing: BuildRequires: libxml2-devel
Fixed.
- From the build log:
| --- Checking for PortAudio ------------------------------------- | checking whether PORTAUDIOPATH environment variable is set... | PORTAUDIOPATH is not set. No PortAudio support. | -----------------------------------------------------------------
PortAudio is available in Extras. Should it be enabled?
No, I don't think so. jack is the audio driver of choice for these kinds of apps.
- rpmlint hydrogen-0.9.3-2.i386.rpm
E: hydrogen shared-lib-without-dependency-information /usr/lib/hydrogen/plugins/wasp_noisifier.so E: hydrogen shared-lib-without-dependency-information /usr/lib/hydrogen/plugins/wasp_booster.so E: hydrogen shared-lib-without-dependency-information /usr/lib/hydrogen/plugins/wasp_xshaper.so E: hydrogen script-without-shellbang /usr/share/applications/hydrogen.desktop
The first three are safe to ignore. The latter is not. .desktop file is executable. The review guidelines say it must be installed with desktop-file-install.
Fixed. I've also added related post and postun scripts.
- The explicit Requires on package names are not good. Kill them and
rely on rpmbuild's automatic dependencies on SONAMES.
Fixed.
- The hardcoded Qt version is not good. Source /etc/profile.d/qt.sh
which sets QTDIR correctly. Further, since many Qt configure checks fail due to multilib locations, set QTLIB/QTINC like this:
Cool. Fixed.
Thanks for spending time on this!
AG
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
Bug 190040 depends on bug 189313, which changed state.
Bug 189313 Summary: Review Request: liblrdf https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=189313
What |Old Value |New Value ---------------------------------------------------------------------------- Resolution| |NEXTRELEASE Status|ASSIGNED |CLOSED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From seg@haxxed.com 2006-05-15 15:24 EST ------- Yeah, portaudio is redundant, and OSS is deprecated. One thing I've noticed about hydrogen is it eats quite a bit of RAM, and then jack wants to lock it. If the memlock rlimit isn't set to around 128mb, it will crash when used with jack with mem locking enabled.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From seg@haxxed.com 2006-05-15 16:07 EST ------- (noted in more detail on the jack review)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
seg@haxxed.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bugzilla-sink@leemhuis.info |seg@haxxed.com
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
seg@haxxed.com changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163776 |163778 nThis| |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From seg@haxxed.com 2006-05-21 04:34 EST ------- Michael: You have dibs on this review, but you hadn't assigned it to yourself. I do want to work on this tommorow. Sorry if I'm stepping on your toes. Reassign it if you want it?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From seg@haxxed.com 2006-05-23 05:25 EST ------- MUST items:
- rpmlint: Ok?
$ rpmlint hydrogen-0.9.3-4.fc5.i386.rpm E: hydrogen shared-lib-without-dependency-information /usr/lib/hydrogen/plugins/wasp_xshaper.so E: hydrogen shared-lib-without-dependency-information /usr/lib/hydrogen/plugins/wasp_noisifier.so E: hydrogen shared-lib-without-dependency-information /usr/lib/hydrogen/plugins/wasp_booster.so
$ ldd /usr/lib/hydrogen/plugins/* /usr/lib/hydrogen/plugins/wasp_booster.so: statically linked /usr/lib/hydrogen/plugins/wasp_noisifier.so: statically linked /usr/lib/hydrogen/plugins/wasp_xshaper.so: statically linked
Statically linked dynamic libraries? That's a new one to me. These are apparently LADSPA plugins. Perhaps these should go in %{_libdir}/ladspa instead. And even go in a "hydrogen-wasp-plugins" subpackage.
- Package name: Ok - Spec name: Ok - Meets packaging guidelines: Ok - License: Ok - Spec in American English: Ok - Spec legible: Ok - Sources match upstream: Ok - Builds: Ok - BuildRequires: Ok - Locales: Ok - ldconfig: NEEDSWORK - Relocation: Ok - Directory ownership: Ok - %files: Ok - %clean: Ok - Macros: Ok - Code vs. Content: Ok - Documentation: Ok - devel package: Ok - .desktop file: Ok
SHOULD:
- Includes license text: Ok - Mock build: Yes - Builds on all archs: Built on i386, x86_64 - Package functional: Yes! http://www.haxxed.com/music/909fun.ogg - Scriptlets: NEEDSWORK - Subpackages: Ok
NEEDSWORK:
Source0 should be a full URL. It should be http://dl.sf.net/sourceforge/hydrogen/hydrogen-0.9.3.tar.gz
You have a lingering buildreq on portaudio-devel.
I would recommend disabling OSS support. (%configure --disable-oss-support) AFAIK OSS has been deprecated for some time now.
I don't think you need all that QTDIR stuff. It seems to build just fine without it. Mock sources profile.d properly.
I don't think you need to update the icon cache, its not installing any into /usr/share/icons
Don't need ldconfig, its not installing systemwide libraries.
The update-desktop-database doesn't match what's in ScriptletSnippets, and according to ScriptletSnippets, you only need it if there's a MimeType key.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From hugo@devin.com.br 2006-05-23 10:21 EST ------- (In reply to comment #8)
Source0 should be a full URL. It should be http://dl.sf.net/sourceforge/hydrogen/hydrogen-0.9.3.tar.gz
Discussing this in IRC some people realized that dl.sf.net is not always available (as it is a dns round-robin load-balance). In experience, some said that the easynews mirror never gets trouble so using it is better in general. I'm using it on some specs, so maybe you should consider using it too.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
Bug 190040 depends on bug 183912, which changed state.
Bug 183912 Summary: Review Request: jack-audio-connection-kit - The Jack Audio Connection Kit https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=183912
What |Old Value |New Value ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Resolution| |NEXTRELEASE Status|ASSIGNED |CLOSED
------- Additional Comments From nightbox@hotmail.com 2006-06-26 12:50 EST ------- So, is this going to be included any time soon?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From green@redhat.com 2006-06-26 14:22 EST ------- (In reply to comment #10)
So, is this going to be included any time soon?
I'll work on it this week. Thanks for the reminder. I forgot that I had received feedback.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From green@redhat.com 2006-07-02 21:41 EST ------- (In reply to comment #8)
NEEDSWORK:
Source0 should be a full URL. It should be http://dl.sf.net/sourceforge/hydrogen/hydrogen-0.9.3.tar.gz
I've updated this to the easynews link.
You have a lingering buildreq on portaudio-devel.
Fixed.
I would recommend disabling OSS support. (%configure --disable-oss-support) AFAIK OSS has been deprecated for some time now.
Fixed.
I don't think you need all that QTDIR stuff. It seems to build just fine without it. Mock sources profile.d properly.
I think all that stuff is a good idea in order to ensure a reproducable build outside of mock.
I don't think you need to update the icon cache, its not installing any into /usr/share/icons
Fixed.
Don't need ldconfig, its not installing systemwide libraries.
Fixed.
The update-desktop-database doesn't match what's in ScriptletSnippets, and according to ScriptletSnippets, you only need it if there's a MimeType key.
Fixed (it does have a MimeType key).
Here are the updated bits:
Spec URL: http://people.redhat.com/green/FE/FC5/hydrogen.spec SRPM URL: http://people.redhat.com/green/FE/FC5/hydrogen-0.9.3-5.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From green@redhat.com 2006-07-22 20:02 EST ------- Updated bits...
Spec URL: http://people.redhat.com/green/FE/FC5/hydrogen.spec SRPM URL: http://people.redhat.com/green/FE/FC5/hydrogen-0.9.3-6.src.rpm
This just adds a patch for a crasher that I will submit upstream now as well.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From opensource@till.name 2006-09-12 16:25 EST ------- You should not convert the icon in your spec file. The different sizes are only needed if they are manually edited in the other sizes to look better. This simple resizing is done automatically by a window manager if needed.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From opensource@till.name 2006-09-12 16:27 EST ------- Please ignore my last comment it was addressed to another review request.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From green@redhat.com 2007-03-17 16:51 EST ------- Hi seg - I thought we were very close to wrapping this one up. Could you please review the last package? Thanks! AG
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From opensource@till.name 2007-03-26 00:19 EST -------
(In reply to comment #12)
(In reply to comment #8)
NEEDSWORK:
Source0 should be a full URL. It should be http://dl.sf.net/sourceforge/hydrogen/hydrogen-0.9.3.tar.gz
I've updated this to the easynews link.
Recommended is: http://downloads.sourceforge.net/%%7Bname%7D/%%7Bname%7D-%%7Bversion%7D.tar.... see: http://fedoraproject.org/wiki/Packaging/SourceURL#head-e27982f18a3bfd26b5b6e...
The update-desktop-database doesn't match what's in ScriptletSnippets, and according to ScriptletSnippets, you only need it if there's a MimeType key.
Fixed (it does have a MimeType key).
Here are the updated bits:
Spec URL: http://people.redhat.com/green/FE/FC5/hydrogen.spec SRPM URL: http://people.redhat.com/green/FE/FC5/hydrogen-0.9.3-5.src.rpm
The changelog says: - Remove post/postun scriptlets.
Because the .desktop files MimeType entry you need the scriptlets from: http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#head-de6770dd9867f...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From green@redhat.com 2007-03-26 12:25 EST ------- (In reply to comment #17)
Recommended is: http://downloads.sourceforge.net/%%7Bname%7D/%%7Bname%7D-%%7Bversion%7D.tar.... see:
http://fedoraproject.org/wiki/Packaging/SourceURL#head-e27982f18a3bfd26b5b6e...
Fixed.
The changelog says:
- Remove post/postun scriptlets.
I don't know why I did this. I've added these with update-desktop-database and gtk-update-icon-cache,
Because the .desktop files MimeType entry you need the scriptlets from:
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#head-de6770dd9867f...
Done.
New bits here: http://people.redhat.com/green/FE/devel/hydrogen-0.9.3-7.src.rpm http://people.redhat.com/green/FE/devel/hydrogen.spec
Thanks!
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From opensource@till.name 2007-03-26 16:36 EST ------- (In reply to comment #18)
The changelog says:
- Remove post/postun scriptlets.
I don't know why I did this. I've added these with update-desktop-database and gtk-update-icon-cache,
Maybe you removed the gtk-update-icon-cache stuff, because you need only update-desktop-database, because the icons are not in the cached directories.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium Priority|normal |medium
tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-review?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From lmacken@redhat.com 2007-06-30 21:03 EST ------- Trying to build this on F7...
src/lib/FLACFile.cpp: In member function 'void FLACFile_real::load(std::string)': src/lib/FLACFile.cpp:167: error: 'set_filename' was not declared in this scope src/lib/FLACFile.cpp:169: error: no matching function for call to 'FLACFile_real::init()' /usr/include/FLAC++/decoder.h:226: note: candidates are: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(FILE*) /usr/include/FLAC++/decoder.h:227: note: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(const char*) /usr/include/FLAC++/decoder.h:228: note: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(const std::string&) src/lib/FLACFile.cpp:170: error: 'FLAC__FILE_DECODER_OK' was not declared in this scope src/lib/FLACFile.cpp:174: error: 'process_until_end_of_file' was not declared in this scope make[1]: *** [src/FLACFile.o] Error 1 make[1]: Leaving directory `/home/lmacken/rpmbuild/BUILD/hydrogen-0.9.3' make: *** [hydrogenPlayer] Error 2 make: *** Waiting for unfinished jobs.... src/lib/FLACFile.cpp: In member function 'void FLACFile_real::load(std::string)': src/lib/FLACFile.cpp:167: error: 'set_filename' was not declared in this scope src/lib/FLACFile.cpp:169: error: no matching function for call to 'FLACFile_real::init()' /usr/include/FLAC++/decoder.h:226: note: candidates are: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(FILE*) /usr/include/FLAC++/decoder.h:227: note: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(const char*) /usr/include/FLAC++/decoder.h:228: note: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(const std::string&) src/lib/FLACFile.cpp:170: error: 'FLAC__FILE_DECODER_OK' was not declared in this scope src/lib/FLACFile.cpp:174: error: 'process_until_end_of_file' was not declared in this scope make[1]: *** [src/FLACFile.o] Error 1 make[1]: Leaving directory `/home/lmacken/rpmbuild/BUILD/hydrogen-0.9.3' make: *** [hydrogen] Error 2 error: Bad exit status from /var/tmp/rpm-tmp.93730 (%build)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
cr33dog@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |cr33dog@gmail.com
------- Additional Comments From cr33dog@gmail.com 2007-07-10 02:13 EST ------- I hit the bump in #20 also. I did not know this was up for review and was building from source - I wandered in from google. The meat of this patch will get past the FLAC error: http://cvs.archlinux.org/cgi-bin/viewcvs.cgi/multimedia/hydrogen/hydrogen-0....
It needs some work :)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040
------- Additional Comments From lmacken@redhat.com 2007-08-22 04:12 EST ------- I'm getting this build error with that patch:
src/lib/FLACFile.cpp: In member function 'void FLACFile_real::load(std::string)': src/lib/FLACFile.cpp:167: error: 'set_filename' was not declared in this scope src/lib/FLACFile.cpp:169: error: no matching function for call to 'FLACFile_real::init()' /usr/include/FLAC++/decoder.h:226: note: candidates are: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(FILE*) /usr/include/FLAC++/decoder.h:227: note: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(const char*) /usr/include/FLAC++/decoder.h:228: note: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(const std::string&) src/lib/FLACFile.cpp:170: error: 'FLAC__FILE_DECODER_OK' was not declared in this scope src/lib/FLACFile.cpp:174: error: 'process_until_end_of_file' was not declared in this scope make[1]: *** [src/FLACFile.o] Error 1
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
matthias@rpmforge.net changed:
What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163778 | nThis| | Status|NEW |ASSIGNED
------- Additional Comments From matthias@rpmforge.net 2007-09-01 12:06 EST ------- Removing old FE-REVIEW blocker and changed from NEW to ASSIGNED since the review flag is set.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
j.w.r.degoede@hhs.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |j.w.r.degoede@hhs.nl
------- Additional Comments From j.w.r.degoede@hhs.nl 2007-10-03 18:06 EST ------- I noticed that this seems kinda stuck and I would like to offer to help.
The patch from comment #21 looks good, I haven't tried it, but it should work, If it doesn't then the used define probably isn't getting set correctly.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-06 04:34 EST ------- Luke: for me the patch works. What version of flac and flac-devel packages do you have?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-06 04:43 EST ------- Another thing:
# Conditionally apply patch to read ladspa plugins from lib64 dir. %ifarch x86_64 %patch1 -p0 %patch2 -p0 %endif
Please never do this. Never make arch-specific patches: http://www.redhat.com/archives/fedora-devel-list/2007-September/msg01518.htm...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-06 04:48 EST ------- The License: tag is also not correct. GPLv2+ is the correct license here.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-06 04:49 EST ------- The patch names should mention the version they were created against, like name-version-description.patch
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-06 05:30 EST ------- This happens in hydrogen-0.9.3/plugins/wasp during the build
ld -shared -o wasp_booster.so plugins/booster.o ld -shared -o wasp_noisifier.so plugins/noisifier.o ld -shared -o wasp_xshaper.so plugins/xshaper.o
linking this way doesn't honor the Build id http://fedoraproject.org/wiki/Releases/FeatureBuildId
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-06 05:32 EST ------- [root@gopher SPECS]# rpm -Uvh /home/lkundrak/rpmbuild/RPMS/i386/hydrogen-0.9.3-7.1.fc8.i386.rpm Preparing... ########################################### [100%] 1:hydrogen ########################## ( 60%) ########################################### [100%] /var/tmp/rpm-tmp.60215: line 1: 29061 Segmentation fault (core dumped) update-desktop-database >&/dev/null [root@gopher SPECS]#
Ayeeee :)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-06 05:40 EST ------- The above is most likely not hydrogen's fault, I'll investigate that later... after the weekend. Below are patches I had to apply to hydrogen to get it built:
http://people.redhat.com/lkundrak/patches/hydrogen/
I did not address all the issues mentioned above; just enough to get that built. Here's the SPEC file:
http://people.redhat.com/lkundrak/SPECS/hydrogen.spec
Anthony: Are you still interested in finishing the package? Apart from the problems I made fixes for, are there any other outstanding problems?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-06 08:03 EST ------- g++ -c -pipe -g -w -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic -fasynchronous-unwind-tables -D_REENTRANT -DQT_NO_DEBUG -DQT_THREAD_SUPPORT -DQT_SHARED -DQT_TABLET_SUPPORT -I/usr/lib/qt-3.3/mkspecs/default -I. -I. -Isrc -I/usr/lib/qt-3.3/include -o src/Button.o src/gui/widgets/Button.cpp src/AlsaMidiDriver.o: In function `alsaMidiDriver_thread(void*)': /builddir/build/BUILD/hydrogen-0.9.3/src/lib/drivers/AlsaMidiDriver.cpp:80: undefined reference to `Preferences::getInstance()' src/DiskWriterDriver.o: In function `DiskWriterDriver::getSampleRate()': /builddir/build/BUILD/hydrogen-0.9.3/src/lib/drivers/DiskWriterDriver.cpp:162: undefined reference to `Preferences::getInstance()' src/DiskWriterDriver.o: In function `diskWriterDriver_thread(void*)': /builddir/build/BUILD/hydrogen-0.9.3/src/lib/drivers/DiskWriterDriver.cpp:40: undefined reference to `Preferences::getInstance()' src/JackDriver.o: In function `JackDriver::init(unsigned int)': /builddir/build/BUILD/hydrogen-0.9.3/src/lib/drivers/JackDriver.cpp:242: undefined reference to `Preferences::getInstance()' /builddir/build/BUILD/hydrogen-0.9.3/src/lib/drivers/JackDriver.cpp:243: undefined reference to `Preferences::getInstance()' src/JackDriver.o:/builddir/build/BUILD/hydrogen-0.9.3/src/lib/drivers/JackDriver.cpp:392: more undefined references to `Preferences::getInstance()' follow collect2: ld returned 1 exit status make[1]: *** [hydrogenPlayer] Error 1 make[1]: Leaving directory `/builddir/build/BUILD/hydrogen-0.9.3' make: *** [hydrogenPlayer] Error 2 make: *** Waiting for unfinished jobs....
In mock for FC7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From green@redhat.com 2007-10-06 09:25 EST ------- (In reply to comment #31)
Anthony: Are you still interested in finishing the package? Apart from the problems I made fixes for, are there any other outstanding problems?
I'm interested, but don't have time now. Could somebody else pick this up?
Thanks!
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-07 15:35 EST ------- I've made a couple more fixes. Here are the new files:
Spec URL: http://people.redhat.com/lkundrak/SPECS/hydrogen.spec SRPM URL: http://people.redhat.com/lkundrak/mock-results/hydrogen-0.9.3-8.fc8.i386/hyd...
The mock result with binary package and build log is here: http://people.redhat.com/lkundrak/mock-results/hydrogen-0.9.3-8.fc8.i386/
The package builds correctly in mock in Fedora 7 and Rawhide on i386. rpmlint is all silent and I am not aware of any outstanding issues. Callum: could you please review the new packages?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
j.w.r.degoede@hhs.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|seg@haxxed.com |j.w.r.degoede@hhs.nl
------- Additional Comments From j.w.r.degoede@hhs.nl 2007-10-08 10:45 EST ------- Taking over as reviewer, since Callum is very busy with other stuff ATM AFAIK. I hope you don't mind Callum.
Full review done (sources match upstream, license ok, everything else also checked), I've found a few items to fix:
MUST FIX -------- - You currently pass LIBDIR=%{_libdir} to make but not to make install! I couldn't test this at a 64 bit machine atm, but either you should use it in both places or not all - Remove these obsolete or unneeded options when installing the .desktop file: --add-category X-Fedora \ --add-category AudioVideo \ --add-category Application \ - Add the following to remove the obsolete Application category that is in upstreams .desktop file: --remove-category Application \ - Remove "MimeType=text/xml" from hydrogen.desktop, thats way too generic! - Remove obsolete "Version=1.0" from hydrogen.desktop - No longer run update-desktop-database from %post[un] now the .desktop file no longer defines a MimeType - The update icon cache scriptlets should check the existence of gtk-update-icon-cache before calling it, see: http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#head-7103f6c38d1b5...
SHOULD FIX ---------- - Add the following Categories from: http://standards.freedesktop.org/menu-spec/latest/apa.html : Midi - Add the following X-foo categories, to mathc what CCRMA has so that this package will work with CCRMA's nested audio utilities menus: X-Drumming X-MIDI X-Jack
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
j.w.r.degoede@hhs.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |nando@ccrma.stanford.edu
------- Additional Comments From j.w.r.degoede@hhs.nl 2007-10-08 10:46 EST ------- Fernando, this is moving to Fedora too, I guess you want to know, and maybe take a quick peek for any errors.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-08 11:44 EST ------- Much thanks for the comments!
- You currently pass LIBDIR=%{_libdir} to make but not to make install! I couldn't test this at a 64 bit machine atm, but either you should use it in both places or not all
That is no longer needed, I just forgot it there. Makefiles use ${libdir} from autoconf and set -DLIBDIR for the ladspa plugins path via .pro files.
- Remove these obsolete or unneeded options when installing the .desktop file: --add-category X-Fedora \ --add-category AudioVideo \ --add-category Application \
- Add the following to remove the obsolete Application category that is in upstreams .desktop file: --remove-category Application \
- Remove "MimeType=text/xml" from hydrogen.desktop, thats way too generic!
- Remove obsolete "Version=1.0" from hydrogen.desktop
- No longer run update-desktop-database from %post[un] now the .desktop file no longer defines a MimeType
- Add the following Categories from: http://standards.freedesktop.org/menu-spec/latest/apa.html : Midi
- Add the following X-foo categories, to mathc what CCRMA has so that this package will work with CCRMA's nested audio utilities menus: X-Drumming X-MIDI X-Jack
Changed the options to desktop-file-install, removed hydrogen-0.9.3-sound-category.patch patch.
- The update icon cache scriptlets should check the existence of gtk-update-icon-cache before calling it, see:
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#head-7103f6c38d1b5...
I've modified the scriptlets accordingly.
Here's the new package:
Spec URL: http://people.redhat.com/lkundrak/SPECS/hydrogen.spec SRPM URL: http://people.redhat.com/lkundrak/mock-results/hydrogen-0.9.3-9.fc8.i386/hyd... The mock build results: http://people.redhat.com/lkundrak/mock-results/hydrogen-0.9.3-9.fc8.i386/
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From nando@ccrma.stanford.edu 2007-10-08 13:16 EST ------- (In reply to comment #36)
Fernando, this is moving to Fedora too, I guess you want to know, and maybe take a quick peek for any errors.
_Thanks_ for the heads up. Looks fine so far (just took a quick look at the spec file), thanks for including the Planet CCRMA menu categories so that it meshes with the existing menus... one more for the migration!
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
j.w.r.degoede@hhs.nl changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From j.w.r.degoede@hhs.nl 2007-10-08 15:47 EST ------- Latest version looks fine: Approved!
Which brings us to the interesting question who will maintain it, may I suggest having both Anthony and Lubomir as owners / co-maintainers? And I think it would be good to have Fernando from CCRMA as co-maintainer too, for a grand total of 3 :) Note Fernando's account system name is nando.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
lkundrak@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From lkundrak@redhat.com 2007-10-08 16:28 EST ------- Hans: This will be CVS extras writable, so who'll be a formal maintainer is not an issue at all. Anyways, I do not think I would want to maintain this, but as Anthony says he doesn't have time at the time I'd be glad to help with package mainteance tasks for now.
New Package CVS Request ======================= Package Name: hydrogen Short Description: Advanced drum machine Owners: green,lkundrak,nando Branches: FC-6 F-7 Cvsextras Commits: yes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2007-10-09 00:18 EST ------- cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
------- Additional Comments From lkundrak@redhat.com 2007-10-09 02:16 EST ------- Thanks Antony, Michael, Callum, Hugo, Till, Hans, Fernando and Kevin. The package was imported into CVS and built.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
lkundrak@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|devel |rawhide
lkundrak@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs+ |fedora-cvs?
------- Additional Comments From lkundrak@redhat.com 2008-04-11 17:06 EST ------- Package Change Request ======================= Package Name: hydrogen Branches: EL-5 Cvsextras Commits: yes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: hydrogen - Advanced drum machine
https://bugzilla.redhat.com/show_bug.cgi?id=190040
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2008-04-12 16:41 EST ------- cvs done.
package-review@lists.fedoraproject.org