As per $summary: x86_64-pc-linux-gnu-g++ -DHAVE_CONFIG_H -I. -I.. -march=native -O2 -Wall -W -g -Wall -D_DEBUG -O3 -ffast-math -fno-exceptions -fomit-frame-pointer -Dwith_oss -Dwith_alsa -Dwith_jack -Dwith_sndfile -DENABLE_BINRELOC -march=native -O2 -Wall -W -g -Wall -D_DEBUG -O3 -ffast-math -fno-exceptions -fomit-frame-pointer -Dwith_oss -Dwith_alsa -Dwith_jack -Dwith_sndfile -MT amSynth-main.o -MD -MP -MF .deps/amSynth-main.Tpo -c -o amSynth-main.o `test -f 'main.cc' || echo './'`main.cc amd64box maildrop # emerge --info | grep CXX CXXFLAGS="-march=native -O2" Please drop -O3 -g -fomit-frame-pointers and that do you think is more.
Fixed
the committed patch is bogus as it drops upstream preprocessor flag -D_DEBUG to enable different codepath, then it removes warning flags hiding also portage QA warnings you can remove -O3 since it's unsafe taking account different gcc versions, and different arches and -g since it's up to user to set -g -ggdb and to what level but leave the other optimization flags in too, upstream has specifically picked them to be compatible with his code, and unless we get a bug report about them, there is absolutely no need to remove them
Right. Sorry about that. I removed the patch for now. I will try to come up with an improved patch when I get back home
Created attachment 282849 [details] amsynth-1.2.3-cflags.patch this patch works for me. Dropped also -fomit-frame-pointers.
Created attachment 282865 [details, diff] proposed patch patch based on Samuli's suggestions
Markos, your patch works for me and respect that samuli sayd. You can commit, imho.
+*amsynth-1.2.3-r1 (30 Aug 2011) + + 30 Aug 2011; Tony Vroon <chainsaw@gentoo.org> +amsynth-1.2.3-r1.ebuild, + +files/amsynth-1.2.3-respect-cflags.patch: + Revision bump for EAPI 4 porting and CFLAGS respect patch by Markos + "hwoarang" Chandros with input from Samuli Suominen in bug #376665 filed by + Agostino "ago" Sarubbo.
