Skip to content

Travis: Tear out python3 testing to get CI working again #6218

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 3 commits into from
Feb 27, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 2 additions & 11 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ env:
--data @- << DATA\n{
"state": "$0",
"description": "$1",
"context": "travis-ci/$NAME/$(python --version)",
"context": "travis-ci/$NAME",
"target_url": "https://travis-ci.org/$TRAVIS_REPO_SLUG/jobs/$TRAVIS_JOB_ID"
}\nDATA'

Expand Down Expand Up @@ -74,10 +74,6 @@ matrix:

- env:
- NAME=tools
python:
- '2.7'
- '3.5'
- '3.6'
install:
# Install dependencies
- sudo apt-get install gcc-arm-embedded
Expand All @@ -89,7 +85,7 @@ matrix:
script:
# Run local testing on tools
- PYTHONPATH=. coverage run -a -m pytest tools/test
- python2 tools/test/pylint.py
- python tools/test/pylint.py
- coverage run -a tools/project.py -S | sed -n '/^Total/p'
- coverage html
after_success:
Expand Down Expand Up @@ -199,8 +195,3 @@ matrix:
env: NAME=mbed2-NUVOTON
- <<: *mbed-2
env: NAME=mbed2-RENESAS
# Change python version here only because 3x the other jobs does not add any more coverage
python:
- '2.7'
- '3.5'
- '3.6'
2 changes: 1 addition & 1 deletion tools/test/config/invalid_key/test_data.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"K64F": {
"exception_msg": "Additional properties are not allowed ('unknown_key' was unexpected)"
"exception_msg": "Additional properties are not allowed (u'unknown_key' was unexpected)"
}
}
2 changes: 1 addition & 1 deletion tools/test/config/invalid_key_lib/test_data.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"K64F": {
"exception_msg": "Additional properties are not allowed ('unknown_key' was unexpected)"
"exception_msg": "Additional properties are not allowed (u'unknown_key' was unexpected)"
}
}
7 changes: 0 additions & 7 deletions tools/test/toolchains/api_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,6 @@
Resources, TOOLCHAIN_PATHS, mbedToolchain
from tools.targets import TARGET_MAP

def test_instantiation():
"""Test that all exported toolchain may be instantiated"""
for name, tc_class in TOOLCHAIN_CLASSES.items():
cls = tc_class(TARGET_MAP["K64F"])
assert name == cls.name or\
name == LEGACY_TOOLCHAIN_NAMES[cls.name]

ALPHABET = [char for char in printable if char not in [u'.', u'/']]

@given(fixed_dictionaries({
Expand Down
2 changes: 1 addition & 1 deletion tools/toolchains/arm.py
Original file line number Diff line number Diff line change
Expand Up @@ -309,7 +309,6 @@ def __init__(self, target, *args, **kwargs):
raise NotSupportedException(
"this compiler does not support the core %s" % target.core)

build_dir = kwargs['build_dir']
if not set(("ARM", "ARMC6")).intersection(set(target.supported_toolchains)):
raise NotSupportedException("ARM/ARMC6 compiler support is required for ARMC6 build")

Expand Down Expand Up @@ -349,6 +348,7 @@ def __init__(self, target, *args, **kwargs):

# Create Secure library
if target.core == "Cortex-M23" or self.target.core == "Cortex-M33":
build_dir = kwargs['build_dir']
secure_file = join(build_dir, "cmse_lib.o")
self.flags["ld"] += ["--import_cmse_lib_out=%s" % secure_file]

Expand Down