Skip to content

add circle ci #279

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 4 commits into from
Mar 3, 2020
Merged

add circle ci #279

merged 4 commits into from
Mar 3, 2020

Conversation

jamesbeyond
Copy link
Contributor

Add circle ci for building ble example

@jamesbeyond jamesbeyond requested a review from pan- February 21, 2020 22:53
find . -name mbed-os.lib -delete
mbed deploy
for i in BLE_* ; do cd $i ; ln -s $PWD/../../mbed-os mbed-os ; ln -s $PWD/../../BUILD BUILD ; cd .. ; done
for i in BLE_* ; do cd $i ; mbed compile -t GCC_ARM -m NRF52840_DK || break ; cd .. ; done
Copy link
Member

Choose a reason for hiding this comment

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

Can we compile for DISCO_L475VG_IOT01A as well ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added DISCO target

- checkout:
path: mbed-os-example-ble
- run: |
git clone https://github.com/ARMmbed/mbed-os.git
Copy link
Member

Choose a reason for hiding this comment

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

We expect master to always be compatible with the head of the repo.
But it doesn't mean the release branch will be compatible with the head of the repo in case of breaking change.

mkdir BUILD
cd mbed-os-example-ble
find . -name mbed-os.lib -delete
mbed deploy
Copy link
Member

Choose a reason for hiding this comment

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

maybe we can run deploy in each example to avoid running it in the deprecated folder.

@jamesbeyond
Copy link
Contributor Author

Hi @pan- I update the configuration as you requested.

  • current mbed-os.lib been update to matching mbed-os master
  • I have to use mbed update command before each build, seem the mbed update triggers mbed deploy as well
  • mbed update been added before each example build, just in case if mbed-os.lib in each example pointing to different mbed-os SHA

@jamesbeyond
Copy link
Contributor Author

ping @pan- @donatieng

Copy link
Member

@pan- pan- left a comment

Choose a reason for hiding this comment

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

LGTM

@pan- pan- merged commit 028e185 into ARMmbed:master Mar 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants