-
Notifications
You must be signed in to change notification settings - Fork 512
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
Inaccurate version info in Pimoroni specific micropython images #752
Labels
micropython
This issue or request relates to micropython (either code or bindings)
Comments
Gadgetoid
added a commit
that referenced
this issue
Apr 26, 2023
Gadgetoid
added a commit
that referenced
this issue
Apr 26, 2023
Gadgetoid
added a commit
that referenced
this issue
Apr 26, 2023
Well this is going well for me 😆
Hmm:
Edit: Okay...
I just need to fix that |
Gadgetoid
added a commit
that referenced
this issue
Apr 26, 2023
Gadgetoid
added a commit
that referenced
this issue
Apr 26, 2023
This is now rolled out and seems to work. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When I run thonny with a Pimoroni micropython uf2 file I see:
I wrote a little bash script named mp_uf2_version.sh to extract version info directly from the uf2 file.
When I run this against the Pimoroni uf2 files in my Downloads directory I see:
The word dirty indicates an uncontrolled version.
The git commit hash is from the micropython repo.
Not from any repo owned by Pimoroni.
But, for the 'vanilla' uf2 files for pico w board I see:
The word dirty is missing and it contains the git tag and commit hash.
It would be good if your uf2 files contained the git tags and the git hash.
That way users and your folks could easily tell which version is running.
I took a look at makeversionhdr.py.
MICROPY_GIT_TAG and MICROPY_GIT_HASH can be overridden using environment variables.
The important information is in MICROPY_GIT_TAG.
In the current builds the value of MICROPY_GIT_HASH is not used.
You can set environment variable MICROPY_GIT_TAG with version info from the badger2040 or pimoroni-pico repo as appropriate.
The text was updated successfully, but these errors were encountered: