Merge lp:~mandel/ubuntuone-client/ignore-ds-store into lp:ubuntuone-client
| 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 | ||||
| Related bugs: |
|
| 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:
|
|||
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).
- 1305. By Manuel de la Peña on 2012-08-30
-
Do not mix tabs and spaces.
| 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.


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.