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

Allow exporters for NUCLEO_L486RG #5084

Merged
merged 1 commit into from Oct 5, 2017

Conversation

Projects
None yet
5 participants
@adustm
Member

adustm commented Sep 12, 2017

Description

NUCLEO_L486RG is not handled by tool scripts for exporters.
With this PR we can export and debug this target.

Status

READY

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 12, 2017

@adustm Please be aware that SW4STM32 currently DOES NOT WORK CORRECTLY and that #4779 may clober your changes.

@adustm

This comment has been minimized.

Member

adustm commented Sep 12, 2017

Hello @theotherjimmy
Thanks for pointing, you're right.
I way wait for #4779 to be merged before rebasing and modifying the current one, if you like.
kind regards

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 14, 2017

Hello @adustm That's my preference! thanks.

@adustm adustm force-pushed the adustm:NUCLEO_L486_export branch to 4817f06 Sep 22, 2017

@adustm

This comment has been minimized.

Member

adustm commented Sep 22, 2017

Hello, rebase and rework done as agreed after the merge of #4779

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Oct 2, 2017

/morph export-build

@mbed-bot

This comment has been minimized.

mbed-bot commented Oct 3, 2017

Result: SUCCESS

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

/morph export-build

Output

mbed Build Number: 163

All exports and builds passed!

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Oct 3, 2017

@adbridge adbridge added needs: CI and removed ready for merge labels Oct 3, 2017

@adustm

This comment has been minimized.

Member

adustm commented Oct 3, 2017

Hello @adbridge @0xc0170
Could you let me know what is wrong with the continuous-integration/jenkins/pr-head check ?
Is there something I could do ?

Kind regards
Armelle

@adbridge

This comment has been minimized.

Contributor

adbridge commented Oct 3, 2017

@adustm I think they are having some problems with their CI! I will restart it.

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Oct 4, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Oct 4, 2017

all CI resolved

@theotherjimmy theotherjimmy merged commit 41998d5 into ARMmbed:master Oct 5, 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

@adustm adustm deleted the adustm:NUCLEO_L486_export branch Jan 15, 2018

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