-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Conversation
@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. |
There was a problem hiding this 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 |
There was a problem hiding this comment.
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), |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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 .
There was a problem hiding this comment.
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 :)
There was a problem hiding this comment.
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.
There was a problem hiding this 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
retest uvisor |
Fixed issue where some deleted files/folders where not being removed from
the repo.
Removed code no longer used/needed.