Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Failing to build the firmware with -werror #1054

Closed
samyduc opened this issue Jul 3, 2016 · 4 comments

Comments

@samyduc
Copy link

commented Jul 3, 2016

On a new setup:
Win10 with the latest tool chain / source.

Failing to build the firmware from scratch with -werror. Here we go, error: "BYTE_ORDER" redefined :

`In file included from ./src/photon/wiced/network/LwIP/WWD/FreeRTOS/arch/cc.h:69:0,
from ./src/photon/wiced/network/LwIP/ver1.4.0.rc1/src/include/lwip/arch.h:43,
from ./src/photon/wiced/network/LwIP/ver1.4.0.rc1/src/include/lwip/debug.h:35,
from ./src/photon/wiced/network/LwIP/ver1.4.0.rc1/src/include/lwip/opt.h:46,
from ./src/photon/wiced/network/LwIP/ver1.4.0.rc1/src/include/ipv4/lwip/ip_addr.h:35,
from ./src/photon/wiced/network/LwIP/WICED/wiced_network.h:43,
from ./src/photon/include/wiced_tcpip.h:44,
from ./src/photon/include/wiced.h:46,
from src/photon/core_hal.c:28:
./src/photon/wiced/network/LwIP/WWD/FreeRTOS/cpu.h:43:0: error: "BYTE_ORDER" redefined [-Werror]
#define BYTE_ORDER LITTLE_ENDIAN
^

In file included from c:\program files (x86)\gnu tools arm embedded\5.4 2016q2\arm-none-eabi\include\sys\types.h:67:0,
from c:\program files (x86)\gnu tools arm embedded\5.4 2016q2\arm-none-eabi\include\stdio.h:48,
from ./src/photon/wiced/WWD/include/wwd_debug.h:40,
from ./src/photon/wiced/WWD/include/wwd_assert.h:42,
from ./src/photon/wiced/RTOS/FreeRTOS/WWD/ARM_CM3/FreeRTOSConfig.h:41,
from ./src/photon/wiced/RTOS/FreeRTOS/ver7.5.2/Source/include/FreeRTOS.h:78,
from ./src/photon/wiced/RTOS/FreeRTOS/WWD/wwd_rtos.h:46,
from ./src/photon/wiced/WWD/include/RTOS/wwd_rtos_interface.h:47,
from ./src/photon/wiced/platform/MCU/STM32F2xx/peripherals/platform_mcu_peripheral.h:56,
from ./src/photon/wiced/platform/include/platform_peripheral.h:42,
from ./src/photon/include/wiced_platform.h:46,
from ./src/photon/include/wiced.h:44,
from src/photon/core_hal.c:28:
c:\program files (x86)\gnu tools arm embedded\5.4 2016q2\arm-none-eabi\include\machine\endian.h:20:0: note: this is the location of the previous definition
#define BYTE_ORDER _BYTE_ORDER
^

cc1.exe: all warnings being treated as errors
make[2]: *** [../build/target/hal/platform-6-m/./src/photon/core_hal.o] Error 1
make[2]: Leaving directory /c/dev/photon/firmware/hal' make[1]: *** [hal] Error 2 make[1]: Leaving directory/c/dev/photon/firmware/modules/photon/system-part1'
make: *** [/c/dev/photon/firmware/modules/photon/system-part1/makefile] Error 2`

@m-mcgowan

This comment has been minimized.

Copy link
Contributor

commented Jul 3, 2016

Thanks for the issue report. I imagine this is because you are using arm-gcc 5.4. Please use an earlier version of arm-none-eabi - 5.4 has only just been released and clearly breaks some things in our toolchain.

@m-mcgowan m-mcgowan added this to the 0.7.x milestone Jul 3, 2016

@samyduc

This comment has been minimized.

Copy link
Author

commented Jul 3, 2016

It works by downgrading to 5.3 . Thanks.

@Babbleshack

This comment has been minimized.

Copy link

commented Aug 6, 2016

This issue persists for me, even after downgrading.

I am using Arch Linux.
I have tried with toolchains 5.1/5.3/6.1.1 and the latest source.

@jhodapp

This comment has been minimized.

Copy link

commented Sep 10, 2016

Any chance of getting this fixed with gcc-5.4? I use the PPA from the linaro gcc Launchpad page and there isn't an older build previous to gcc-5.4. How often does the Particle test to make sure their firmware is up to date with the latest gcc?

@m-mcgowan m-mcgowan referenced this issue Sep 13, 2016
3 of 3 tasks complete

@sergeuz sergeuz self-assigned this Nov 6, 2016

@sergeuz sergeuz referenced this issue Nov 6, 2016
6 of 7 tasks complete

@technobly technobly modified the milestones: 0.7.x, 0.6.1 Nov 29, 2016

@technobly technobly closed this Nov 29, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.