Skip to content

Commit a18a739

Browse files
committed
travis: Reject changes to tools
As part of freezing the current tools, reject any changes to tools that would require a re-release of the tools for the online compiler. This is to free up our effort on existing tool releases which are very error prone, time consuming, and labor intensive, so we can focus on developing a far more maintainable set of tools. As the online compiler contains a copy of the whole mbed-os folder, it is not clear which parts of mbed-os are actually used. This change estimates the dependency between the online compiler, online IDE, and Mbed OS by searching for "from tools" in Python code in the os.mbed.com repo (https://github.com/ARMmbed/os.mbed.com/search?q=%22from+tools%22&unscoped_q=%22from+tools%22) For further context and explanation on why we've frozen the legacy tools, please see the blog post at https://os.mbed.com/blog/entry/Introducing-the-new-Mbed-Tools/
1 parent 5ea5159 commit a18a739

File tree

1 file changed

+28
-0
lines changed

1 file changed

+28
-0
lines changed

.travis.yml

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -176,6 +176,34 @@ matrix:
176176
find -name "*.s" | tee BUILD/badasm |
177177
sed -e "s/^/Bad Assembler file name found: /" && [ ! -s BUILD/badasm ]
178178
179+
- <<: *docs-vm
180+
name: "Frozen tools check"
181+
env: NAME=frozen_tools_check
182+
script:
183+
# Reject any changes to tools that would require a re-release of the
184+
# tools for the online compiler.
185+
- >-
186+
frozen_files=`\
187+
git diff --name-only --diff-filter=d FETCH_HEAD..HEAD \
188+
| egrep \
189+
-e "^tools/build_api*" \
190+
-e "^tools/config*" \
191+
-e "^tools/export*" \
192+
-e "^tools/notifier*" \
193+
-e "^tools/paths*" \
194+
-e "^tools/resources*" \
195+
-e "^tools/targets*" \
196+
-e "^tools/toolchains*" \
197+
-e "^tools/utils*" \
198+
-e "^$"`
199+
if [ -z "$frozen_files" ]; then
200+
echo "Success!";
201+
else
202+
echo -e "Failure: Frozen files were modified\n$frozen_files";
203+
echo -e "Please see https://os.mbed.com/blog/entry/Introducing-the-new-Mbed-Tools/" \
204+
"\nfor why we've frozen the legacy tools.";
205+
false;
206+
fi
179207
180208
### Python Tests ###
181209
- &pytools-vm

0 commit comments

Comments
 (0)