Ticket #1656 (closed defect: fixed)
Incorrect processing of cases the lack of rights (~/.mc/*) to save preferences
Reported by: | alex_zol | Owned by: | slavazanko |
---|---|---|---|
Priority: | major | Milestone: | 4.7.0-pre4 |
Component: | mc-config-ini | Version: | 4.7.0-pre3 |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Branch state: | Votes for changeset: | committed-master |
Description
In the case where insufficient rights to the contents of the directory ~/.mc/* to save preferences I get "settings saved in ~ /.mc/ini", but really nothing is saved.
It would be correct to issue a message like "Not enough permissions to save the ~/.mc/ini"
Also, when autosave should be issued a message about the lack of rights.
Accidentally situation is the lack of rights can occur when you first start mc under sudo.
The files in ~/.mc/* created with owner root (but not in the home directory of root, and in the home directory of a user caused sudo).
For example:
sudo apt-get install mc
sudo -s
mc
save settings and exit from mc
alex@alex-laptop:~/.mc$ ls -l ~/.mc
drwx------ 2 root root 4096 2009-09-17 22:38 cedit
-rw-r--r-- 1 root root 69 2009-09-20 09:40 filepos
-rw-r--r-- 1 root root 116 2009-10-02 21:26 history
-rw-r--r-- 1 root root 2602 2009-10-02 21:26 ini
-rw-r--r-- 1 root root 714 2009-10-02 21:26 panels.ini
-rw-r--r-- 1 root root 35 2009-10-02 21:26 Tree
Change History
comment:3 Changed 15 years ago by zaytsev
Also have same problem here :-( have to check every time manually...
comment:6 Changed 15 years ago by slavazanko
- Status changed from new to accepted
- Owner set to slavazanko
comment:7 Changed 15 years ago by slavazanko
- Component changed from mc-core to mc-config-ini
- severity changed from no branch to on review
Created branch 1656_lack_rights.
changesets:
- 6ae97ddfe9e8f650c387e7fff045f36535881bbd: Incorrect processing of cases the lack of rights to save preferences.
- 20ce3df02d7ec0a9b06fb647ef1c1b38be7be904: Added debug options for --enable-maintainer mode
- 96287e5574d11ca4c5bd86b8ae08845f39b53266: Code cleanup for compile with new CFLAGS value.
- bb24773bab6e53cf32ba2d38085ac7e761490bb9: Fix segfault in charset confirmation dialog at start of mc.
Review, please.
comment:8 Changed 15 years ago by slavazanko
- 9a94dd54f436e5ae23fdef26a19be3d48bff4fc2: REBASE: rename boolean variable is_warning -> show_warning
- 4736754a2a0d7245403f827ea1d707a0d4997b02: REBASE: cleanup: change type name
- 93f48f91e61ecc017b74b93a98e757f0e7be200f: debug options: added -Wall
- 00e9ea2c10c2951979e9ea502424dfa647f2abe6: REBASE: Cleanup after adding -Wall compiler option
comment:9 Changed 15 years ago by slavazanko
Branch rebased.
- fd2978bf67de8a163126efdafa18ee455aba6337: Incorrect processing of cases the lack of rights to save preferences.
- 54239e9bb9d81702c8e1ab4d839ecea2b0ce3808: Added debug options for --enable-maintainer mode
- 2b889a03aa1e7e8dee7d2d429ee669b210f23fc5: Code cleanup for compile with new CFLAGS value.
- 10b118ea1b251f0f14fe68813e36d5d018831b1c: Fix segfault in charset confirmation dialog at start of mc.
Review, please, again
comment:10 Changed 15 years ago by slavazanko
Branch again rebased.
- 04588189a8ac9c30e1d837fde0542970a65c42cc: Incorrect processing of cases the lack of rights to save preferences.
- 7f2d666634e02bbc0d07a66d130c120a9311fd1c: Added debug options for --enable-maintainer mode
- d7be09330634b378337db09de120b8426db484e5: Code cleanup for compile with new CFLAGS value.
- 1075de90f2b85490f16ee010fcd475c4444a3b3a: Fix segfault in charset confirmation dialog at start of mc.
comment:12 Changed 15 years ago by angel_il
- Votes for changeset changed from andrew_b to andrew_b angel_il
- severity changed from on review to approved
comment:13 Changed 15 years ago by slavazanko
- Status changed from accepted to testing
- Votes for changeset changed from andrew_b angel_il to commited-master
- Resolution set to fixed
- severity changed from approved to merged