[riot-notifications] [RIOT-OS/RIOT] makefiles: do not remove defines from CFLAGS (#12262)

Gaëtan Harter notifications at github.com
Tue Sep 17 16:54:25 CEST 2019


This works for the #12219 case with `DOCKER_ENVIRONMENT_CMDLINE`:

The `space` is still escaped when defined from the environment. There may be another way, but I do not find it.

```
env RIOT_CI_BUILD=1 QUIET=0 DOCKER="sudo docker"  BUILD_IN_DOCKER=1 DOCKER_ENVIRONMENT_CMDLINE=$'-e USEMODULE=esp_wifi -e CFLAGS=-DESP_WIFI_SSID=\'\\\"has\ space\\\"\'' make BOARD=esp32-wroom-32 -C tests/lwip
```


And the `CFLAGS` and the `riotbuild.h` have the expected value

```
grep SSID tests/lwip/bin/esp32-wroom-32/riotbuild/riotbuild.h  
/* Generated from CFLAGS: -DESP_WIFI_SSID="has space" -DDEVELHELP -Werror -DSCHED_PRIO_LEVELS=32 -DSDK_NOT_USED -DCONFIG_FREERTOS_UNICORE=1 -DESP_PLATFORM -DLOG_TAG_IN_BRACKETS -Wno-unused-parameter -Wformat=0 -mlongcalls -mtext-section-literals -fstrict-volatile-bitfields -fdata-sections -ffunction-sections -fzero-initialized-in-bss -DSCHED_PRIO_LEVELS=32 -Os -DFLASH_MODE_DOUT=1 -DFLASH_MODE_DOUT=1 -DFLASH_MODE_DOUT=1 -DFLASH_MODE_DOUT=1 -DFLASH_MODE_DOUT=1 -DFLASH_MODE_DOUT=1 -DRIOT_APPLICATION="tests_lwip" -DBOARD_ESP32_WROOM_32="esp32-wroom-32" -DRIOT_BOARD=BOARD_ESP32_WROOM_32 -DCPU_ESP32="esp32" -DRIOT_CPU=CPU_ESP32 -DMCU_ESP32="esp32" -DRIOT_MCU=MCU_ESP32 -std=c99 -fno-common -Wall -Wextra -Wmissing-include-dirs -fno-delete-null-pointer-checks -fdiagnostics-color -Wstrict-prototypes -Wold-style-definition -Wformat=2 -DSOCK_HAS_IPV6 -include /data/riotbuild/riotbase/tests/lwip/bin/esp32-wroom-32/riotbuild/riotbuild.h -DRIOT_VERSION="buildtest" -DMODULE_AUTO_INIT -DMODULE_BOARD -DMODULE_BOARDS_COMMON_ESP32 -DMODULE_CORE -DMODULE_CORE_MBOX -DMODULE_CORE_MSG -DMODULE_CPU -DMODULE_DIV -DMODULE_ESP_IDF -DMODULE_ESP_IDF_DRIVER -DMODULE_ESP_IDF_ESP32 -DMODULE_ESP_IDF_HEAP -DMODULE_ESP_IDF_NVS_FLASH -DMODULE_ESP_IDF_SOC -DMODULE_ESP_IDF_SPI_FLASH -DMODULE_ESP_IDF_WPA_SUPPLICANT_CRYPTO -DMODULE_ESP_IDF_WPA_SUPPLICANT_PORT -DMODULE_ESP_WIFI -DMODULE_ESP_WIFI_ANY -DMODULE_FMT -DMODULE_GNRC -DMODULE_GNRC_NETAPI -DMODULE_GNRC_NETIF -DMODULE_GNRC_NETIF_ETHERNET -DMODULE_GNRC_NETIF_HDR -DMODULE_GNRC_NETREG -DMODULE_GNRC_PKT -DMODULE_GNRC_PKTBUF -DMODULE_GNRC_PKTBUF_STATIC -DMODULE_IPV6_ADDR -DMODULE_ISRPIPE -DMODULE_L2UTIL -DMODULE_LOG -DMODULE_LUID -DMODULE_LWIP -DMODULE_LWIP_API -DMODULE_LWIP_CONTRIB -DMODULE_LWIP_CORE -DMODULE_LWIP_ETHERNET -DMODULE_LWIP_IPV6 -DMODULE_LWIP_IPV6_AUTOCONFIG -DMODULE_LWIP_NETDEV -DMODULE_LWIP_NETIF -DMODULE_LWIP_RAW -DMODULE_LWIP_SOCK -DMODULE_LWIP_SOCK_IP -DMODULE_LWIP_SOCK_TCP -DMODULE_LWIP_SOCK_UDP -DMODULE_LWIP_TCP -DMODULE_LWIP_UDP -DMODULE_MTD -DMODULE_NETDEV_DEFAULT -DMODULE_NETDEV_ETH -DMODULE_NETIF -DMODULE_NETOPT -DMODULE_NEWLIB -DMODULE_NEWLIB_SYSCALLS_DEFAULT -DMODULE_OD -DMODULE_PERIPH -DMODULE_PERIPH_ADC_CTRL -DMODULE_PERIPH_COMMON -DMODULE_PERIPH_CPUID -DMODULE_PERIPH_FLASH -DMODULE_PERIPH_GPIO -DMODULE_PERIPH_HWRNG -DMODULE_PERIPH_PM -DMODULE_PERIPH_RTC -DMODULE_PERIPH_TIMER -DMODULE_PERIPH_UART -DMODULE_PRNG -DMODULE_PRNG_TINYMT32 -DMODULE_PS -DMODULE_PTHREAD -DMODULE_RANDOM -DMODULE_RIOT_FREERTOS -DMODULE_SEMA -DMODULE_SHELL -DMODULE_SHELL_COMMANDS -DMODULE_SOCK_IP -DMODULE_SOCK_TCP -DMODULE_SOCK_UDP -DMODULE_STDIN -DMODULE_STDIO_UART -DMODULE_STDIO_UART_RX -DMODULE_SYS -DMODULE_TIMEX -DMODULE_TINYMT32 -DMODULE_TSRB -DMODULE_XTENSA -DMODULE_XTIMER */
#define ESP_WIFI_SSID "has space"
```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/12262#issuecomment-532257908
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190917/cd4df5e6/attachment.htm>


More information about the notifications mailing list