46568 2004-04-02 01:04 0000 hydrogen-0.8.2 compile errors 2004-07-21 14:32:14 0000 1 1 1 Unclassified Gentoo Linux Ebuilds unspecified All All RESOLVED FIXED P2 normal --- 54340 1 schnake.newsletter@t-online.de fvdpol@gentoo.org harold.martin@gmail.com hoek@linuxartist.org makerik@gmail.com sound@gentoo.org schnake.newsletter@t-online.de 2004-04-02 01:04:40 0000 Compile aborts with following error: source='AudioEngineInfoForm_UI.cpp' object='AudioEngineInfoForm_UI.o' libtool=no \ depfile='.deps/AudioEngineInfoForm_UI.Po' tmpdepfile='.deps/AudioEngineInfoForm_UI.TPo' \ depmode=gcc3 /bin/sh ../admin/depcomp \ g++ -DHAVE_CONFIG_H -I. -I. -I.. -I../lib -I/usr/qt/3/include -I/usr/X11R6/include -DQT_THREAD_SUPPORT -D_REENTRANT -O3 -c -o AudioEngineInfoForm_UI.o `test -f 'AudioEngineInfoForm_UI.cpp' || echo './'`AudioEngineInfoForm_UI.cpp AudioEngineInfoForm_UI.cpp: In constructor ` AudioEngineInfoForm_UI::AudioEngineInfoForm_UI(QWidget*, const char*, unsigned int)': AudioEngineInfoForm_UI.cpp:75: error: `Spacer2' undeclared (first use this function) AudioEngineInfoForm_UI.cpp:75: error: (Each undeclared identifier is reported only once for each function it appears in.) AudioEngineInfoForm_UI.cpp:80: error: `Spacer1' undeclared (first use this function) make[3]: *** [AudioEngineInfoForm_UI.o] Fehler 1 make[3]: Leaving directory `/var/tmp/portage/hydrogen-0.8.2/work/hydrogen-0.8.2/hydrogen_gui' make[2]: *** [all-recursive] Fehler 1 make[2]: Leaving directory `/var/tmp/portage/hydrogen-0.8.2/work/hydrogen-0.8.2/hydrogen_gui' make[1]: *** [all-recursive] Fehler 1 make[1]: Leaving directory `/var/tmp/portage/hydrogen-0.8.2/work/hydrogen-0.8.2' make: *** [all] Fehler 2 !!! ERROR: media-sound/hydrogen-0.8.2 failed. !!! Function src_compile, Line 72, Exitcode 2 !!! (no error message) After looking through the code the reason is two missing QSpacerItem type member declarations from the header. This also applies to LadspaFXSelector_UI.h and PreferencesDialog_UI.h schnake.newsletter@t-online.de 2004-04-02 01:11:59 0000 Created an attachment (id=28549) Fixes missing declarations from the hydrogen headers. To be placed into the files/ dir. schnake.newsletter@t-online.de 2004-04-02 01:16:47 0000 Created an attachment (id=28550) Adds epatch the ui-spacer patch and the missing addwrite call This includes the sandbox violation fix for bug #40347. eradicator@gentoo.org 2004-04-03 02:04:48 0000 torbenh, you're listed as maintainer in this package, so I'll hand this off to you. I removed some old ebuilds and added the qt addwrite to the remaining ones, but you should check out this patch to see if this is how you want to handle it. nlissne@linux01.gwdg.de 2004-04-04 15:11:13 0000 Michael, your patch to the ebuild (comment #2) contains a "#" before epatch, so nothing is patched - but after removing this "#" it works really fine. Thank you. schnake.newsletter@t-online.de 2004-04-05 00:51:42 0000 Oops, yes, I commented out the epatch line to reconstruct the exact error message for filing this bug, and accidently diffed agains that ebuild ;-) hoek@linuxartist.org 2004-04-11 21:53:59 0000 Hmm, this is very strange.. When patching the unpacked directory "by hand" and then running configure; make install, everything compiles ok.. but when i let the ebuild patch the sources, it's getting patched, but still runs into the same compile errors as if it wasn't (and yes, i *have* uncommented the epatch line.. =) the unpacked sources in the /var/tmp/portage/ directory *really* get patched (i even diffed the two directories, i didn't trust emerge saying it was patching it when i saw that it still produced the same error... but no, they're identical %\ can anyone reproduce this, or do i just need some sleep..? aent@invisiontools.com 2004-04-15 20:07:41 0000 I'm also having the problem, its not just you. schnake.newsletter@t-online.de 2004-04-16 01:31:53 0000 I just cleaned all and emerged again to check, and for me it worked without problems (using portage-2.0.50-r6). But I've found a related description of a "manual configure works, but econf does not"-Phenomenon at http://bugs.gentoo.org/show_bug.cgi?id=47329#c7 So I assume it's a problem with portage. Perhaps you should file a separate bug on this. eradicator@gentoo.org 2004-05-04 20:15:22 0000 setting WORKSFORME eldad@gentoo.org 2004-05-08 03:29:42 0000 I've got the same problem, on a clean system - same error message as the first comment. ONLY With the patch it compiles fine. dan.devel@bullok.com 2004-05-26 08:58:27 0000 What's the deal with this patch not being added to the ebuild? Is there something wrong with it? The patch works for me, and hydrogen was broken without it. ferringb@gmail.com 2004-05-26 12:55:12 0000 Re-opening, I just hit the same compilation failure. Running w/ audiofile-0.2.5, qt-3.3.0-r1, gcc 3.3.2-r5. Fails regardless of use flags, eradicator@gentoo.org 2004-05-27 22:59:19 0000 torbenh is non-responsive... reassigning... ferringb@gmail.com 2004-05-28 11:58:28 0000 Jeremy, thoughts on the addwrite? iirc, I'd thought you were futzing w/ a similiar issue... Aside from that, looking at the upstream code (coz I'm lazy), they threw out the Spacer* from the ui definition, between ver 1.12 and 1.15. http://cvs.sourceforge.net/viewcvs.py/hydrogen/hydrogen/hydrogen_gui/AudioEngineInfoForm_UI.ui?r1=text&tr1=1.13&r2=text&tr2=1.12&diff_format=h I'd be inclined to do the same, although it is varying from the original a bit.. Thoughts? I'd rather chuck the spacer* definitions, since they're not even used, although it ought to change the layout a bit. I don't use hydrogen, although I can emerge it for testing... eradicator@gentoo.org 2004-05-28 12:47:02 0000 the addwrite gets set when you use 'inherit kde' and use the kde_src_install (i believe that's correct)... or youcan do it manually... as for the spacer stuff... I don't really use this app, so I'll trust your judgement =) ferringb@gmail.com 2004-05-28 17:03:26 0000 *** Bug 51056 has been marked as a duplicate of this bug. *** fvdpol@gentoo.org 2004-06-16 17:05:25 0000 I do use the hydrogen drummachine (although from the cvs release); I can bake and test a compilation patch if you wish. fvdpol@gentoo.org 2004-06-18 13:00:51 0000 Committed to CVS as hydrogen-0.8.2-r1. thanks for the patch! makerik@gmail.com 2004-07-21 14:29:23 0000 I just had this problem because the unpatched ebuild is still the one available by default in portage. Using "ACCEPT_KEYWORDS=~x86 emerge hydrogen" I was able to compile hydrogen-0.8.2-r4. Since the actual ebuild(hydrogen-0.8.2.ebuild) does not work should not it be removed in favor of a working one(r4 is fine here) before saying the problem is fixed? 28549 2004-04-02 01:11 0000 Fixes missing declarations from the hydrogen headers. hydrogen-0.8.2-ui-spacer.patch text/plain ZGlmZiAtcnUgaHlkcm9nZW4tMC44LjIvaHlkcm9nZW5fZ3VpL0F1ZGlvRW5naW5lSW5mb0Zvcm1f VUkuaCBoeWRyb2dlbi0wLjguMi1maXhlZC9oeWRyb2dlbl9ndWkvQXVkaW9FbmdpbmVJbmZvRm9y bV9VSS5oCi0tLSBoeWRyb2dlbi0wLjguMi9oeWRyb2dlbl9ndWkvQXVkaW9FbmdpbmVJbmZvRm9y bV9VSS5oCTIwMDQtMDQtMDIgMTA6MzY6NTMuMTE0Mjc4NzI4ICswMjAwCisrKyBoeWRyb2dlbi0w LjguMi1maXhlZC9oeWRyb2dlbl9ndWkvQXVkaW9FbmdpbmVJbmZvRm9ybV9VSS5oCTIwMDQtMDQt MDIgMTA6Mzg6MzEuNzgxMjc5MDY0ICswMjAwCkBAIC0xOSw2ICsxOSw3IEBACiBjbGFzcyBRR3Jv dXBCb3g7CiBjbGFzcyBRTGFiZWw7CiBjbGFzcyBRUHVzaEJ1dHRvbjsKK2NsYXNzIFFTcGFjZXJJ dGVtOwogCiBjbGFzcyBBdWRpb0VuZ2luZUluZm9Gb3JtX1VJIDogcHVibGljIFFXaWRnZXQKIHsK QEAgLTcwLDYgKzcxLDggQEAKICAgICBRSEJveExheW91dCogbGF5b3V0OV8yXzI7CiAgICAgUUhC b3hMYXlvdXQqIExheW91dDk7CiAgICAgUUhCb3hMYXlvdXQqIExheW91dDlfMjsKKyAgICBRU3Bh Y2VySXRlbSogU3BhY2VyMTsKKyAgICBRU3BhY2VySXRlbSogU3BhY2VyMjsKIAogcHJvdGVjdGVk IHNsb3RzOgogICAgIHZpcnR1YWwgdm9pZCBsYW5ndWFnZUNoYW5nZSgpOwpkaWZmIC1ydSBoeWRy b2dlbi0wLjguMi9oeWRyb2dlbl9ndWkvTGFkc3BhRlhTZWxlY3Rvcl9VSS5oIGh5ZHJvZ2VuLTAu OC4yLWZpeGVkL2h5ZHJvZ2VuX2d1aS9MYWRzcGFGWFNlbGVjdG9yX1VJLmgKLS0tIGh5ZHJvZ2Vu LTAuOC4yL2h5ZHJvZ2VuX2d1aS9MYWRzcGFGWFNlbGVjdG9yX1VJLmgJMjAwNC0wNC0wMiAxMDoz Njo1My4xMDYyNzk5NDQgKzAyMDAKKysrIGh5ZHJvZ2VuLTAuOC4yLWZpeGVkL2h5ZHJvZ2VuX2d1 aS9MYWRzcGFGWFNlbGVjdG9yX1VJLmgJMjAwNC0wNC0wMiAxMDozODo0MS42MTQ3ODQxNDQgKzAy MDAKQEAgLTIzLDYgKzIzLDcgQEAKIGNsYXNzIFFMaXN0Qm94SXRlbTsKIGNsYXNzIFFQdXNoQnV0 dG9uOwogY2xhc3MgUUxhYmVsOworY2xhc3MgUVNwYWNlckl0ZW07CiAKIGNsYXNzIExhZHNwYUZY U2VsZWN0b3JfVUkgOiBwdWJsaWMgUURpYWxvZwogewpAQCAtNTgsNiArNTksOCBAQAogCiBwcm90 ZWN0ZWQ6CiAgICAgUUhCb3hMYXlvdXQqIExheW91dDMzOworICAgIFFTcGFjZXJJdGVtKiBTcGFj ZXIxOworICAgIFFTcGFjZXJJdGVtKiBTcGFjZXIyOwogCiBwcm90ZWN0ZWQgc2xvdHM6CiAgICAg dmlydHVhbCB2b2lkIGxhbmd1YWdlQ2hhbmdlKCk7CmRpZmYgLXJ1IGh5ZHJvZ2VuLTAuOC4yL2h5 ZHJvZ2VuX2d1aS9QcmVmZXJlbmNlc0RpYWxvZ19VSS5oIGh5ZHJvZ2VuLTAuOC4yLWZpeGVkL2h5 ZHJvZ2VuX2d1aS9QcmVmZXJlbmNlc0RpYWxvZ19VSS5oCi0tLSBoeWRyb2dlbi0wLjguMi9oeWRy b2dlbl9ndWkvUHJlZmVyZW5jZXNEaWFsb2dfVUkuaAkyMDA0LTA0LTAyIDEwOjM2OjUzLjExMjI3 OTAzMiArMDIwMAorKysgaHlkcm9nZW4tMC44LjItZml4ZWQvaHlkcm9nZW5fZ3VpL1ByZWZlcmVu Y2VzRGlhbG9nX1VJLmgJMjAwNC0wNC0wMiAxMDozOToxNy4xNTEzODE3NjAgKzAyMDAKQEAgLTI1 LDYgKzI1LDcgQEAKIGNsYXNzIFFMaW5lRWRpdDsKIGNsYXNzIFFTcGluQm94OwogY2xhc3MgUUNv bWJvQm94OworY2xhc3MgUVNwYWNlckl0ZW07CiAKIGNsYXNzIFByZWZlcmVuY2VzRGlhbG9nX1VJ IDogcHVibGljIFFEaWFsb2cKIHsKQEAgLTk5LDYgKzEwMCw4IEBACiAKIHByb3RlY3RlZDoKICAg ICBRSEJveExheW91dCogTGF5b3V0MzM7CisgICAgUVNwYWNlckl0ZW0qIFNwYWNlcjE7CisgICAg UVNwYWNlckl0ZW0qIFNwYWNlcjI7CiAKIHByb3RlY3RlZCBzbG90czoKICAgICB2aXJ0dWFsIHZv aWQgbGFuZ3VhZ2VDaGFuZ2UoKTsK 28550 2004-04-02 01:16 0000 Adds epatch the ui-spacer patch and the missing addwrite call hydrogen-0.8.2.ebuild.patch text/plain LS0tIGh5ZHJvZ2VuLTAuOC4yLmVidWlsZAkyMDA0LTAzLTI5IDE2OjIwOjAyLjAwMDAwMDAwMCAr MDIwMAorKysgL3Vzci9sb2NhbC9wb3J0YWdlL21lZGlhLXNvdW5kL2h5ZHJvZ2VuL2h5ZHJvZ2Vu LTAuOC4yLmVidWlsZAkyMDA0LTA0LTAyIDEwOjU3OjM5LjAyNjg3MTIyNCArMDIwMApAQCAtMjgs NiArMjgsOCBAQAogc3JjX3VucGFjaygpIHsKIAl1bnBhY2sgJHtBfQogCisJIyBlcGF0Y2ggJHtG SUxFU0RJUn0vJHtQfS11aS1zcGFjZXIucGF0Y2gKKwogCSMgRHJ1bSBraXRzIHRvIGluc3RhbGwK IAlsb2NhbCBLSVRTCiAJS0lUUz0iMzM1NTYwNmtpdCBFYXN0ZXJuSG9wLTEgSGlwSG9wLTEgSGlw SG9wLTIgU3ludGhpZS0xIFRSODA4OTA5IFRlY2huby0xIgpAQCAtNTgsNiArNjAsOCBAQAogfQog CiBzcmNfY29tcGlsZSgpIHsKKwlhZGR3cml0ZSAke1FURElSfS9ldGMvc2V0dGluZ3MKKwkKIAll aW5mbyAiUmVjb25maWd1cmluZy4uLiIKIAlleHBvcnQgV0FOVF9BVVRPQ09ORj0yLjUKIAlleHBv cnQgV0FOVF9BVVRPTUFLRT0xLjYK