Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: alsa-patch-bay - Simple GUI for ALSA sequencers
https://bugzilla.redhat.com/show_bug.cgi?id=597596
Summary: Review Request: alsa-patch-bay - Simple GUI for ALSA sequencers Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: MostAwesomeDude@gmail.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://people.freedesktop.org/~csimpson/alsa-patch-bay.spec SRPM URL: http://people.freedesktop.org/~csimpson/alsa-patch-bay-1.0.0-1.fc12.src.rpm Description: alsa-patch-bay is a simple, plain GUI tool for patching together ALSA sequencers.
I found this tool during the Planet CCRMA days and fell in love with it, but it hasn't been maintained. I've got a github (http://github.com/MostAwesomeDude/alsa-patch-bay) with all the patches, and I'm waiting for upstream to get back to me on this.
My SPEC file is probably pretty iffy; I am very new to RPM as opposed to DEB, and auto-br-rpmbuild appears to have been a bit too eager in picking deps. Additionally, this package can be built against GTKMM and JACK in a mix'n'match; I should depend on (GTKMM || FLTK) && (ALSA || JACK) instead of what I've got now.
Thanks!
~ C.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=597596
Corbin Simpson MostAwesomeDude@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |177841(FE-NEEDSPONSOR)
--- Comment #1 from Corbin Simpson MostAwesomeDude@gmail.com 2010-05-29 14:07:29 EDT --- I should add that this is my first package review request in Fedora, and that I am looking for a sponsor, per https://fedoraproject.org/wiki/PackageMaintainers/Join .
~ C.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=597596
--- Comment #2 from Corbin Simpson MostAwesomeDude@gmail.com 2010-05-29 16:38:31 EDT --- I read more docs and did some rpmlint. I have the unstripped binary issue, and also the no-documentation issue. These are all warnings, but I should fix them once I know how. Incidentally, no documentation exists on these.
I also actually tried installing this RPM, and it appears to work. I'm kind of amazed, actually. Anyway, links to the new stuff:
SPEC: http://people.freedesktop.org/~csimpson/alsa-patch-bay.spec RPM: http://people.freedesktop.org/~csimpson/alsa-patch-bay-1.0.0-2.fc12.i386.rpm SRPM: http://people.freedesktop.org/~csimpson/alsa-patch-bay-1.0.0-2.fc12.src.rpm
~ C.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=597596
--- Comment #3 from Thomas Spura tomspur@fedoraproject.org 2010-05-29 17:48:31 EDT --- *** Bug 597592 has been marked as a duplicate of this bug. ***
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=597596
--- Comment #4 from Corbin Simpson MostAwesomeDude@gmail.com 2010-05-29 19:02:07 EDT --- And now it builds on koji! Amazing what can get done when there's a take-home final sitting above one's head. Links:
SPEC: http://people.freedesktop.org/~csimpson/alsa-patch-bay.spec SRPM: http://people.freedesktop.org/~csimpson/alsa-patch-bay-1.0.0-3.fc12.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=597596
--- Comment #5 from Jason Tibbitts tibbs@math.uh.edu 2010-11-19 15:05:09 EST --- Just taking a look at some older review tickets.
This one builds but fails to install:
Error: Package: alsa-patch-bay-1.0.0-3.fc15.x86_64 (/alsa-patch-bay-1.0.0-3.fc15.x86_64) Requires: fltk.so.0
Any reason why you specify all of the library dependencies manually instead of letting rpm figure them out for you? You really should never need to do anything remotely like this:
Requires: alsa.so.0 fltk.so.0 libasound.so.2 libasound.so.2(ALSA_0.9) libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) libdl.so.2 libdl.so.2(GLIBC_2.0) libdl.so.2(GLIBC_2.1) libfltk.so.1.1 libgcc_s.so.1 libgcc_s.so.1(GCC_3.0) libm.so.6 libpthread.so.0 libpthread.so.0(GLIBC_2.0) libpthread.so.0(GLIBC_2.1) libstdc++.so.6 libstdc++.so.6(CXXABI_1.3) libstdc++.so.6(CXXABI_1.3.1) libstdc++.so.6(GLIBCXX_3.4) libstdc++.so.6(GLIBCXX_3.4.11) libstdc++.so.6(GLIBCXX_3.4.9) rtld(GNU_HASH)
rpm will happily figure out all of those (and, indeed, the currect ones) for itself.
We don't usually usually ship libtool archives unless there's some specific reason to do so. Do things break if you remove the two .la files?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=597596
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status Whiteboard| |StalledSubmitter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=597596
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Blocks|177841(FE-NEEDSPONSOR) |201449(FE-DEADREVIEW) Resolution| |NOTABUG Status Whiteboard|StalledSubmitter | Last Closed| |2011-01-19 17:50:30
--- Comment #6 from Jason Tibbitts tibbs@math.uh.edu 2011-01-19 17:50:30 EST --- No response in two months; closing.
package-review@lists.fedoraproject.org
-
Red Hat Bugzilla