The Wayback Machine - https://web.archive.org/web/20210111224319/https://github.com/Canop/broot/issues/200
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

hard to tell what's wrong in case of root installed zsh #200

Open
Canop opened this issue Feb 14, 2020 · 5 comments
Open

hard to tell what's wrong in case of root installed zsh #200

Canop opened this issue Feb 14, 2020 · 5 comments
Labels

Comments

@Canop
Copy link
Owner

@Canop Canop commented Feb 14, 2020

This probably applies to other cases.

What was observed for a user having zsh non used and installed as root was this:

Broot should be launched using a shell function (see https://github.com/Canop/broot for explanations).
The function is either missing, old or badly installed.
Can I install it now? [Y n]

Removing /home/solus/.local/share/broot/launcher/bash/1.
Writing br shell function in /home/solus/.local/share/broot/launcher/bash/1.
Removing /home/solus/.config/broot/launcher/bash/br.
Creating link from /home/solus/.config/broot/launcher/bash/br to /home/solus/.local/share/broot/launcher/bash/1.
/home/solus/.bashrc already patched, no change made.
IO Error : "Permission denied (os error 13)"

and the log contained

07:27:23 [ INFO] Starting Broot v0.13.1 with log level DEBUG
07:27:23 [DEBUG] (1) broot::shell_install: No 'installed' : we ask if we can install
07:27:27 [DEBUG] (1) broot::shell_install: proceed: true
07:27:27 [DEBUG] (1) broot::shell_install: Starting install
07:27:27 [ INFO] Writing br shell function in "/home/solus/.local/share/broot/launcher/bash/1"
07:27:27 [ INFO] Creating link from "/home/solus/.config/broot/launcher/bash/br" to "/home/solus/.local/share/broot/launcher/bash/1"
07:27:27 [DEBUG] (1) broot::shell_install::bash: considering path: "/home/solus/.bashrc"
07:27:27 [DEBUG] (1) broot::shell_install::bash: considering path: "/home/solus/.bash_profile"
07:27:27 [DEBUG] (1) broot::shell_install::bash: considering path: "/home/solus/.zshrc"
07:27:27 [DEBUG] (1) broot::shell_install::bash: considering path: "/home/solus/$ZDOTDIR/.zshrc/.zshrc"
07:27:27 [ WARN] Error: IO Error : "Permission denied (os error 13)"
07:27:27 [ INFO] bye

The problem was the non accessible .zshrc.

This should be easier to diagnostic and should not block the whole installation.

@Canop Canop added the bug label Feb 14, 2020
@Canop Canop self-assigned this Feb 14, 2020
@HagaiHargil
Copy link

@HagaiHargil HagaiHargil commented May 13, 2020

Did you manage to solve it?

@Canop
Copy link
Owner Author

@Canop Canop commented May 13, 2020

@HagaiHargil No, I didn't tackle it yet. I'm on a big new change introducing multi-panels and completion (in the "panels" branch) and everything else is frozen until I've finished it.

I've noted the interest in the issue, which matters a lot when I prioritize tasks.

@HagaiHargil
Copy link

@HagaiHargil HagaiHargil commented May 13, 2020

@Canop I completely understand, thanks for taking the time to do such a large and important refactor!

I'm sorry if my message was hinting otherwise, I was simply trying to understand whether there's a "manual" fix I can try, since your initial report suggested that you are aware of a way to overcome it.

@Canop
Copy link
Owner Author

@Canop Canop commented May 13, 2020

There's absolutely no problem in asking, no worry.

I don't think there's a hard problem here, other than finding the time to do a not terribly exciting change, and a rather painful one when it comes to thoroughly testing all paths.

@Canop Canop removed their assignment Jul 1, 2020
@HagaiHargil
Copy link

@HagaiHargil HagaiHargil commented Aug 27, 2020

I just wanted to document that if I simply run sudo /path/to/broot/executable --install then this issue is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.