Re: [uracoli-devel] MacOS compile problems

[axel . wachtler]

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. ;-)
>