From 75672699e5b4ef25310c247977e32fa90a1c0c18 Mon Sep 17 00:00:00 2001 From: Jared Allard Date: Fri, 10 May 2024 16:49:00 -0700 Subject: [PATCH] app-admin/1password: remove keyringhelper logic 1Password no longer provides/needs a KeyringHelper, so we can remove this logic. Closes #3 --- app-admin/1password/1password-8.10.30.ebuild | 4 ---- 1 file changed, 4 deletions(-) diff --git a/app-admin/1password/1password-8.10.30.ebuild b/app-admin/1password/1password-8.10.30.ebuild index 3f0fefc..d9d1d4e 100644 --- a/app-admin/1password/1password-8.10.30.ebuild +++ b/app-admin/1password/1password-8.10.30.ebuild @@ -63,10 +63,6 @@ pkg_postinst() { # See https://github.com/electron/electron/issues/17972 chmod 4755 /opt/1Password/chrome-sandbox - # The binary requires setuid so it may interact with the Kernel keyring facilities - chmod u+s /opt/1Password/1Password-KeyringHelper - chmod g+s /opt/1Password/1Password-KeyringHelper - # This gives no extra permissions to the binary. It only hardens it against environmental tampering. chgrp onepassword /opt/1Password/1Password-BrowserSupport chmod g+s /opt/1Password/1Password-BrowserSupport