Skip to content

Print the warning messages in debug mode #8222

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

Closed

Conversation

deepikabhavnani
Copy link

@deepikabhavnani deepikabhavnani commented Sep 21, 2018

Description

Warning string passed in MBED_WARN was not used, adding debug message to print it in debug mode

Pull request type

[X] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Breaking change

@cmonr
Copy link
Contributor

cmonr commented Oct 12, 2018

Running this through CI on its own.

Note: When this was attempted to be used in a rollup PR, it seems that this PR caused issues.

/morph build

@mbed-ci
Copy link

mbed-ci commented Oct 12, 2018

Build : SUCCESS

Build number : 3339
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/8222/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Oct 12, 2018

@mbed-ci
Copy link

mbed-ci commented Oct 12, 2018

@0xc0170
Copy link
Contributor

0xc0170 commented Oct 12, 2018

Looks like networking issue in our env, will restart later

@cmonr
Copy link
Contributor

cmonr commented Oct 12, 2018

@0xc0170

Note: When this was attempted to be used in a rollup PR, it seems that this PR caused issues.

This now confirms that the issue with the original rollup was with this PR. (Before: #8345 After: #8356)

For some reason, this change reliably hits the time_start bug with networked devices, and needs to be solved.

@deepikabhavnani
Copy link
Author

deepikabhavnani commented Oct 12, 2018

Please lower the priority of this PR, will look into this later next week

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.

7 participants