Skip to content

Conversation

@phyokyaw
Copy link

Description

We have a Open OCD lunch file included in the exporter template. However Open OCD ID has changed

from gnuarmeclipse to gnumcueclipse. Hence this patch adds updates to existing launch file with new ID and also kept the old launch file for e2 studio 5.x compatibility.

Status

READY

Migrations

NO

Steps to test or reproduce

Export a project to e2 studio and import in e2 studio 6.x. Launch file doesn't appear on debug configurations dialog.

@mbed-ci
Copy link

mbed-ci commented Jan 25, 2018

Automatic CI verification build not done, please verify manually.

@phyokyaw
Copy link
Author

@theotherjimmy Please review if possible. Thank you.

Copy link
Contributor

@theotherjimmy theotherjimmy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@theotherjimmy theotherjimmy changed the title Updated exporter for e2 studio with new launch file for Open OCD Add OpenOCD 5x launch file to e2 studio and update naming Jan 25, 2018
@cmonr
Copy link
Contributor

cmonr commented Jan 25, 2018

/morph build

1 similar comment
@0xc0170
Copy link
Contributor

0xc0170 commented Jan 26, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Jan 26, 2018

Build : SUCCESS

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

Triggering tests

/morph test
/morph uvisor-test
/morph export-build

@mbed-ci
Copy link

mbed-ci commented Jan 26, 2018

@cmonr
Copy link
Contributor

cmonr commented Jan 26, 2018

Odd. Result didn't report back.
/morph export-build

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 29, 2018

/morph export-build

@mbed-ci
Copy link

mbed-ci commented Jan 29, 2018

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 29, 2018

/morph export-build

@mbed-ci
Copy link

mbed-ci commented Jan 29, 2018

@cmonr cmonr merged commit 6da5d54 into ARMmbed:master Jan 29, 2018
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.

6 participants