Re: [Paparazzi-devel] Fwd: Problems with make after running software upd

On Tue, Jan 9, 2018 at 1:56 PM, Helge Walle <address@hidden> wrote:

Thank you Michal,

I looked into the the update log on my pc and noticed that gcc-arm-embedded seems to have been updated.
Just to try something, I did the following:

           sudo apt-get remove gcc-arm-embedded
           sudo apt autoremove
           sudo apt-get install gcc-arm-none-eabi
           sudo apt-get install gdb-arm-none-eabi

Now make runs in a normal way as far as I can see, and building the aircraft also seems ok.
I'll test the build on my physical hardware hopefully tomorrow.

Would it be ok to keep it like this until stable paparazzi is updated with the warnings fix?

Helge.

2018-01-09 20:42 GMT+01:00 Michal Podhradsky <address@hiddenusu.edu>:
Hi Helge,

this is an artifact of the arm-none-eabi-gcc update. Version 7 has some stricter warnings on as defaults. These warnings were already fixed in master but haven't been backported to the stable branches yet (which we should do soon).

Sorry for the inconvenience.

Regards
Michal

On Tue, Jan 9, 2018 at 10:33 AM, Helge Walle <address@hidden> wrote:

Hi,

I use Ubuntu 16.04 LTS 64-bit on my laptop, and I got a problem yesterday 08.01.2017.
When running make, both on v5.10 and v5.12, I get many messages like the ones shown below in this mail.

And when trying to build a fixed wing in v5.10, I get many messages in the console, like the ones at the bottom of this mail.

I am not sure if GCC was updated during the Ubuntu system update, but when I run gcc -v I get:
...
gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.5)
...

Could this problem be caused by changes that came with the Ubuntu update?

Thanks,
Helge.

WHEN RUNNING make:
...
BUILD   lib/stm32/f1
CP      libopencm3_stm32f1.ld
CC      adc.c
CC      adc_common_v1.c
CC      can.c
../can.c: In function 'can_transmit':
../can.c:417:18: warning: this statement may fall through [-Wimplicit-fallthrough=]
   tdhxr.data8[3] = data[7];
   ~~~~~~~~~~~~~~~^~~~~~~~~
../can.c:419:2: note: here
case 7:
^~~~
../can.c:420:18: warning: this statement may fall through [-Wimplicit-fallthrough=]
   tdhxr.data8[2] = data[6];
   ~~~~~~~~~~~~~~~^~~~~~~~~
...
...
...

CC      timer_common_all.c
../common/timer_common_all.c: In function 'timer_interrupt_source':
../common/timer_common_all.c:247:5: warning: this use of "defined" may not be portable [-Wexpansion-to-defined]
#if ADVANCED_TIMERS
     ^~~~~~~~~~~~~~~
../common/timer_common_all.c:247:5: warning: this use of "defined" may not be portable [-Wexpansion-to-defined]
../common/timer_common_all.c: In function 'timer_set_output_idle_state':
../common/timer_common_all.c:548:5: warning: this use of "defined" may not be portable [-Wexpansion-to-defined]
#if ADVANCED_TIMERS
     ^~~~~~~~~~~~~~~
...
...
...
BUILD   lib/stm32/f4
CP      libopencm3_stm32f4.ld
CC      adc.c
CC      adc_common_v1.c
CC      can.c
../can.c: In function 'can_transmit':
../can.c:417:18: warning: this statement may fall through [-Wimplicit-fallthrough=]
   tdhxr.data8[3] = data[7];
   ~~~~~~~~~~~~~~~^~~~~~~~~
../can.c:419:2: note: here
case 7:
^~~~
../can.c:420:18: warning: this statement may fall through [-Wimplicit-fallthrough=]
   tdhxr.data8[2] = data[6];
   ~~~~~~~~~~~~~~~^~~~~~~~~

WHEN BUILDING A FIXED WING:
...
...
...
CC /home/helge/paparazzi/var/aircrafts/SKYWALKERHW_TWOG_V1_SEN10121_IMU_MS5611/ap/firmwares/fixedwing/nav.o
CC /home/helge/paparazzi/var/aircrafts/SKYWALKERHW_TWOG_V1_SEN10121_IMU_MS5611/ap/subsystems/navigation/common_flight_plan.o
In file included from ./firmwares/fixedwing/nav.h:41:0,
                 from firmwares/fixedwing/nav.c:34:
/home/helge/paparazzi/var/aircrafts/SKYWALKERHW_TWOG_V1_SEN10121_IMU_MS5611/ap/generated/flight_plan.h: In function 'auto_nav':
./subsystems/navigation/common_flight_plan.h:44:21: warning: this statement may fall through [-Wimplicit-fallthrough=]
#define InitStage() nav_init_stage();
                     ^~~~~~~~~~~~~~~~
./subsystems/navigation/common_flight_plan.h:51:36: note: in expansion of macro 'InitStage'
#define NextStage() { nav_stage++; InitStage(); }
                                    ^~~~~~~~~
/home/helge/paparazzi/var/aircrafts/SKYWALKERHW_TWOG_V1_SEN10121_IMU_MS5611/ap/generated/flight_plan.h:163:9: note: in expansion of macro 'NextStage'
         NextStage();
         ^~~~~~~~~
./subsystems/navigation/common_flight_plan.h:55:18: note: here
#define Label(x) label_ ## x:
                  ^
/home/helge/paparazzi/var/aircrafts/SKYWALKERHW_TWOG_V1_SEN10121_IMU_MS5611/ap/generated/flight_plan.h:164:7: note: in expansion of macro 'Label'
       Label(while_1)
       ^~~~~
...
...
...

_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel

_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel

_______________________________________________
Paparazzi-devel mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/paparazzi-devel

From:	Michal Podhradsky
Subject:	Re: [Paparazzi-devel] Fwd: Problems with make after running software updater in Ubuntu.
Date:	Wed, 10 Jan 2018 09:25:43 -0800