Skip to content

Updated to allow for new directory structure for mbed-dev source. #3689

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 1 commit into from
Feb 7, 2017

Conversation

adbridge
Copy link
Contributor

@adbridge adbridge commented Feb 3, 2017

Fixed issue where some deleted files/folders where not being removed from
the repo.
Removed code no longer used/needed.

@adbridge
Copy link
Contributor Author

adbridge commented Feb 3, 2017

@bridadan @theotherjimmy Please just review changes added, rest of the script would also need some work if this was going to be maintained long term, however the plan is to deprecate in the nearish future so just enough has been done to make this work again.

Copy link
Contributor

@bridadan bridadan left a comment

Choose a reason for hiding this comment

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

I think an unwanted file snuck in here. @0xc0170 Do you use the sync.py script any more? Might be interested in these changes then.

Other than that should be ok I think.

test_file.h Outdated
@@ -0,0 +1,104 @@
/* mbed Microcontroller Library
Copy link
Contributor

Choose a reason for hiding this comment

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

This file wasn't meant to be included was it?

@@ -46,60 +46,8 @@
# Code that does have a mirror in the mbed SDK
# Tuple data: (repo_name, list_of_code_dirs, [team])
# team is optional - if not specified, the code is published under mbed_official
OFFICIAL_CODE = (
("mbed-dev" , [MBED_DRIVERS, MBED_PLATFORM, MBED_HAL]),
("mbed-rtos", RTOS),
Copy link
Contributor

Choose a reason for hiding this comment

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

@adbridge are we still syncing the mbed-rtos library?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

arggh neither of the latter two commits should have come in, they came as a result of playing with something else! Will remove and re-do this PR !!No we no longer synchronise mbed-rtos .

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh and no @0xc0170 wont use the synch.py any more , only used during releasing and I do most of that now :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh and no @0xc0170 wont use the synch.py any more , only used during releasing and I do most of that now :)

+1

 Fixed issue where some deleted files/folders where not being removed from
 the repo.
 Removed code no longer used/needed.
Copy link
Contributor

@bridadan bridadan left a comment

Choose a reason for hiding this comment

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

Thanks for the follow up @adbridge, LGTM

@0xc0170
Copy link
Contributor

0xc0170 commented Feb 6, 2017

retest uvisor

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.

5 participants