Skip to content

pin pixi version in PixiWrapperTask - #409

Open
holodorum wants to merge 3 commits into
mainfrom
pixi-pin
Open

pin pixi version in PixiWrapperTask#409
holodorum wants to merge 3 commits into
mainfrom
pixi-pin

Conversation

@holodorum

Copy link
Copy Markdown
Collaborator

The Pixi install shell script downloads the latest version of pixi. Setting the PIXI_VERSION env variable in our pixi wrapper scripts allows us to pin the version of Pixi.

Also remove two pixi wrapper files that must have been committed before we started gitignoring them.

The pixi install shell script downloads the latest version of pixi.
Setting the `PIXI_VERSION` env variable in the pixiw scripts allows us to
 pin the version of Pixi.

Also remove two pixiw files that must have been committed before we
started gitignoring them.
@dmarcotte

Copy link
Copy Markdown
Contributor

Thanks @holodorum! I have a couple of questions here that are easier realtime—let's have a quick look at this together before we merge.

Pinning PIXI_VERSION only affects a fresh install, and the wrapper was
only generated when `pixiw` was missing, so anyone with a pixiw already
on disk kept an unpinned wrapper and an off-pin Pixi forever. The
generated wrappers now compare `pixi --version` against the pin and
reinstall when it differs, and PixiExecTask regenerates the wrapper on
every run so wrapper changes propagate. Regenerating unconditionally
exposed a latent clash: generateWrapper registered a task under a fixed
name, which failed for the second Pixi task in a project, so the name is
now derived from the task. pixiw.bat stays untested since it needs
cmd.exe.
Ran `pixi lock` to re-solve the four remaining v6 lock files using their
locked content, bringing them in line with lib-rust (already v7) and the
pixi version developers now run. No dependency changes.
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