Merge lp:~mandel/ubuntuone-client/ignore-ds-store into lp:ubuntuone-client

Proposed by Manuel de la Peña on 2012-08-30
Status: Merged
Approved by: dobey on 2012-08-30
Approved revision: 1305
Merged at revision: 1304
Proposed branch: lp:~mandel/ubuntuone-client/ignore-ds-store
Merge into: lp:ubuntuone-client
Diff against target: 11 lines (+1/-0)
1 file modified
data/syncdaemon.conf (+1/-0)
To merge this branch: bzr merge lp:~mandel/ubuntuone-client/ignore-ds-store
Reviewer Review Type Date Requested Status
dobey (community) Approve on 2012-08-30
Eric Casteleijn (community) 2012-08-30 Approve on 2012-08-30
Review via email: mp+122084@code.launchpad.net

Commit Message

- Add the pattern to ignore the .DS_Store files coming from darwin (LP: #1034138).

Description of the Change

- Add the pattern to ignore the .DS_Store files coming from darwin (LP: #1034138).

To post a comment you must log in.
Eric Casteleijn (thisfred) :
review: Approve
dobey (dobey) wrote :

Please fix the indentation to use only spaces instead of tabs and spaces, like the rest of the lines in the config file. This is why the diff appears off.

review: Needs Fixing
1305. By Manuel de la Peña on 2012-08-30

Do not mix tabs and spaces.

dobey (dobey) :
review: Approve
Vincent Ladeuil (vila) wrote :

I can't help but notice that .DS_Store contains data relevant to OSX users and as such may be interesting to sync between OSX machines.

Now, like you, I'm annoyed to see .DS_Store come up on my non-OSX machines...

But the right solution may not be to just ignore them no ?

Manuel de la Peña (mandel) wrote :

Vincent,

The DS_Store file is a hidden file that stores metadata. Those files are created by the finder and are simple custom attributes (icon positions, views settings, etc..) While it might be useful for the user to sync those settings that does not out weight how annoying it will be to have such files on Windows and Linux. Apple already had a bug filled prior 10.4 because users did not like the fact that the finder would create .ds_store files in remote drives, I think we can assume that the most common use case would be users being annoyed by these files.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/syncdaemon.conf'
2--- data/syncdaemon.conf 2012-07-17 10:51:43 +0000
3+++ data/syncdaemon.conf 2012-08-30 15:21:17 +0000
4@@ -86,6 +86,7 @@
5 \A\.~lock\..*#\Z
6 \A\.goutputstream-.*\Z
7 \A.*-Spotlight\Z
8+ \A\.DS_Store\Z
9
10 use_trash.default = True
11 use_trash.parser = bool

Subscribers

People subscribed via source and target branches