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

Shell uart0 newlib distinction #3555

Closed

Conversation

OlegHahm
Copy link
Member

@OlegHahm OlegHahm commented Aug 4, 2015

Workaround for #3550 as long as uart0 is not phased out.

@OlegHahm OlegHahm added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Platform: ARM Platform: This PR/issue effects ARM-based platforms labels Aug 4, 2015
@OlegHahm OlegHahm added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Aug 4, 2015
@OlegHahm OlegHahm added the CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable label Aug 4, 2015
OlegHahm added a commit to OlegHahm/applications that referenced this pull request Aug 4, 2015
@haukepetersen
Copy link
Contributor

shouldn't #3402 fix this situation for all platforms?

@OlegHahm
Copy link
Member Author

OlegHahm commented Aug 5, 2015

Only if all platforms support getchar() which is currently not the case.

@OlegHahm
Copy link
Member Author

OlegHahm commented Aug 5, 2015

Agreed, #3402 would be the nicer way, but seems on some platforms. (Builds, but won't react to input any more.)

@haukepetersen
Copy link
Contributor

As we are fixing (or waiting for Travis to accept our fixes) these errors, we go with #3402, right? So should we close this PR?

@OlegHahm
Copy link
Member Author

OlegHahm commented Aug 5, 2015

Agreed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: needs squashing Commits in this PR need to be squashed; If set, CI systems will mark this PR as unmergable CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants