Re: [uracoli-devel] MacOS compile problems

[Torsten Meißner]

Hello,


I've checked out the sources per mercurial and have built them with my macports 
avr-gcc versions.
I can confirm that it compiles now without problems. Very good. So, now i have 
to find the time to work
with it :-)


Greetz,
Torsten


--

Mail:   address@hidden
Blog:   http://blog.goodcleanfun.de
Github:   https://github.com/tmeissner



Am 27.03.2012 um 11:08 schrieb address@hidden:

> Thanks Joerg, for figuring this out, I will apply this patch to
> the release 0.2.x branch to and release package 0.2.1 a.s.a.p.
> 
> Axel
> 
> -------- Original-Nachricht --------
>> Datum: Tue, 27 Mar 2012 10:58:37 +0200
>> Von: Joerg Wunsch <address@hidden>
>> An: address@hidden
>> Betreff: Re: [uracoli-devel] MacOS compile problems
> 
>> As Torsten Meißner wrote:
>> 
>>> … The Library compiles with these avr-gcc version. So, came there a
>>> problem into the avr-gcc from 4.5.1 to 4.6.3 which is fixed in
>>> 4.7.0? Or is it just the combination of these specific versions
>>> which macports use? Mystery, mystery ;)
>> 
>> We found it.  We've been fooled by scons not picking up avr-gcc from
>> $PATH (as one would assume it to be).  That's why my separately built
>> AVR-GCC 4.7.0 (actually it's even named 4.8.0, I just noticed —
>> obviously, I updated shortly after the release of 4.7.0) hadn't been
>> picked up by scons when building.
>> 
>> As already assumed (and I've been close to it during the workshop ;),
>> a couple of more "const" qualifiers is required:
>> 
>> diff -r b0ae8bf70282 -r 0dec6e540037 uracoli/Src/App/diagradio.c
>> --- a/uracoli/Src/App/diagradio.c    Tue Mar 27 06:38:18 2012 +0200
>> +++ b/uracoli/Src/App/diagradio.c    Tue Mar 27 10:50:42 2012 +0200
>> @@ -106,7 +106,14 @@
>> timer_hdl_t  th_blink;
>> 
>> /** factory defaults of radio parameter  */
>> -trx_param_t PROGMEM trxp_flash = {chan: 13, txp: 0, cca: 1, edt: 11,
>> clkm: 0};
>> +const trx_param_t PROGMEM trxp_flash =
>> +{
>> +    chan: 13,
>> +    txp: 0,
>> +    cca: 1,
>> +    edt: 11,
>> +    clkm: 0
>> +};
>> 
>> /**
>>  * @brief Main function of diagradio application.
>> diff -r b0ae8bf70282 -r 0dec6e540037 uracoli/Src/App/wuart.c
>> --- a/uracoli/Src/App/wuart.c        Tue Mar 27 06:38:18 2012 +0200
>> +++ b/uracoli/Src/App/wuart.c        Tue Mar 27 10:50:42 2012 +0200
>> @@ -109,7 +109,7 @@
>> 
>> /* === globals ====================================== */
>> 
>> -node_config_t PROGMEM nc_flash = { .short_addr = 0xBEEF,
>> +const node_config_t PROGMEM nc_flash = { .short_addr = 0xBEEF,
>>                                    .pan_id = 0x2222,
>>                                    .channel = DEFAULT_RADIO_CHANNEL,
>>                                  };
>> diff -r b0ae8bf70282 -r 0dec6e540037
>> uracoli/Src/Lib/Rf230/trx_datarate_str.c
>> --- a/uracoli/Src/Lib/Rf230/trx_datarate_str.c       Tue Mar 27 06:38:18 2012
>> +0200
>> +++ b/uracoli/Src/Lib/Rf230/trx_datarate_str.c       Tue Mar 27 10:50:42 2012
>> +0200
>> @@ -112,7 +112,7 @@
>> 
>> 
>> 
>> -PGM_P rate_strtable[] PROGMEM =
>> +PGM_P const rate_strtable[] PROGMEM =
>> {
>> # ifdef TRX_BPSK20
>>     C_BPSK20_STR,
>> 
>> (Change already committed and pushed to the savannah repository.)
>> 
>> Thanks for nagging us, Torsten.
>> -- 
>> cheers, J"org               .-.-.   --... ...--   -.. .  DL8DTL
>> 
>> http://www.sax.de/~joerg/                        NIC: JW11-RIPE
>> Never trust an operating system you don't have sources for. ;-)
>> 
>