I'm getting annoyed at various things happening during bulk-builds.
Specifically, I think that stuff that gets installed should be cleaner.
I'm not talking about fixing all strcpy linker warnings (though looking
at the mktemp stuff never hurts), I'm talking about bad headers that
generate warnings.
Case in point: lcms (which I'm fixing)
/usr/local/include/lcms.h:146:5: warning: "TARGET_CPU_PPC" is not defined
/usr/local/include/lcms.h:150:5: warning: "macintosh" is not defined
/usr/local/include/lcms.h:156:5: warning: "__BIG_ENDIAN__" is not defined
In file included from icctransform.cpp:40,
from libdimgfilters_la.all_cpp.cpp:4:
/usr/local/include/lcms.h: In function `void* _cmsMalloc(long unsigned int)':
/usr/local/include/lcms.h:1410: warning: comparison of unsigned expression < 0
is always false
this shit happens all the time, and it distracts from real problems.
There are also a lot of autoconf macros (.m4) that are not yet properly
quoted, and that show all kind of shit when you autoreconf.
I think we've reached a point where we should try to fix those things.
Not all headers at once, as this will be a lot of work, but at least
stuff that gets installed shouldn't warn when it's used under normal
circumstances.
Of course, as usual, this should be sent back upstream in a proper format.
Ask for advice if you don't know how to do the fixes.
