Skip to content

Disable response files on export #5093

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

Merged
merged 3 commits into from
Sep 14, 2017

Conversation

theotherjimmy
Copy link
Contributor

None of the exporters can handle them, so just don't do it

Resolves #5092

None of the exporters can handle them, so just don't do it
@theotherjimmy
Copy link
Contributor Author

/morph export-build

@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph export-build

Output

mbed Build Number: 141

Exporter Build failed!

@theotherjimmy
Copy link
Contributor Author

17:37:26 startup_LPC17xx.S: error: A3907U: Via file '.\build\startup_lpc17xx._ia' command too long for buffer.

... oh boy

@theotherjimmy
Copy link
Contributor Author

/morph export-build

@theotherjimmy theotherjimmy force-pushed the export-no-response-files branch from 6806046 to a62f89d Compare September 14, 2017 14:39
@mbed-bot
Copy link

Result: FAILURE

Your command has finished executing! Here's what you wrote!

/morph export-build

Output

mbed Build Number: 143

Build Prep failed!

@theotherjimmy
Copy link
Contributor Author

Turns out if you rebase while /morph export-bulid is cloning it fails! Who knew? 😁

@theotherjimmy
Copy link
Contributor Author

/morph export-build

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph export-build

Output

mbed Build Number: 144

All exports and builds passed!

@theotherjimmy
Copy link
Contributor Author

👍

@theotherjimmy
Copy link
Contributor Author

Time to merge.

@theotherjimmy theotherjimmy merged commit 0906be6 into ARMmbed:master Sep 14, 2017
@theotherjimmy theotherjimmy deleted the export-no-response-files branch September 14, 2017 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants