Re: [uracoli-devel] MacOS compile problems

[Joerg Wunsch]

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