Skip to content

Set correct permissions for linux#9

Closed
resterle wants to merge 1 commit into
NeoHsu:masterfrom
resterle:linux_permissions
Closed

Set correct permissions for linux#9
resterle wants to merge 1 commit into
NeoHsu:masterfrom
resterle:linux_permissions

Conversation

@resterle
Copy link
Copy Markdown

As described under "Linux->Manual" here: https://developer.1password.com/docs/cli/get-started/

@NeoHsu
Copy link
Copy Markdown
Owner

NeoHsu commented May 28, 2026

Thanks for the contribution, and sorry for the long delay in handling this.

The Linux app integration permissions issue has now been addressed on master in a slightly different way. The installer now applies the onepassword-cli group and setgid bit on a best-effort basis when the group exists and the current user has permission, so asdf install will not fail for users who cannot run chgrp. The README also documents the manual sudo chgrp / sudo chmod g+s steps for users who need to enable 1Password app integration.

Because this PR's core change is now covered, and the current implementation avoids failing installs when permissions are unavailable, I'm going to close this PR without merging. Thanks again for pointing this out.

@NeoHsu NeoHsu closed this May 28, 2026
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