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

Feature Request: Configure "platformio device monitor" from platformio.ini #787

Closed
blackketter opened this Issue Sep 18, 2016 · 9 comments

Comments

Projects
None yet
2 participants
@blackketter

blackketter commented Sep 18, 2016

It should be possible to configure “platformio device monitor” options from platformio.ini

I'd love to be able to add a couple lines to my platformio.ini file like:

monitor_port = /dev/cu.wchusbserial1424120
monitor_speed = 9600

and this would configure "platformio device monitor" on a per-target basis.

Ideally, all the "device monitor" options would be available this way.

Bonus idea: "platformio run -t monitor" would monitor, similarly to uploading via "platformio run -t upload"

@ivankravets ivankravets added the feature label Sep 18, 2016

@blackketter

This comment has been minimized.

Show comment
Hide comment
@blackketter

blackketter Sep 18, 2016

To elaborate on the last idea, I think it would be awesome to be able to do something like this:

platformio run -e nodemcu -t clean -t upload -t monitor

Which would clean, build, upload and then monitor the nodemcu environment. (Maybe this should be a separate feature request?)

blackketter commented Sep 18, 2016

To elaborate on the last idea, I think it would be awesome to be able to do something like this:

platformio run -e nodemcu -t clean -t upload -t monitor

Which would clean, build, upload and then monitor the nodemcu environment. (Maybe this should be a separate feature request?)

@ivankravets

This comment has been minimized.

Show comment
Hide comment
@ivankravets

ivankravets Sep 18, 2016

Member

pio run is responsible for building and uploading. I'm not sure that we should mix pio device commands with pio run. If you need that, you can create alias and add to .bashrc

alias piobatch="pio -t clean && pio -t upload && pio device monitor"
Member

ivankravets commented Sep 18, 2016

pio run is responsible for building and uploading. I'm not sure that we should mix pio device commands with pio run. If you need that, you can create alias and add to .bashrc

alias piobatch="pio -t clean && pio -t upload && pio device monitor"
@blackketter

This comment has been minimized.

Show comment
Hide comment
@blackketter

blackketter Sep 18, 2016

Fair enough, I already have bunch of similar aliases. :)

I figured it would be generally useful for folks to be able to do this kind of build/install/monitor cycle within a single command and it wasn't obvious to me why pio run uploads but doesn't monitor.

I guess then, we'd need to have a -e option on pio device monitor so it knew which environment to pull the monitor options from in platformio.ini.

So the with an environment-aware device monitor the alias would be:

alias pionodebatch="pio -e nodemcu -t clean && pio -e nodemcu -t upload && pio -e nodemcu device monitor "

blackketter commented Sep 18, 2016

Fair enough, I already have bunch of similar aliases. :)

I figured it would be generally useful for folks to be able to do this kind of build/install/monitor cycle within a single command and it wasn't obvious to me why pio run uploads but doesn't monitor.

I guess then, we'd need to have a -e option on pio device monitor so it knew which environment to pull the monitor options from in platformio.ini.

So the with an environment-aware device monitor the alias would be:

alias pionodebatch="pio -e nodemcu -t clean && pio -e nodemcu -t upload && pio -e nodemcu device monitor "

@ivankravets

This comment has been minimized.

Show comment
Hide comment
@ivankravets

ivankravets Sep 18, 2016

Member

Good, please open a separate issue. I think we will find solution for that 😊

P.S: How is PlatformIO 3.0? Do you have problems with it?

Member

ivankravets commented Sep 18, 2016

Good, please open a separate issue. I think we will find solution for that 😊

P.S: How is PlatformIO 3.0? Do you have problems with it?

@blackketter

This comment has been minimized.

Show comment
Hide comment
@blackketter

blackketter Sep 18, 2016

PlatformIO 3.0 seems great so far, no problems! Great job!

blackketter commented Sep 18, 2016

PlatformIO 3.0 seems great so far, no problems! Great job!

@ivankravets

This comment has been minimized.

Show comment
Hide comment
@ivankravets

ivankravets Sep 18, 2016

Member

Thanks!!! PlatformIO 3.1 is coming this week! :)

Member

ivankravets commented Sep 18, 2016

Thanks!!! PlatformIO 3.1 is coming this week! :)

@ivankravets

This comment has been minimized.

Show comment
Hide comment
Member

ivankravets commented Sep 18, 2016

@blackketter

This comment has been minimized.

Show comment
Hide comment
@blackketter

blackketter Sep 18, 2016

SWEET!

On Sep 18, 2016, at 2:45 PM, Ivan Kravets notifications@github.com wrote:

Dynamic variables http://docs.platformio.org/en/latest/projectconf.html#dynamic-variables http://docs.platformio.org/en/latest/projectconf.html#dynamic-variables
Also, +50 new boards :)


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #787 (comment), or mute the thread https://github.com/notifications/unsubscribe-auth/AAICX9ANtwSlnx9Yj6mOslAS-pFBY2PCks5qrbDvgaJpZM4KAAjf.

blackketter commented Sep 18, 2016

SWEET!

On Sep 18, 2016, at 2:45 PM, Ivan Kravets notifications@github.com wrote:

Dynamic variables http://docs.platformio.org/en/latest/projectconf.html#dynamic-variables http://docs.platformio.org/en/latest/projectconf.html#dynamic-variables
Also, +50 new boards :)


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #787 (comment), or mute the thread https://github.com/notifications/unsubscribe-auth/AAICX9ANtwSlnx9Yj6mOslAS-pFBY2PCks5qrbDvgaJpZM4KAAjf.

@ivankravets

This comment has been minimized.

Show comment
Hide comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment