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

Enhancement - exporter/cdt #4828

Merged
merged 3 commits into from Aug 10, 2017

Conversation

Projects
None yet
4 participants
@alzix
Contributor

alzix commented Jul 28, 2017

Description

  1. Rename Eclipse .launch file according to project name:
    Since Eclipse supports working with multiple projects in the same workspace, .launch file name should include project name for creating unique per-project .launch file.
  2. Fix CDT debugger settings on Linux - remove .exe suffix
    On Linux machines arm-none-eabi-gdb executable do not have .exe suffix.
    On Windows machines - it is possible to run an executable without explicit .exe suffix.
    This change fixes debugger settings for Linux hosts

Status

READY

Migrations

NO

Related PRs

N/A

Todos

N/A

Deploy notes

N/A

Steps to test or reproduce

On a Linux host execute following steps:

  1. cd mbed-os-example-blinky
  2. mbed export -m K64F -i eclipse_gcc_arm
  3. Launch eclipse
  4. Click File->Import->General->Existing Projects into Workspace
  5. Build
  6. Launch the debugger
@theotherjimmy

Nice. I like it.

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Jul 31, 2017

/morph export-build

@mbed-bot

This comment has been minimized.

mbed-bot commented Jul 31, 2017

Result: SUCCESS

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

/morph export-build

Output

mbed Build Number: 100

All exports and builds passed!

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Jul 31, 2017

@alzix Could you rebase to include #4839 and fix the travis build?

@theotherjimmy theotherjimmy added needs: work and removed needs: CI labels Jul 31, 2017

alzix added some commits Jul 28, 2017

Rename Eclipse .launch file according to project name
Since Eclipse supports working with multiple projects in the same workspace, .launch file name
should include project name for creating unique per-project .launch file.
Fix CDT debugger settings on Linux - remove .exe suffix
On Linux machines arm-none-eabi-gdb executable do not have .exe suffix.
On Windows machines - it is possible to run an executable without explicit .exe suffix.
This change fixes debugger settings for Linux hosts

@alzix alzix force-pushed the alzix:enhancement/exporter/cdt branch to 4b5535f Aug 1, 2017

@alzix

This comment has been minimized.

Contributor

alzix commented Aug 1, 2017

@theotherjimmy - please check again.

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Aug 1, 2017

Excelent. It passed travis. Thanks for the update.

@theotherjimmy theotherjimmy added needs: CI and removed needs: work labels Aug 1, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Aug 7, 2017

/morph export-build

@mbed-bot

This comment has been minimized.

mbed-bot commented Aug 7, 2017

Result: FAILURE

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

/morph export-build

Output

mbed Build Number: 114

Exporter Build failed!

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Aug 9, 2017

/morph export-build

@mbed-bot

This comment has been minimized.

mbed-bot commented Aug 9, 2017

Result: SUCCESS

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

/morph export-build

Output

mbed Build Number: 116

All exports and builds passed!

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Aug 9, 2017

@theotherjimmy theotherjimmy merged commit 0dcf71b into ARMmbed:master Aug 10, 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment