Skip to content

Fix :cd ~ by handling bare tilde in expandHome (fixes #2072)#3133

Open
MavenRain wants to merge 6 commits intoGaloisInc:masterfrom
MavenRain:fix/cd-standalone-tilde
Open

Fix :cd ~ by handling bare tilde in expandHome (fixes #2072)#3133
MavenRain wants to merge 6 commits intoGaloisInc:masterfrom
MavenRain:fix/cd-standalone-tilde

Conversation

@MavenRain
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@sauclovian-g sauclovian-g left a comment

Choose a reason for hiding this comment

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

Also, as is this only fixes the first half of the issue; it doesn' expand paths of the form ~username or ~username/dir. That's ok as a starting point of course.

Comment thread saw-script/src/SAWScript/REPL/Command.hs Outdated
…ilde

# Conflicts:
#	saw-script/src/SAWScript/Interpreter.hs
  The `fails` builtin emits a trailing blank line after each stack
  trace. The reference output included blank-line separators between
  the first three error blocks but was missing the trailing blank
  line after the fourth, causing test_set_base to fail with a
  1-line diff on both ubuntu-24.04 and macos-15 integration-tests.
@sauclovian-g
Copy link
Copy Markdown
Contributor

Properly speaking test_set_base should have gone in the set_base PR... but we can overlook that I think :-)

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