timidity++ supports this API, so I'd like to get it into all archs that timidity++ supports.
The best way to make sure an ebuild gets tested and fixed is to test it yourself and then post the results. I got portaudio to compile by patching the makefile to honor cflags and then adding CFLAGS="-fPIC" after make -f Makefile.linux
Created attachment 28110 [details, diff] patch to make portaudio honor the CFLAGS variable this patch makes the portaudio Makefile honor the CFLAGS variable.
Travis, I'm not quite sure what you mean here? This is a request for a keyword to be added to a current ebuild. Were you haveing problems with the ebuild before? If so, please open a separate bug report. Thanks.
Created attachment 28112 [details, diff] patch to make portaudio honor CFLAGS This patch makes portaudio honor CFLAGS... and is actually in a usable format, unlike the last patch.
portaudio wont compile on amd64, why would you want to add a keyword without making it able to compile first?
To clarify, without any modification, portaudio fails with the following error: /usr/lib/gcc-lib/x86_64-pc-linux-gnu/3.3.3/../../../../x86_64-pc-linux-gnu/bin/ld: ./pa_common/pa_lib.o: relocation R_X86_64_32S can not be used when making a shared object; recompile with -fPIC ./pa_common/pa_lib.o: could not read symbols: Bad value collect2: ld returned 1 exit status make: *** [sharedlib] Error 1 This is easily fixed just by adding -fPIC on amd64, but the makefile doesn't honor CFLAGS, so I first patched it to honor CFLAGS and edited the ebuild to add -fPIC to CFLAGS on amd64. hppa and alpha would also need -fPIC, I think... and a few other archs.
Created attachment 28115 [details, diff] patches the portaudio ebuild to check for amd64 and add -fPIC to CFLAGS this patches the portaudio ebuild to append-flags -fPIC if running on an amd64 box, fixing PIC related compile errors. I'd like to note that it makes use of the system CFLAGS setting as well, so it might need a filter-flags?
Rather than using append-flags, it'd be better to patch the Makefile to append -fPIC for any libraries. On x86 it provides performance benefits, and on other archs it makes it actually work. http://dev.gentoo.org/~ciaranm/tmp/347.txt has information about it.
Regarding comment #5, that's the point of this bug ;). I don't have amd64 hardware, so I have to put in a bug request to get them to test out ebuilds on that arch. I didn't realize that your problem was amd64 specific. Thanks for helping out with this effort.
well, portaudio is a library... but if it's ok to have -fPIC on all archs, then patching the makefile to honor cflags is a bit overkill. I'll up another patch that just adds -fPIC.
Created attachment 28143 [details, diff] patch to make portaudio compile with -fPIC
eradicator - Well dont I feel silly, I had assumed that you did have an amd64 box. ^_^;
Created attachment 28144 [details, diff] patch to portaudio ebuild to apply fpic patch
I'm not done with my new developer mentoring period, so I need someone to either commit this to cvs and close this bug or let me know what further changes need to be made to do so.
fix in cvs, wee
