-
Notifications
You must be signed in to change notification settings - Fork 1.1k
tools/build.py: Add "path" entry to index.json. #910
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
tools/build.py: Add "path" entry to index.json. #910
Conversation
@mattytrentini hopefully this is what's needed for mim.
This points to the package's base directory of the within the micropython-lib directory structure. Signed-off-by: Damien George <damien@micropython.org>
9c1e3c3 to
910af18
Compare
HowManyOliversAreThere
commented
Aug 20, 2024
@dpgeorge This looks like it should do the trick, I'll try this out to generate the json and insert it into mim to sanity check the path but should be great.
For reference, mim currently uses https://wb5lsudgvpt4f4j4pxomg5z3ui0dcgjt.lambda-url.ap-southeast-2.on.aws/ for the current proof of concept, where we've injected the url value into the existing json object by trawling the repo. Will be much nicer to use the official json directly!
HowManyOliversAreThere
commented
Aug 20, 2024
Added support for this parameter into mim and generated the json from this repo to test against and found it to be working as expected (from the spot checks I did), including things like finding the aioble README that was missed with my naive implementation. Thanks @dpgeorge !
@HowManyOliversAreThere thanks for testing, great that it works! Now merged.
@dpgeorge when will this appear at https://micropython.org/pi/v2/index.json? I had thought it would update the build machine overnight but I'm seeing v1...
This points to the package's base directory of the within the micropython-lib directory structure.