Subject: Re: pkgsrc IRIX 6.5/mipseb bulk build results 20060704.1102
To: None <pkgsrc-bulk@netbsd.org>
From: Roland Illig <rillig@NetBSD.org>
List: pkgsrc-bulk
Date: 07/11/2006 06:46:56
Jan Schaumann wrote:
> Roland Illig <rillig@NetBSD.org> wrote:
>>The C++ preprocessor should be "CC -E", not "CC".
>
>
> I agree. Could whoever b0rked that since the last stable branch please
> fix this?
I fixed it but never broke it. The setting in
bootstrap/mods/mk/IRIX.sys.mk has been there since revision 1.1 from
March 2004.
By the way, in mk/compiler.mk, we have:
ALL_ENV+= CXXCPP=${CPP:Q}
Isn't that calling for bugs, too? In my opinion this line should be:
CXXCPP?= ${CXX} -E
ALL_ENV+= CXXCPP=${CXXCPP:Q}
This looks more sane to me. I think it just worked up to now because no
package depends on CPP being a real C preprocessor and CXXCPP being a
real C++ preprocessor.
Roland