Skip to content

misc: Add package file for mip installation. #53

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
Apr 11, 2023

Conversation

sebromero
Copy link
Contributor

This PR adds compatibility with the mip command (see here).
It's not clear to me if the cbor and senml packages are going to be hosted via the micropython-libs index. If so, the version specified in the dependencies may need to be adjusted accordingly.

@iabdalkader
Copy link
Collaborator

If this for installing the package on MicroPython, the dependencies should just be senml >= 0.1.0

@sebromero sebromero force-pushed the mip-compatibility branch 2 times, most recently from 764d4fd to 6109827 Compare April 11, 2023 10:58
@sebromero
Copy link
Contributor Author

@iabdalkader I changed it. As far as I can see mip doesn't support semantic versioning so you have to pin an exact version. I didn't find any documentation on this but it seems to translate the dependency directly into a URL.

@sebromero
Copy link
Contributor Author

I just ran a test with the new release of senml and it worked fine:

$ mpremote connect /dev/cu.usbmodem3871345733302 mip install github:sebromero/arduino-iot-cloud-py@mip-compatibility
Install github:sebromero/arduino-iot-cloud-py
Installing github:sebromero/arduino-iot-cloud-py/package.json to /flash/lib
Installing: /flash/lib/arduino_iot_cloud/__init__.py
Installing: /flash/lib/arduino_iot_cloud/ucloud.py
Installing: /flash/lib/arduino_iot_cloud/umqtt.py
Installing: /flash/lib/arduino_iot_cloud/ussl.py
Installing senml (0.1.0) from https://micropython.org/pi/v2 to /flash/lib
Installing: /flash/lib/cbor2/__init__.mpy
Installing: /flash/lib/cbor2/encoder.mpy
Installing: /flash/lib/cbor2/decoder.mpy
Installing: /flash/lib/senml/senml_pack.mpy 
Installing: /flash/lib/senml/__init__.mpy   
Installing: /flash/lib/senml/senml_record.mpy
Installing: /flash/lib/senml/senml_base.mpy
Installing: /flash/lib/senml/senml_unit.mpy
Done

@sebromero sebromero marked this pull request as ready for review April 11, 2023 11:00
@iabdalkader iabdalkader merged commit b46095d into arduino:main Apr 11, 2023
@sebromero sebromero deleted the mip-compatibility branch April 11, 2023 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants