Skip to content

Tilde expansion in REPL shell completion raises an error #11281

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
dhoegh opened this issue May 15, 2015 · 2 comments
Closed

Tilde expansion in REPL shell completion raises an error #11281

dhoegh opened this issue May 15, 2015 · 2 comments
Labels
REPL Julia's REPL (Read Eval Print Loop)

Comments

@dhoegh
Copy link
Contributor

dhoegh commented May 15, 2015

When using tilde expansion then tabbing just after the tilde causes an error to be raised.

julia> "~<tab>
julia> "~WARNING: Caught an exception in the keymap:
ERROR: BoundsError
ERROR: syntax: incomplete: invalid string syntax

Another issue is the regex on

julia/base/REPLCompletions.jl

Line 114 in 8d3a42b

path = homedir() * path[2:end]
is not windows compatible. So using back slashes on windows do not work

julia> "~\\<tab>

This was introduced in #7617
CC @garborg
@blakejohnson blakejohnson added the REPL Julia's REPL (Read Eval Print Loop) label May 15, 2015
blakejohnson added a commit that referenced this issue May 18, 2015
@blakejohnson
Fix issue #11281.
12e4464 
The problem is that `prefix` ought to be "" when the string is just the tilde.
@blakejohnson
Copy link
Contributor

I see what the problem is now. I'll have to think about a slight generalization to make it work on windows.

@blakejohnson blakejohnson mentioned this issue May 22, 2015
Merged
blakejohnson added a commit that referenced this issue May 23, 2015
@blakejohnson
Fix issue #11281.
39e83f7 
The problem is that `prefix` ought to be "" when the string is just the tilde.
blakejohnson added a commit that referenced this issue May 24, 2015
@blakejohnson
Merge pull request #11396 from JuliaLang/brj/fix-tilde-expansion-error
220801b 
Fix issue #11281.
@blakejohnson
Copy link
Contributor

Fixed in 39e83f7

mbauman pushed a commit to mbauman/julia that referenced this issue Jun 6, 2015
@blakejohnson @mbauman
Fix issue JuliaLang#11281.
f85da9c 
The problem is that `prefix` ought to be "" when the string is just the tilde.
tkelman pushed a commit to tkelman/julia that referenced this issue Jun 6, 2015
@blakejohnson @tkelman
Fix issue JuliaLang#11281.
d46b676 
The problem is that `prefix` ought to be "" when the string is just the tilde.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
REPL Julia's REPL (Read Eval Print Loop)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@blakejohnson @dhoegh