[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Discuss-gnuradio] GnuradioConfig.cmake not working properly ?
From: |
Sylvain Munaut |
Subject: |
Re: [Discuss-gnuradio] GnuradioConfig.cmake not working properly ? |
Date: |
Tue, 12 Nov 2013 18:54:43 +0100 |
Hi Tom,
I'm afraid my patch broke something :( gr-osmosdr and probably some
other OOT projects that were using GnuradioConfig but without using
the _ALL_ variable but using the individual ones.
Turns out that after the patch, GNURADIO_XXX_LIBRARIES is no longer
directly build from find_library, but from a custom foreach loop (to
build up the proper list)
But that also mean that it's scope is not restricted to the function
and is not global and so not available elsewhere.
The attached patch fixes that by re-declaring the variable in the parent scope.
This fixes the gr-osmosdr build for me.
@Martin : could you give it a shot to check it fixes it for you too ?
Cheers,
Sylvain
On Mon, Nov 11, 2013 at 8:40 PM, Tom Rondeau <address@hidden> wrote:
> On Sun, Nov 10, 2013 at 11:56 AM, Sylvain Munaut <address@hidden> wrote:
>>> I'm having a couple of issues when trying to switch from
>>> FindGnuradioRuntime to GnuradioConfig
>>
>> Attached a patch that deals with issue 1 & 3. Issue 2 doesn't seem to
>> be an issue since ALL_LIBRARIES has absolute path in it.
>>
>> Cheers,
>>
>> Sylvain
>
>
> Thanks, Sylvain. I was actually working on a tutorial for gnuradio.org
> to describe the use of GnuradioConfig when I noticed the exact same
> issue with the _ALL_ variables. Your patch did the trick and has been
> pushed.
>
> Tom
gnuradioconfig-make-global.diff
Description: Text document
- [Discuss-gnuradio] GnuradioConfig.cmake not working properly ?, Sylvain Munaut, 2013/11/10
- Re: [Discuss-gnuradio] GnuradioConfig.cmake not working properly ?, Sylvain Munaut, 2013/11/10
- Re: [Discuss-gnuradio] GnuradioConfig.cmake not working properly ?, Tom Rondeau, 2013/11/11
- Re: [Discuss-gnuradio] GnuradioConfig.cmake not working properly ?, Sylvain Munaut, 2013/11/11
- Re: [Discuss-gnuradio] GnuradioConfig.cmake not working properly ?,
Sylvain Munaut <=
- Re: [Discuss-gnuradio] GnuradioConfig.cmake not working properly ?, Martin Braun (CEL), 2013/11/12
- Re: [Discuss-gnuradio] GnuradioConfig.cmake not working properly ?, Tom Rondeau, 2013/11/13
- Re: [Discuss-gnuradio] GnuradioConfig.cmake not working properly ?, Douglas Geiger, 2013/11/14
- Re: [Discuss-gnuradio] GnuradioConfig.cmake not working properly ?, Tom Rondeau, 2013/11/14