Skip to content

Syncronize mbed-ls minimum version with icetea requirements #8075

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
Sep 14, 2018

Conversation

OPpuolitaival
Copy link
Contributor

Description

Fix for: #8064

Root cause is that icetea, mbed-flasher requires newer mbed-ls than mbed-os and mbed-cli install requirements based on mbed-os requirements

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Breaking change

@cmonr
Copy link
Contributor

cmonr commented Sep 13, 2018

This is probably needed anyways, but looking at the issue, this doesn't appear to solve the problem.

/morph build

@mbed-ci
Copy link

mbed-ci commented Sep 13, 2018

Build : SUCCESS

Build number : 3068
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/8075/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Sep 13, 2018

@cmonr
Copy link
Contributor

cmonr commented Sep 13, 2018

Not sure this completely solves the problem according to the issue.

@OPpuolitaival
Copy link
Contributor Author

Mbed tools are checking that mbed-os project requirements.txt related tools are installed when run some mbed-command. When we have this in here the check can see it. The check does not care about libraries dependencies and therefore issue was possible. @theotherjimmy can you confirm the analyse?

@adbridge
Copy link
Contributor

/morph test

@cmonr cmonr added the risk: A label Sep 14, 2018
@mbed-ci
Copy link

mbed-ci commented Sep 14, 2018

@cmonr cmonr merged commit 6addb89 into ARMmbed:master Sep 14, 2018
@cmonr cmonr removed the risk: G label Sep 14, 2018
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.

4 participants