-
Notifications
You must be signed in to change notification settings - Fork 176
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
MBED python requirements issues ? #782
Comments
Internal Jira reference: https://jira.arm.com/browse/MBOCUSTRIA-139 |
Fix is available on mbed-cli master. Could you please try it out and let me know if that fixes the problem? |
This should be addressed in the new release (1.8.3). Please close the issue if it's fixed. |
Hi 1.8.3 release is not ok TEST$ pip install colorama==0.3.9 OK $ pip install colorama -U ==> It crashes….
|
Hi |
Hi @jeromecoutant, sorry for the slow reply. I have a change up to bump the dependency version of colorama to include 0.4.x: ARMmbed/greentea#291 |
Hi |
I'm sure you don't 😄 but that was the version error you posted above.
This isn't an issue with Mbed CLI in particular, this is just how pip handles dependencies. If another package (or a user) comes in and upgrades/downgrades an installed dependency, then the previously installed packages may break due to the mismatch in required version numbers. The usual way to avoid this is to use Python Virtual Environments. That way the package has full control over dependency versions. |
@jeromecoutant Have you tried CLI v1.9.1 to see if #833 improves things for you? |
Hi
Since ARMmbed/mbed-os#8542,
python modules version check is more strict.
But I think this doesn't really work ?
In my environment, urllib3 is installed.
But if I print pkg here:
mbed-cli/mbed/mbed.py
Line 1626 in e993263
=> I always got urllib3[secure]
"pip install -r requirements.txt" command is executed only if "missing" is not empty:
mbed-cli/mbed/mbed.py
Line 1628 in e993263
=> this means that if each package is present, module version is not checked
and ARMmbed/mbed-os#8542 is not useful?
The text was updated successfully, but these errors were encountered: