Skip to content

fix: only import SSH_AUTH_SOCK when it is set - #233

Open
kajusnau wants to merge 1 commit into
pop-os:masterfrom
kajusnau:fix-ssh-auth-sock-import
Open

kajusnau wants to merge 1 commit into
pop-os:masterfrom
kajusnau:fix-ssh-auth-sock-import

Conversation

@kajusnau

@kajusnau kajusnau commented Oct 9, 2026

Copy link
Copy Markdown

Small change to only import SSH_AUTH_SOCK when it is set
Removes the somewhat confusing ...$SSH_AUTH_SOCK not set... print to tty1 during greeter handoff


  • I have not included any LLM (also known as AI) generated content in this PR, including code, comments, and descriptions.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.
  • I have read and followed all of the above requirements. For more information, see our contributor guide.

Signed-off-by: Kajus Naujokaitis <kajusn@gmail.com>
@mmstick
mmstick requested review from a team October 9, 2026 12:15
@kajusnau kajusnau mentioned this pull request Nov 4, 2026
11 of 19 tasks
@joshuamegnauth54

Copy link
Copy Markdown
Contributor

Closes: #242

I didn't notice your patch. I opened a similar fix in #244 but I think both of our patches serve different purposes so they're both viable. Your fix is very clean, so I prefer it as a fix to #242. 😁

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.

3 participants