-
Notifications
You must be signed in to change notification settings - Fork 3k
Makefile: fix for overlong command line #12646
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
Makefile: fix for overlong command line #12646
Conversation
The list of object files was so long, that it got truncated by the bash (git-bash). Error was /usr/bin/sh: -c: line 0: unexpected EOF while looking for matching `"' /usr/bin/sh: -c: line 1: syntax error: unexpected end of file mingw32-make[1]: *** [Makefile:679: mbed-os-example-blinky-baremetal.elf] Error 1 mingw32-make: *** [Makefile:26: all] Error 2 Such a problem has aleady been reported in: #10943 (comment) I fixed this problem using this answer: https://stackoverflow.com/a/37506805/5534993
@dhebbeker, thank you for your changes. |
CI started |
Test run: FAILEDSummary: 1 of 7 test jobs failed Failed test jobs:
|
Known issue, will be fixed and test restarted |
@dhebbeker you're using a really old version of our PR template. Could you please update to the latest version ? E.g. like this one #12662 . Thanks |
In the meantime I will restart CI |
Test run: SUCCESSSummary: 7 of 7 test jobs passed |
Yes, somehow I mistakenly forked from an old fork of the original repository. I deleted my old fork. Thank you @adbridge for merging this PR! |
Hello @dhebbeker - can you please explain how I tested this on macOS (GNU make 3.81) and see an error:
The error persists when I add an extra brace at the end of $(file ) command. Basically this broke all exporters that depend on Makefile. The issue #10943 has a reference to 6918e6a which I introduced a year ago. The commit message provides an explanation:
|
@vmedcy, you are right, the change is not valid!
Also I did not read your commit 6918e6a which documents the issue in detail. This is bad, my apologies. I am going to create a PR to revert the changes right away. @adbridge, can you please re-open #10943 again, as it is not fixed by this issue? |
Thanks @vmedcy for the report and explanation, will revert the change |
Makefile: render compatible with some GNU make versions (revert #12646)
Description
The list of object files was so long, that it got truncated by the bash (git-bash).
Error was
This may fix #10943
I fixed this problem using this answer.
Pull request type
Reviewers
Release Notes