Skip to content

chore(zshrc): move linux-only vars to dotfiles linux.zshrc - #84

Merged
mrsauravsahu merged 1 commit into
mainfrom
worktree-lovely-yawning-planet
Aug 9, 2026
Merged

mrsauravsahu merged 1 commit into
mainfrom
worktree-lovely-yawning-planet

Conversation

@mrsauravsahu

Copy link
Copy Markdown
Owner

Summary

  • GDK_SCALE=2 and the linuxbrew PATH_PREFIX entry in profiles/mrsauravsahu/.zshrc were Linux-only concerns living in the cross-platform base profile.
  • The linuxbrew path was already duplicated in ~/.mrsauravsahu/dotfiles/linux.zshrc (that file is already sourced only when currentOs=linux, via the existing . ${HOME}/.mrsauravsahu/dotfiles/${currentOs}.zshrc line), so it just needed deleting here.
  • GDK_SCALE was moved into ~/.mrsauravsahu/dotfiles/linux.zshrc (a separate, personal dotfiles repo, not part of this PR's diff).

Note on the stash

The stashed WIP (stash@{0}) adds a WSL-specific open() function to this same file. linux.zshrc already defines its own open() for native Ubuntu/GNOME (env open ...). Per your instruction, I left that conflict alone rather than merging — the stash is still sitting untouched in the main working copy for you to resolve (e.g. gate on $WSL_DISTRO_NAME).

Test plan

  • zsh -n profiles/mrsauravsahu/.zshrc (syntax check)
  • Source the profile on both a WSL box and the native Ubuntu box to confirm GDK_SCALE and linuxbrew's PATH entry are still present via linux.zshrc

GDK_SCALE and the linuxbrew PATH entry were being set unconditionally
in the cross-platform profile. Both are Linux-only concerns; the
linuxbrew path was already duplicated in ~/.mrsauravsahu/dotfiles/linux.zshrc
(sourced only when currentOs=linux), so it just needed removing here.
GDK_SCALE was added there.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@mrsauravsahu
mrsauravsahu marked this pull request as ready for review August 9, 2026 07:32
@mrsauravsahu
mrsauravsahu merged commit 6f7e076 into main Aug 9, 2026
1 check passed
@mrsauravsahu
mrsauravsahu deleted the worktree-lovely-yawning-planet branch August 9, 2026 07:33
mrsauravsahu added a commit to mrsauravsahu/dotfiles that referenced this pull request Aug 9, 2026
Companion to mrsauravsahu/cli-config#84, which removed this from the
cross-platform base .zshrc since it's a Linux/GTK-only concern.

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
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.

1 participant