--enable-flakes doesn't seem to work #17

Open
opened 2024-07-01 16:56:42 +00:00 by iliana · 2 comments

When running:

$ curl -sSf -L https://install.lix.systems/lix | sh -s -- install --enable-flakes

the installer still prompts me explicitly whether to enable flakes or not. When running:

curl -sSf -L https://install.lix.systems/lix | sh -s -- install --enable-flakes --no-confirm

no prompt is displayed, but flakes are not enabled.


$ curl -sSf -L https://install.lix.systems/lix | sh -s -- --version
info: downloading installer (https://install.lix.systems/lix/lix-installer-x86_64-linux)
lix-installer 0.17.1
When running: ``` $ curl -sSf -L https://install.lix.systems/lix | sh -s -- install --enable-flakes ``` the installer still prompts me explicitly whether to enable flakes or not. When running: ``` curl -sSf -L https://install.lix.systems/lix | sh -s -- install --enable-flakes --no-confirm ``` no prompt is displayed, but flakes are not enabled. --- ``` $ curl -sSf -L https://install.lix.systems/lix | sh -s -- --version info: downloading installer (https://install.lix.systems/lix/lix-installer-x86_64-linux) lix-installer 0.17.1 ```
ktemkin self-assigned this 2025-09-21 16:51:25 +00:00

It actually appears to do the inverse of --enable-flakes when also set with --no-confirm, enabling flakes when unset and disabling flakes when set. Just ran into this when setting up a CI.

It actually appears to do the inverse of `--enable-flakes` when also set with `--no-confirm`, enabling flakes when unset and disabling flakes when set. Just ran into this when setting up a CI.
Member

Yeah, it's ArgAction::SetFalse, for some unholy reason.

https://git.lix.systems/lix-project/lix-installer/src/branch/main/src/settings.rs#L168-L179

In general, CLI code is kinda cursed, especially when interaction is layered on top of it. I'll see if the easy fix of ArgAction::SetTrue makes sense here. Thanks for the report!

Yeah, it's `ArgAction::SetFalse`, for some unholy reason. https://git.lix.systems/lix-project/lix-installer/src/branch/main/src/settings.rs#L168-L179 In general, CLI code is kinda cursed, especially when interaction is layered on top of it. I'll see if the easy fix of `ArgAction::SetTrue` makes sense here. Thanks for the report!
Sign in to join this conversation.
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lix-project/lix-installer#17
No description provided.