Semi-OT: makedepend isn't aware of /usr/lib/gcc/$ARCH-redhat-linux/4.3.0/include/

[Dan Nicholson]

On Mon, Nov 3, 2008 at 7:21 AM, Jakub Jelinek <jakub at redhat.com> wrote:
> On Mon, Nov 03, 2008 at 07:07:29AM -0800, Dan Nicholson wrote:
>> 2008/11/3 Adam Jackson <ajax at redhat.com>:
>> > On Mon, 2008-11-03 at 15:58 +0200, Gilboa Davara wrote:
>> >> I'm not sure if this is a bug or not, so I'm posting here before I open
>> >> up a BZ.
>> >
>> > makedepend is part of the imake package in Fedora. imake is - shall we
>> > say - under a policy of malign neglect by upstream.  Rightly so, in my
>> > opinion, but makedepend is legitimately useful on its own.
>> >
>> > I think the only semi-serious problem with teaching makedepend more
>> > about gcc is what to do in the face of multiple gcc versions.  We could
>> > certainly make makedepend search %{_libdir}/gcc/`gcc -dumpmachine`/`gcc
>> > -dumpversion`/include but that only works if you're building with gcc
>> > and not say compat-gcc34.  Maybe inherit $CC from the environment and
>> > use that?  Who knows.
>> >
>> > Ideally you'd just use gccmakedep instead, but it doesn't support the
>> > same set of options as makedepend so that may not be an option.
>> >
>> > I'd take a patch, but that's about the extent to which I care about
>> > being a build tools maintainer.
>>
>> I have a patch somewhere that adds a --with-extra-include build option
>> for makedepend. It still means you have to rebuild makedepend whenever
>> gcc gets a major upgrade, but that was probably happening anyway. If
>> that sounds useful, I'll go locate it.
>
> If that would be just a major upgrade, it wouldn't be so bad, but
> the directory changes even with gcc patchlevel changes, which happen
> usually at least once during each Fedora release.

Good point. Just for the record, this could be done today without
patching makedepend since there are already macros for handling extra
directories. This is what I was doing for my makedepend build before:

gccincdir=`${CC-gcc} -print-file-name=include`
echo "#define EXTRAINCDIR \"$gccincdir\"" >> makedepend-config.h.in

--
Dan