-
Notifications
You must be signed in to change notification settings - Fork 700
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
sudo usage clobbers autojump_py to root ownership #39
Comments
Hi and thanks for the report! Are you using the latest git version or a previous release? I've had the same problem in the past, but I thought it was fixed now. Also, do you have an idea of the command that caused the problem? What distro are you using? |
I have the same problem. I'm using autojump 14 on OS X installed via brew. To reproduce just
|
I can confirm this as a bug as well. Using autojump 14 on OSX, but appears to be present in the latest git as well. Same issue as luciferous. |
I've just pushed a tentative fix. I can't test it because sudo is properly better configured under Arch, so I can't reproduce the problem. However I believe this patch should fix the problem. Can you confirm? |
I can't do any in-depth testing right now, but so far the issue looks fixed! ~/.autojump_py didn't change ownership after a Thanks! |
Great! I'll close the issue for now; don't hesitate to re-open if needed. I'll probably wait a week or so to make sure everything is ok, then I'll make a new release. |
Should fix issue wting#39
I'm not entirely positive, but I just got and error from jumpstat:
IOError: [Errno 13] Permission denied: '{$HOME}.local/share/autojump/autojump_py'
Sure enough, autojump_py was owned by root. Seems like a cd or something I ran with sudo wrote to the file as root (?) and chowning it back to me fixed the problem
The text was updated successfully, but these errors were encountered: