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

Configuration file parameters partly ignored when using --project-conf #3034

Closed
daniel-starke opened this issue Sep 18, 2019 · 5 comments

Comments

@daniel-starke
Copy link

commented Sep 18, 2019

PlatformIO Core

Configuration

Operating system: Windows 7

PlatformIO Version (platformio --version): 4.0.3

Description of problem

platformio section is not evaluated if a non-default configuration file is provided with -c.

Steps to Reproduce

  1. platformio run -c test.ini

Actual Results

.pio is used as default.

Expected Results

The path defined in workspace_dir is used.

Problems with PlatformIO Build System:

The content of test.ini:

[platformio]
workspace_dir = bin

[env:wavrec]
platform = espressif32
framework = arduino
board = wemosbat
monitor_speed = 115200

Source file to reproduce issue:

setup(void) {}
loop(void) {}

Additional info

Defining the environment variable PLATFORMIO_WORKSPACE_DIR works regardlessly.

@ivankravets ivankravets added this to the 4.1.0 milestone Sep 20, 2019
@ivankravets ivankravets added the bug label Sep 20, 2019
ivankravets added a commit that referenced this issue Sep 26, 2019
…ing custom ``--project-conf`` // Resolve #3034
@ivankravets

This comment has been minimized.

Copy link
Member

commented Sep 27, 2019

Oh, this issue took a lot of efforts, but it worth it. See d2abac9

Could you re-test with pio upgrade --dev?

@daniel-starke

This comment has been minimized.

Copy link
Author

commented Sep 30, 2019

Thank you. Works for me as expected.

@ivankravets

This comment has been minimized.

Copy link
Member

commented Sep 30, 2019

Thanks for the report!

Happy coding with PlatformIO!

@daniel-starke

This comment has been minimized.

Copy link
Author

commented Sep 30, 2019

Hm... would had been nice to have a way to "upgrade" back to stable...

@ivankravets

This comment has been minimized.

Copy link
Member

commented Sep 30, 2019

pip uninstall platformio
pip install platformio

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