[ci] Sync zutils v0.11.0#249
Merged
Merged
Conversation
ppenna
force-pushed
the
automation/update-zutils-version
branch
from
cb4afb4 to
9390ee4
Compare
ada-x64
force-pushed
the
automation/update-zutils-version
branch
from
9390ee4 to
dc5512c
Compare
There was a problem hiding this comment.
Pull request overview
This PR updates the repository’s Nanvix/zutils integration to align with zutils v0.11.0, refreshing the bootstrapping scripts and adjusting the Nanvix build/release staging flow so zutils can collect artifacts from standard output directories.
Changes:
- Bumps the pinned zutils version to
v0.11.0and syncs wrapper templates (z.sh,z.ps1) accordingly. - Updates the Nanvix Makefile flow to stage build outputs via a new
installtarget (and removes the old packaging targets). - Updates
.nanvix/z.pyto pass zutils “paths” outputs (e.g.,OUT_DIR,LIB_OUT,INCLUDE_OUT,BIN_OUT) intoMakefile.nanvixand to copy back staged artifacts in Windows Docker mode.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
z.sh |
Syncs the bootstrapper template and simplifies pre-forwarding argument handling. |
z.ps1 |
Syncs the Windows bootstrapper template and simplifies pre-forwarding argument handling. |
Makefile.nanvix |
Adds install staging and updates required variables/output cleanup accordingly. |
.zutils-version |
Pins zutils to v0.11.0. |
.nanvix/z.py |
Switches to zutils paths helpers and passes staging/output variables into the build Makefile. |
.nanvix/.gitignore |
Updates ignored paths formatting for Nanvix local artifacts. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
532
to
538
| run( | ||
| "make", | ||
| "-f", | ||
| "Makefile.nanvix", | ||
| "clean", | ||
| cwd=self.repo_root, | ||
| cwd=repo_root(), | ||
| ) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated sync with
v0.11.0:.zutils-versiontov0.11.0(source of truth on workflows@v2.1.0+).z,z.sh,z.ps1) from release assets.nanvix-versionto0.16.17in.nanvix/nanvix.toml.Generated by the Update Zutils workflow.