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

Disable response files on export #5093

Merged
merged 3 commits into from Sep 14, 2017

Conversation

Projects
None yet
2 participants
@theotherjimmy
Contributor

theotherjimmy commented Sep 13, 2017

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

Resolves #5092

Disable response files on export
None of the exporters can handle them, so just don't do it
@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 13, 2017

/morph export-build

@mbed-bot

This comment has been minimized.

mbed-bot commented Sep 13, 2017

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 141

Exporter Build failed!

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 14, 2017

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

... oh boy

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 14, 2017

/morph export-build

@theotherjimmy theotherjimmy force-pushed the theotherjimmy:export-no-response-files branch to a62f89d Sep 14, 2017

@mbed-bot

This comment has been minimized.

mbed-bot commented Sep 14, 2017

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 143

Build Prep failed!

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 14, 2017

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

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 14, 2017

/morph export-build

@mbed-bot

This comment has been minimized.

mbed-bot commented Sep 14, 2017

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

This comment has been minimized.

Contributor

theotherjimmy commented Sep 14, 2017

👍

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 14, 2017

Time to merge.

@theotherjimmy theotherjimmy merged commit 0906be6 into ARMmbed:master Sep 14, 2017

4 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-export-build Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@theotherjimmy theotherjimmy deleted the theotherjimmy:export-no-response-files branch Sep 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment