[Geany-Devel] [PATCH] configure.ac remove additional c++ test

Andreas Müller schnitzeltony at xxxxx
Thu Feb 21 09:47:16 UTC 2013


On Thu, Feb 21, 2013 at 10:17 AM, Lex Trotman <elextr at gmail.com> wrote:
> Hi Andreas,
>
> On 21 February 2013 19:46, Andreas Müller <schnitzeltony at googlemail.com> wrote:
>> For openembedded cross-builds CXX can contain something like:
>>
>> arm-oe-linux-gnueabi-g++  -march=armv5te  -marm -mthumb-interwork --sysroot=/home/jenkins/oe/shr-core-branches/shr-core/tmp-eglibc/sysroots/qemuarm
>>
>> This ends up in
>>
>> configure:5582: error: No C++ compiler found. Please install a C++ compiler.
>>
>> Anyway if [1] is still valid, the combination of AC_PROG_CC and AC_PROG_CXX
>> should thow an error message.
>
> Geany *requires* both a C compiler and a C++ compiler, so both must be
> tested for.  The test you have removed is to get around the stupid
> behavior of AC_PROG_CXX of returning g++ if it finds g++ or if it
> finds nothing.  Perhaps which should only be run if CXX == g++.
>
> I presume you are setting CXX on the configure line to the whole line
> you specified above.  Try setting CXX to the executable only and
> CXXFLAGS to the rest.
>
> Cheers
> Lex
Thanks for prompt response. I understand your objection and think the
easiest way to get around is adding this patch to openembedded. There
both compilers can be assumed available (or in other words if they are
not we are in serious trouble anyway :)

Andreas


More information about the Devel mailing list