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

Moved 'Dropping' print from stdout to stderr in drop_versions.py #6071

Merged
merged 3 commits into from
May 12, 2019
Merged

Moved 'Dropping' print from stdout to stderr in drop_versions.py #6071

merged 3 commits into from
May 12, 2019

Conversation

Subhajitdas298
Copy link
Contributor

Printing 'Dropping' debug strings to stdout leads to write in package json.
This moved that to stderr stream.
Alternatively it can be removed.

Printing 'Dropping' debug strings to stdout leads to write in package json.
This moved that to stderr stream.
Copy link
Collaborator

@d-a-v d-a-v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

better than #6070, thanks (the other will be merged first, CI nearly finished and urgency ahead :)

@d-a-v d-a-v requested a review from earlephilhower May 11, 2019 22:42
Copy link
Collaborator

@earlephilhower earlephilhower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for not only finding the problem, but also offering a solution! Can you see if it's possible to make this version agnostic?

package/drop_versions.py Show resolved Hide resolved
@earlephilhower earlephilhower dismissed their stale review May 12, 2019 13:53

My mistake, sorry!

@earlephilhower earlephilhower merged commit 9b8d4cc into esp8266:master May 12, 2019
@Subhajitdas298 Subhajitdas298 deleted the patch-1 branch May 12, 2019 13:59
earlephilhower added a commit to earlephilhower/Arduino that referenced this pull request May 20, 2019
Changes since 2.5.1 (to 2.5.2)

Core
----
* Add explicit Print::write(char) (esp8266#6101)

Build system
----
* Fix typo in elf2bin for QOUT binary generation (esp8266#6116)
* Support PIO Wl-T and Arduino -T linking properly (esp8266#6095)
* Allow *.cc files to be linked into flash by default (esp8266#6100)
* Use custom "ElfToBin" builder for PIO (esp8266#6091)
* Fail if generated JSON file cannot be read (esp8266#6076)
* Moved 'Dropping' print from stdout to stderr in drop_versions.py (esp8266#6071)
* Fix PIO issue when build environment contains spaces (esp8266#6119)

Libraries
----
* Remove deadlock when server is not acking our data (esp8266#6107)
* Bugfix for stuck in write method of WiFiClient and WiFiClientSecure until the remote peer closed connection (esp8266#6104)
* Re-add original SD FAT info access methods (esp8266#6092)
* Make FILE_WRITE append in SD.h wrapper (esp8266#6106)
* Drop X509 after connection, avoid hang on TLS broken (esp8266#6065)
@earlephilhower earlephilhower mentioned this pull request May 20, 2019
earlephilhower added a commit that referenced this pull request May 20, 2019
Changes since 2.5.1 (to 2.5.2)

Core
----
* Add explicit Print::write(char) (#6101)

Build system
----
* Fix typo in elf2bin for QOUT binary generation (#6116)
* Support PIO Wl-T and Arduino -T linking properly (#6095)
* Allow *.cc files to be linked into flash by default (#6100)
* Use custom "ElfToBin" builder for PIO (#6091)
* Fail if generated JSON file cannot be read (#6076)
* Moved 'Dropping' print from stdout to stderr in drop_versions.py (#6071)
* Fix PIO issue when build environment contains spaces (#6119)

Libraries
----
* Remove deadlock when server is not acking our data (#6107)
* Bugfix for stuck in write method of WiFiClient and WiFiClientSecure until the remote peer closed connection (#6104)
* Re-add original SD FAT info access methods (#6092)
* Make FILE_WRITE append in SD.h wrapper (#6106)
* Drop X509 after connection, avoid hang on TLS broken (#6065)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants