Skip to content
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.

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

gf can't open directories that start with ~ #10948

Open
MyApaulogies opened this issue Jun 14, 2024 · 3 comments · May be fixed by #10964
Open

gf can't open directories that start with ~ #10948

MyApaulogies opened this issue Jun 14, 2024 · 3 comments · May be fixed by #10964
Labels
A-command Area: Commands C-bug Category: This is a bug

Comments

@MyApaulogies
Copy link

Summary

I'm able to gf /home/user/.config/helix and get a file picker prompt, as well as ~/.config/helix/config.toml, but when I try it on ~/.config/helix/, I get an error: Open file failed: Is a directory (os error 21). This bug seems to apply to all directories starting with a ~.

Reproduction Steps

I tried this:

  1. hx
  2. i
  3. ~/.config/helix
  4. gf

I expected the directory to open in a file picker.

Instead, I get the error Open file failed: Is a directory (os error 21).

Helix log

No response

Platform

Linux

Terminal Emulator

Alacritty

Installation Method

dnf

Helix Version

helix 24.3 (2cadec0)

@MyApaulogies MyApaulogies added the C-bug Category: This is a bug label Jun 14, 2024
@krish-r
Copy link
Contributor

krish-r commented Jun 14, 2024

I believe this was fixed in #9065

@MyApaulogies
Copy link
Author

My bad, you're right!

@the-mikedavis
Copy link
Member

It looks like this works when gf is used with a single-width selection but if you select ~/.config/helix and then use gf there is the directory error

@the-mikedavis the-mikedavis reopened this Jun 14, 2024
@the-mikedavis the-mikedavis linked a pull request Jun 14, 2024 that will close this issue
@kirawi kirawi added the A-command Area: Commands label Jun 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-command Area: Commands C-bug Category: This is a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants