Bugzilla Bug 196621

In file included from yelp-window.c:48:
yelp-debug.h:56:35: warning: ISO C does not permit named variadic macros
yelp-window.c: In function 'window_set_section_cursor':
yelp-window.c:1501: warning: ISO C forbids passing argument 6 of
'g_signal_handlers_block_matched' between function pointer and 'void *'
yelp-window.c:1532: warning: ISO C forbids passing argument 6 of
'g_signal_handlers_unblock_matched' between function pointer and 'void *'
yelp-window.c: In function 'window_do_load_html':
yelp-window.c:1589: warning: missing sentinel in function call
yelp-window.c: In function 'window_reload_cb':
yelp-window.c:2162: error: expected expression before '/' token
make[3]: *** [yelp-yelp-window.o] Error 1
make[3]: *** Waiting for unfinished jobs....
In file included from yelp-transform.c:40:
yelp-debug.h:56:35: warning: ISO C does not permit named variadic macros
yelp-transform.c:46: warning: type defaults to 'int' in declaration of
'exslt_registered'
yelp-transform.c: In function 'xslt_yelp_document':
yelp-transform.c:410: warning: pointer targets in passing argument 1 of
'g_strdup' differ in signedness
make[3]: Leaving directory
`/var/tmp/portage/gnome-extra/yelp-2.20.0/work/yelp-2.20.0/src'
make[2]: *** [all] Error 2
make[2]: Leaving directory
`/var/tmp/portage/gnome-extra/yelp-2.20.0/work/yelp-2.20.0/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory
`/var/tmp/portage/gnome-extra/yelp-2.20.0/work/yelp-2.20.0'
make: *** [all] Error 2
 * 
 * ERROR: gnome-extra/yelp-2.20.0 failed.

Reproducible: Always

Created an attachment (id=134043) [details]
emerge --log

Created an attachment (id=134045) [details]
emerge --info

gnome upstream and gnome herd doesn't support optimization level above 2 (-Os
and -O3). Please try again with safer CFLAGS (-O2 -march=native -pipe)

(In reply to comment #3)
> gnome upstream and gnome herd doesn't support optimization level above 2 (-Os
> and -O3). Please try again with safer CFLAGS (-O2 -march=native -pipe)
> 
Same result with cflags :
CFLAGS="-march=native -mtune=native -O2 -pipe"
CXXFLAGS="${CFLAGS}"

(In reply to comment #3)
> gnome upstream and gnome herd doesn't support optimization level above 2 (-Os
> and -O3). Please try again with safer CFLAGS (-O2 -march=native -pipe)
> 
Same result with cflags :
CFLAGS="-march=native -mtune=native -O2 -pipe"
CXXFLAGS="${CFLAGS}"

re-emerge it without the debug use flag.  That sets -pedantic -ansi, which
causes the build to fail on the (otherwise perfectly valid) C++ style comment
at that line.

(In reply to comment #6)
> re-emerge it without the debug use flag.  That sets -pedantic -ansi, which
> causes the build to fail on the (otherwise perfectly valid) C++ style comment
> at that line.
> 
Many thanks for your explanation Daniel, yelp compiles fine with "-debug" USE
flag, but is this an error's workaround before fixing such a bug or a
definitive solution to this compilation problem ?

CFLAGS such as -pedantic or -Werror should _never_ be included in official
release tarballs. It's just plain dumb. You can't predict what compilers your
users are going to use 2 weeks or 6 months from when the tarball is released.

We try as much as we can to convince upstream devs to remove such flags, most
of them understand. I guess we'll have to file a bug upstream about this.

Thanks

More importantly, the debug use flag probably doesn't do what you think it
does.  We need to remove it from almost all gnome packages; currently, it's in
the eclass, and breaks all kinds of things.

Here's the description of the debug use flag:

debug - Enable extra debug codepaths, like asserts and extra output. If you
want to get meaningful backtraces see
http://www.gentoo.org/proj/en/qa/backtraces.xml

Ah, I'm glad that somebody found a solution to this. I thought I would need to 
submit a patch to gnome ;-) 

Given that the package is marked ~x86 (as of a few days ago anyway) it's not 
so bad that it won't compile with +debug.

Thanks for pointing out a work-around.

Created an attachment (id=137728) [details]
C++-style  commentary should be replaced with a plain-C-style commentary

thanks for the bug report, doing what the bug report describes fixed the
compile failures for me also (had debug also in my use flags),


Alex

I've stripped off the silly flags, fix is in tree.

(In reply to comment #13)
> I've stripped off the silly flags, fix is in tree.
> 

The ebuild works fine thanks Gilles.