Merge lp:~mterry/deja-dup/support-user-var into lp:deja-dup/26

Proposed by Michael Terry
Status: Merged
Merged at revision: 1424
Proposed branch: lp:~mterry/deja-dup/support-user-var
Merge into: lp:deja-dup/26
Diff against target: 36 lines (+11/-4)
2 files modified
common/DirHandling.vala (+9/-4)
tests/unit/unit-tests.vala (+2/-0)
To merge this branch: bzr merge lp:~mterry/deja-dup/support-user-var
Reviewer Review Type Date Requested Status
Robert Bruce Park (community) Approve
Review via email: mp+137893@code.launchpad.net

Description of the change

Replace "$USER" in include/exclude paths with username.

To post a comment you must log in.
Revision history for this message
Robert Bruce Park (robru) wrote :

Looks good, Mike! Nice and simple ;-)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'common/DirHandling.vala'
2--- common/DirHandling.vala 2012-04-13 20:46:19 +0000
3+++ common/DirHandling.vala 2012-12-04 16:10:32 +0000
4@@ -49,10 +49,15 @@
5 s = get_trash_path();
6 else if (s == "$VIDEOS")
7 s = Environment.get_user_special_dir(UserDirectory.VIDEOS);
8- else if (Uri.parse_scheme(s) == null && !Path.is_absolute(s))
9- s = Path.build_filename(Environment.get_home_dir(), s);
10- else
11- return File.parse_name(s);
12+ else {
13+ // Some variables can be placed inside a larger path, so replace those
14+ s = s.replace("$USER", Environment.get_user_name());
15+
16+ if (Uri.parse_scheme(s) == null && !Path.is_absolute(s))
17+ s = Path.build_filename(Environment.get_home_dir(), s);
18+ else
19+ return File.parse_name(s);
20+ }
21
22 if (s != null)
23 return File.new_for_path(s);
24
25=== modified file 'tests/unit/unit-tests.vala'
26--- tests/unit/unit-tests.vala 2012-04-30 00:33:47 +0000
27+++ tests/unit/unit-tests.vala 2012-12-04 16:10:32 +0000
28@@ -37,6 +37,8 @@
29 parse_one_dir("$PICTURES", Environment.get_user_special_dir(UserDirectory.PICTURES));
30 parse_one_dir("$PUBLIC_SHARE", Environment.get_user_special_dir(UserDirectory.PUBLIC_SHARE));
31 parse_one_dir("$TEMPLATES", Environment.get_user_special_dir(UserDirectory.TEMPLATES));
32+ parse_one_dir("/backup/$USER", Path.build_filename("/backup", Environment.get_user_name()));
33+ parse_one_dir("backup/$USER", Path.build_filename(Environment.get_home_dir(), "backup", Environment.get_user_name()));
34 parse_one_dir("$VIDEOS", Environment.get_user_special_dir(UserDirectory.VIDEOS));
35 parse_one_dir("VIDEOS", Path.build_filename(Environment.get_home_dir(), "VIDEOS"));
36 parse_one_dir("/VIDEOS", "/VIDEOS");

Subscribers

People subscribed via source and target branches