sudo usage clobbers autojump_py to root ownership

[wting]

sudo usage clobbers autojump_py to root ownership

[joelthelion]

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?

[luciferous]

I have the same problem. I'm using autojump 14 on OS X installed via brew.

To reproduce just cd as root into another directory:

$ ls -l ~/.autojump_py
-rw-------  1 neuman  staff  17025 Mar 28 11:29 /Users/neuman/.autojump_py
$ sudo -s
# cd /tmp
# ls -l /Users/neuman/.autojump_py
-rw-------  1 root  staff  17025 Mar 28 11:44 /Users/neuman/.autojump_py
# exit
$ j
Problem with autojump database, trying to recover from backup...
Traceback (most recent call last):
  File "/usr/local/bin/autojump", line 132, in <module>
    path_dict=open_dic(dic_file)
  File "/usr/local/bin/autojump", line 117, in open_dic
    shutil.copy(dic_file+".bak",dic_file)
  [...snip...]
IOError: [Errno 13] Permission denied: '/Users/neuman/.autojump_py'

[jamilbk]

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.

[joelthelion]

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?

[jamilbk]

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 sudo -s session.

Thanks!

[joelthelion]

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.