Skip to content

Add a user init commands flag#193

Merged
slp merged 1 commit intoAsahiLinux:mainfrom
valpackett:val/lownnoyknpkr
Sep 29, 2025
Merged

Add a user init commands flag#193
slp merged 1 commit intoAsahiLinux:mainfrom
valpackett:val/lownnoyknpkr

Conversation

@valpackett
Copy link
Copy Markdown
Contributor

Some custom init things (like spawning Wayland and D-Bus proxies) have to run after the user account has been set up and XDG_RUNTIME_DIR has been created and set. Let's simply add a big-X flag for executing things as the user?

Some custom init things (like spawning Wayland and D-Bus proxies)
have to run after the user account has been set up and XDG_RUNTIME_DIR
has been created and set. Let's simply add a big-X flag for executing
things as the user.

Signed-off-by: Val Packett <[email protected]>
Copy link
Copy Markdown
Collaborator

@slp slp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds reasonable to me, thanks @valpackett

@teohhanhui
Copy link
Copy Markdown
Collaborator

teohhanhui commented Sep 17, 2025

@slp If we're okay with doing this over a semver major version, should we swap -x and -X? It seems much more intuitive.

(We could of course just merge this but without the -X shorthand for now.)

@slp
Copy link
Copy Markdown
Collaborator

slp commented Sep 17, 2025

@slp If we're okay with doing this over a semver major version, should we swap -x and -X? It seems much more intuitive.

The problem we have is that, for crates, bumping major is reasonably safe since cargo avoids picking it up unless instructed to do it. But, for binaries, I suspect people that may have integrated muvm into their scripts is likely not checking the version.

On the other hand, we don't know how prevalent is this use case, so perhaps we may as well do it once this time and see how many people complain?

@slp slp merged commit 5b35e56 into AsahiLinux:main Sep 29, 2025
2 checks passed
@valpackett valpackett deleted the val/lownnoyknpkr branch September 30, 2025 04:43
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.

4 participants