Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 239929

Summary: app-editors/jove: CFLAGS are ignored
Product: Gentoo Linux Reporter: Diego Elio Pettenò (RETIRED) <flameeyes>
Component: New packagesAssignee: Emacs project <emacs>
Status: RESOLVED WORKSFORME    
Severity: normal    
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Diego Elio Pettenò (RETIRED) gentoo-dev 2008-10-05 11:36:11 UTC
I don't know the build system for this program (and this is a generic bug template so I cannot tell you which program exactly is), but my tests shows that it's not respecting CFLAGS properly.

Please look into it, since it's important to respect user CFLAGS.

Thanks,
Diego
Comment 1 Ulrich Müller gentoo-dev 2008-10-05 11:44:18 UTC
Great, I just committed a new version as fix for bug 239869 one minute ago. :-(

Couldn't you have reported these two build system issues in one single bug, to avoid duplicate work for package maintainers?
Comment 2 Ulrich Müller gentoo-dev 2008-10-05 11:45:22 UTC
Patches are welcome. :p
Comment 3 Diego Elio Pettenò (RETIRED) gentoo-dev 2008-10-05 11:46:58 UTC
I was going through the whole list of app-editors/*, but since I grep the logs for one condition first and the other after, they get reported separately... sorry but you're the _only_ person who ever responded to a bug I've been reporting so quickly...
Comment 4 Ulrich Müller gentoo-dev 2008-10-05 11:58:10 UTC
Hm, src_unpack does:
    sed -i -e "s:^OPTFLAGS =.*:OPTFLAGS = ${CFLAGS}:" Makefile

And my CFLAGS are respected during compilation:
>>> Compiling source in /var/tmp/portage/app-editors/jove-4.16.0.70.3/work/jove4.16.0.70 ...
gcc -O -march=pentium-m -O2 -pipe -fomit-frame-pointer -DBSDPOSIX_STDC   -c -o commands.o commands.c
gcc -O -march=pentium-m -O2 -pipe -fomit-frame-pointer -DBSDPOSIX_STDC   -c setmaps.c
[...]

So what exactly is the issue?
Comment 5 Ulrich Müller gentoo-dev 2008-10-21 21:53:34 UTC
Closing (as discussed on IRC). Maybe the ebuild is not using the most elegant solution, but CFLAGS are respected.