Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 28007 - pd-0.37_pre8 doesn't build (nor does *pre11)
Summary: pd-0.37_pre8 doesn't build (nor does *pre11)
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Sound Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-09-05 10:39 UTC by Kasper Souren
Modified: 2003-10-28 05:48 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
proposed pd-gcc3.3 patch (pd-0.37-gcc3.3.patch,34.50 KB, patch)
2003-10-03 17:30 UTC, Brandy Westcott (RETIRED)
Details | Diff
a proposed pd-gcc3.3 patch with a little less overkill. (pd-0.37_pre8-gentoo.patch,414 bytes, patch)
2003-10-04 00:19 UTC, Brandy Westcott (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kasper Souren 2003-09-05 10:39:06 UTC
Here's some output, which is actually from a pd-0.37_pre11.ebuild (cp of 
pre8): 
 
gcc -DPD -DUNIX  -Wall -W -Wstrict-prototypes -Werror -Wno-unused 
-Wno-parentheses -Wno-switch -g -DDL_OPEN -DPA_USE_OSS -DPA_LITTLE_ENDIAN              
-DUSEAPI_OSS         -I../portaudio/pa_common         -I../portaudio/pablio 
-I../portaudio/portmidi-macosx -Werror -DPA_USE_ALSA -DUSEAPI_ALSA 
-DUSEAPI_JACK -DINSTALL_PREFIX=\"/usr\" -I. -c -o ../obj/s_audio_jack.o 
s_audio_jack.c 
cc1: warnings being treated as errors 
s_audio_jack.c: In function `srate': 
s_audio_jack.c:72: warning: long int format, jack_nframes_t arg (arg 2) 
make: *** [s_audio_jack.o] Error 1 
make: *** Waiting for unfinished jobs.... 
 
 
I have a slight idea that it could be related to some new JACK code... 
And, er, btw, I'm running JACK 0.80, which I emerged by cping the 0.75 
ebuild. (Reported JACK 0.80 upgrade little while ago.)
Comment 1 Kasper Souren 2003-09-23 13:43:33 UTC
pd 0.37 is released 
Comment 2 Brandy Westcott (RETIRED) gentoo-dev 2003-10-03 17:17:22 UTC
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.
Comment 3 J. Ellis (RETIRED) gentoo-dev 2003-10-03 17:22:32 UTC
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.
Comment 4 Brandy Westcott (RETIRED) gentoo-dev 2003-10-03 17:30:55 UTC
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).
Comment 5 Brandy Westcott (RETIRED) gentoo-dev 2003-10-04 00:19:09 UTC
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.
Comment 6 Martin Holzer (RETIRED) gentoo-dev 2003-10-28 05:48:17 UTC
0.37.0 is in cvs

please open a new bug if problems with new version