Skip to content
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

Requirements: mbed-cloud-sdk having exact version match #8194

Closed
0xc0170 opened this issue Sep 20, 2018 · 7 comments
Closed

Requirements: mbed-cloud-sdk having exact version match #8194

0xc0170 opened this issue Sep 20, 2018 · 7 comments

Comments

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 20, 2018

Description

mbed-cloud-sdk==2.0.0 - I would like to understand this requirement. We have a new PR that is updating this to 2.0.1. Are we expecting to have exact match and update for specific patch releases that we choose?

Why not to loose this dependency to accept at least patch releases ? How often is patch release expected vs req update in this repository ?

@ARMmbed/mbed-os-tools @MarceloSalazar

Issue request type

[X] Question
[ ] Enhancement
[ ] Bug

@cmonr
Copy link
Contributor

cmonr commented Sep 20, 2018

Related to: #8160

My opinions: #8160 (comment)

@0xc0170
Copy link
Contributor Author

0xc0170 commented Sep 20, 2018

@arekzaluski Please review

@ARMmbed ARMmbed deleted a comment from ciarmcom Oct 1, 2018
@adbridge
Copy link
Contributor

adbridge commented Oct 4, 2018

Internal Jira reference: https://jira.arm.com/browse/IOTTOO-3457

@0xc0170
Copy link
Contributor Author

0xc0170 commented Nov 27, 2018

Actually , @theotherjimmy added ==2 version . Why req was added so strictly ?

@theotherjimmy
Copy link
Contributor

@0xc0170 If anything, I used to be using internal APIs. https://github.com/ARMmbed/mbed-os/blob/master/tools/device_management.py#L94-L134 is the code affected and no longer uses internal APIs. We should be good to relax that requirement now.

@0xc0170
Copy link
Contributor Author

0xc0170 commented Nov 28, 2018

Good to hear ! Please send a fix and we can close this one.

@0xc0170
Copy link
Contributor Author

0xc0170 commented Aug 26, 2019

Not relevant anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants