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
micropython: init at 1.12 #74964
micropython: init at 1.12 #74964
Conversation
f00dd70
to
1db61a3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed points
- package path fits guidelines
- package name fits guidelines
- package version fits guidelines
- package build on x86_64
- executables tested on x86_64
-
meta.description
is set and fits guidelines -
meta.license
fits upstream license -
meta.platforms
is set -
meta.maintainers
is set - build time only dependencies are declared in
nativeBuildInputs
- source is fetched using the appropriate function
- phases are respected
Possible improvements
Comments
Thanks for your contribution! ✨
@stigtsp Do you have any usecase for this? As it is packaged now it just produces Unix binary, which is not very useful. |
9e764e9
to
ae7c09a
Compare
Thanks for the review, python3 and pkgconfig have been added to nativeBuildInputs. |
I'm using the linux build to write networking code for ESP8266/ESP32 locally rather than directly on the microcontroller. Maybe this is useful for other nixpkgs users as well.
It's quite nice if you write micropython code, and want to run it on Linux. The binary contains a standard library with ussl, usocket, upip, etc. You can for example use it to install micropython-libs in $HOME/.micropython by running |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see my previous comment for details
Are there any issues preventing this PR from being merged? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs a rebase. Would be good to check whether makeFlags
can be used instead. Otherwise, looks good to me.
ae7c09a
to
9e633f5
Compare
Updated PR with new upstream 1.11 -> 1.12 |
Motivation for this change
This PR adds the unix port of MicroPython. A lean implementation of Python 3 for microcontrollers.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)