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

[Frank Lanitz]

Am 21.02.2013 10:47, schrieb Andreas Müller:
> 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 :)

I think it's a dangerous kind of dependencies management.

Cheers,
Frank