Summary: | pd-0.37_pre8 doesn't build (nor does *pre11) | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Kasper Souren <Kasper.Souren> |
Component: | New packages | Assignee: | Gentoo Sound Team <sound> |
Status: | RESOLVED WONTFIX | ||
Severity: | normal | CC: | brandy, gcc-porting, mholzer |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
proposed pd-gcc3.3 patch
a proposed pd-gcc3.3 patch with a little less overkill. |
Description
Kasper Souren
2003-09-05 10:39:06 UTC
pd 0.37 is released Both pd-0.37_pre8 and pd-0.37-0 fail with the above error when using gcc-3.3.1 and having the jack libraries installed which are provided by media-sound/jack-audio-connection-kit. Compiling pd with gcc-3.2.3 produced no errors. I suspect (but don't have time to test right now) that this can be fixed by using j-a-c-k-0.7x. The 0.80 version changed some API and it looks like the mismatch is being caught by gcc-3.3 and not gcc-3.2. If i have time this weekend i'll take a look. Created attachment 18710 [details, diff]
proposed pd-gcc3.3 patch
This patch fixes the above error along with a whole bunch of errors caused
by
shadow declarations:
cc -g -DPD -O2 -funroll-loops -fomit-frame-pointer -Wall -W -Wshadow
-Wstrict-prototypes -Werror -Wno-unused -Wno-parentheses -Wno-switch
-I../../src -o vexp.pd_linux_o -c vexp.c
vexp.c:81: warning: declaration of `exp' shadows a global declaration
<built-in>:0: warning: shadowed declaration is here
.....
vexp.c:1555: warning: declaration of `exp' shadows a global declaration
<built-in>:0: warning: shadowed declaration is here
make[1]: *** [vexp.pd_linux_o] Error 1
make[1]: Leaving directory
`/var/tmp/portage/pd-0.37_pre8/work/pd-0.37-test8/extra/expr~'
make: *** [externs] Error 2
!!! ERROR: media-sound/pd-0.37_pre8 failed.
!!! Function src_compile, Line 43, Exitcode 2
!!! parallel make failed
The patch can be applied against pd-0.37_pre8 (which is in the portage tree)
and pd-0.37-0 (which isn't).
Created attachment 18716 [details, diff] a proposed pd-gcc3.3 patch with a little less overkill. Since all the errors in extra/expr~/vexp.c are due to the -Werror option being specified in extra/expr~/makefile: LINUXCFLAGS = -DPD -O2 -funroll-loops -fomit-frame-pointer \ -Wall -W -Wshadow -Wstrict-prototypes -Werror \ -Wno-unused -Wno-parentheses -Wno-switch it may be worthwhile simply editing the file with sed. Taking out the -Werror option will make the vast majority of the patch in comment #4 redundant. 0.37.0 is in cvs please open a new bug if problems with new version |