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

No error when referred parameter is missing (using extends) #3279

Closed
TD-er opened this issue Nov 19, 2019 · 2 comments
Closed

No error when referred parameter is missing (using extends) #3279

TD-er opened this issue Nov 19, 2019 · 2 comments
Assignees
Labels
bug config platformio.ini
Milestone

Comments

@TD-er
Copy link

TD-er commented Nov 19, 2019

PIO version:

  • Home 3.1.0-beta.1
  • Core 4.1.1b1

I tried to get my platformio.ini file a lot more structured by using extends.

Just a section of my ini file:

[common]
lib_ldf_mode              = chain
lib_archive               = false
framework                 = arduino
upload_speed              = 115200
monitor_speed             = 115200
;targets                   = size, checkprogsize
targets                   =

[regular_platform]
build_unflags             =
build_flags               = ${core_2_6_1.build_flags} ${common.build_flags}
platform                  = ${core_2_6_1.platform}

[hard_esp82xx]
platform                  = ${regular_platform.platform}
build_flags               = ${regular_platform.build_flags}
                            -DBUILD_NO_DEBUG
                            -DPLUGIN_BUILD_CUSTOM

As you can see, the ${regular_platform.build_flags} refers ${common.build_flags}, which doesn't exist.
The result is no error, but it has the build_flags empty.

Part of the verbose output:

Processing hard_SONOFF_POW_4M1M (extends: esp8266_4M1M, hard_esp82xx; platform: https://github.com/Jason2866/platform-espressif8266.git#core_2_6_1; build_flags: ; board: esp12e; board_build.flash_mode: dio; board_upload.maximum_size: 1044464; board_build.f_cpu: 80000000L; build_unflags: -DDEBUG_ESP_PORT; lib_deps: https://github.com/TD-er/ESPEasySerial.git; lib_ignore: ESP32_ping, ESP32WebServer, IRremoteESP8266, HeatpumpIR, SD(esp8266), SDFS; lib_ldf_mode: chain; lib_archive: False; framework: arduino; upload_speed: 115200; monitor_speed: 115200; targets: )
@ivankravets ivankravets added this to the 4.1.1 milestone Nov 28, 2019
@ivankravets ivankravets added bug config platformio.ini labels Feb 7, 2020
@ivankravets
Copy link
Member

Thanks! Please re-test with pio upgrade --dev.

@TD-er
Copy link
Author

TD-er commented Feb 27, 2020

I forgot to reply (and test), but now I have tested it (by accident ;) ) and it works :)

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug config platformio.ini
Projects
None yet
Development

No branches or pull requests

3 participants