Re: [Discuss-gnuradio] FindICE.cmake

[West, Nathan]

On Thu, May 1, 2014 at 8:23 AM, Tom Rondeau <address@hidden> wrote:
> On Wed, Apr 30, 2014 at 9:11 PM, West, Nathan <address@hidden>
> wrote:
>>
>> I'm trying to build gnu radio with control port, but cmake won't find
>> my Ice installation. I have Ice-3.4 installed through apt on Ubuntu
>> 13.10. Digging in to the FindIce.cmake I made the following changes:
>>
>> diff --git a/cmake/Modules/FindICE.cmake b/cmake/Modules/FindICE.cmake
>> index 087ee9b..705c019 100644
>> --- a/cmake/Modules/FindICE.cmake
>> +++ b/cmake/Modules/FindICE.cmake
>> @@ -4,7 +4,7 @@
>>  FIND_PACKAGE(PkgConfig)
>>  PKG_CHECK_MODULES(PC_ICE Ice-3.4 QUIET)
>>
>> -if(NOT ICE_FOUND)
>> +if(NOT PC_ICE_FOUND)
>>    # Maybe we don't have a .pc file for Ice. Check for Config.h. If
>>    # that exists, get the version string and parse it for the proper
>>    # version.
>> @@ -21,11 +21,11 @@ if(NOT ICE_FOUND)
>>        set(ICE_FOUND TRUE)
>>      endif(ICE_FOUND)
>>    endif(ICE_CONFIG_INCLUDE_DIR)
>> -endif(NOT ICE_FOUND)
>> +endif(NOT PC_ICE_FOUND)
>>
>> -if(NOT ICE_FOUND)
>> +if(NOT PC_ICE_FOUND)
>>    message(STATUS "  package 'Ice-3.4' not found")
>> -endif(NOT ICE_FOUND)
>> +endif(NOT PC_ICE_FOUND)
>>
>>
>>  # Recheck if we found the right version of ICE and proceed if true.
>>
>> This at least squelches the 'package Ice-3.4 not found' message, which
>> is an improvement. (I test this with cmake -P <path-to-module>. I'm
>> still not able to get cmake to configure GNU Radio to build
>> control-port, and this is already extending my cmake-fu.
>>
>> I've only managed to get this configured properly through PyBOMBS. Any
>> suggestions on fixing this module to properly detect a distro
>> installed Ice?
>>
>> Nathan
>
>
>
> Nathan,
> Yeah, I suspect you're right on the use of the PC_ for the variables.
>
> Have you made sure that python-zeroc-ice is also installed?
>
> Tom
>

Yea, python-zeroc-ice is installed. If I replace all of the ICE_FOUND
instances with PC_ICE_FOUND then I get a syntax error on line 36:
string(REGEX REPLACE "/include" "" ICE_PATH
${ICE_CONFIG_INCLUDE_DIR}). I Don't see where ICE_PATH is being set in
the first place.

Nathan