[Buildroot] [PATCH 1/1] pkg-cmake: fix host-cmake-package type

Samuel Martin s.martin49 at gmail.com
Fri Jan 23 19:34:48 UTC 2015


Hi Ryan,

On Fri, Jan 23, 2015 at 7:31 PM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> Ryan, All,
>
> On 2015-01-23 12:10 -0600, Ryan Barnett spake thusly:
>> When trying to compile a host-cmake-package type such as host-cdrkit,
>> an error happens in regards to HOSTCC and HOSTCXX not being found.
>>
>>   /bin/sh: HOSTCC: command not found
>>   /bin/sh: HOSTCXX: command not found
>>
>> This due to the fact that the environment is incorrectly configured by
>> having the following:
>>
>>   -DCMAKE_C_COMPILER=""$(HOSTCC)"" -DCMAKE_CXX_COMPILER=""$(HOSTCXX)""
>>
>> This is due to having too many dollar signs. The fix is to remove the
>> extra dollar sign.
>>
>> Also removing the unncessary quotes since the HOSTCC/HOSTCXX will
>> already contain quotes. This could cause an issue with make as double
>> quotes are not interpreted by make.
>>
>> Minor formatting fix too.
>>
>> Signed-off-by: Ryan Barnett <ryan.barnett at rockwellcollins.com>
>> Acked-by: Samuel Martin <s.martin49 at gmail.com>
>> CC: Yann E. MORIN <yann.morin.1998 at free.fr>
>
> Reviewed-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
>
> Regards,
> Yann E. MORIN.
>
>> ---
>> v1 -> v2
>>  - Implemented spacing and quote concerns from Yann E. MORIN and
>>    Samuel Marin.
>>
>> Signed-off-by: Ryan Barnett <ryan.barnett at rockwellcollins.com>
>> ---
>>  package/pkg-cmake.mk | 12 ++++++------
>>  1 file changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/package/pkg-cmake.mk b/package/pkg-cmake.mk
>> index 1037513..e738ba9 100644
>> --- a/package/pkg-cmake.mk
>> +++ b/package/pkg-cmake.mk
>> @@ -22,13 +22,13 @@
>>
>>  # Set compiler variables.
>>  ifeq ($(BR2_CCACHE),y)
>> -CMAKE_HOST_C_COMPILER="$(HOST_DIR)/usr/bin/ccache"
>> -CMAKE_HOST_CXX_COMPILER="$(HOST_DIR)/usr/bin/ccache"
>> -CMAKE_HOST_C_COMPILER_ARG1="$(HOSTCC_NOCCACHE)"
>> -CMAKE_HOST_CXX_COMPILER_ARG1="$(HOSTCXX_NOCCACHE)"
>> +CMAKE_HOST_C_COMPILER = "$(HOST_DIR)/usr/bin/ccache"
>> +CMAKE_HOST_CXX_COMPILER = "$(HOST_DIR)/usr/bin/ccache"

No need for double quotes here (when used, this variable is surrounded
by quotes [1]).
Otherwise, it looks good.


>> +CMAKE_HOST_C_COMPILER_ARG1 = $(HOSTCC_NOCCACHE)
>> +CMAKE_HOST_CXX_COMPILER_ARG1 = $(HOSTCXX_NOCCACHE)
>>  else
>> -CMAKE_HOST_C_COMPILER="$$(HOSTCC)"
>> -CMAKE_HOST_CXX_COMPILER="$$(HOSTCXX)"
>> +CMAKE_HOST_C_COMPILER = $(HOSTCC)
>> +CMAKE_HOST_CXX_COMPILER = $(HOSTCXX)
>>  endif
>>
>>  ################################################################################
>> --
>> 1.9.1
>>
>
> --
> .-----------------.--------------------.------------------.--------------------.
> |  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
> | +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
> '------------------------------^-------^------------------^--------------------'
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

[1] http://git.buildroot.net/buildroot/tree/package/pkg-cmake.mk#n110


Regards,

-- 
Samuel



More information about the buildroot mailing list