diff options
author | Joey Hess <joeyh@joeyh.name> | 2015-03-26 11:44:20 -0400 |
---|---|---|
committer | Joey Hess <joeyh@joeyh.name> | 2015-03-26 11:44:20 -0400 |
commit | 4267f36f058eb88b3d18613acee34d8eb4bb4e4f (patch) | |
tree | 0e0c416cb0ad7456dbe5f1818bca8182e8460fad /doc/todo/inject_on_import | |
parent | 71d824d16cd928693d149950a8eabf2652243350 (diff) |
improve import duplicate docs
Diffstat (limited to 'doc/todo/inject_on_import')
-rw-r--r-- | doc/todo/inject_on_import/comment_2_205ecbc7401f99fc83719acbf5da174e._comment | 26 |
1 files changed, 26 insertions, 0 deletions
diff --git a/doc/todo/inject_on_import/comment_2_205ecbc7401f99fc83719acbf5da174e._comment b/doc/todo/inject_on_import/comment_2_205ecbc7401f99fc83719acbf5da174e._comment new file mode 100644 index 000000000..acd661feb --- /dev/null +++ b/doc/todo/inject_on_import/comment_2_205ecbc7401f99fc83719acbf5da174e._comment @@ -0,0 +1,26 @@ +[[!comment format=mdwn + username="joey" + subject="""comment 2""" + date="2015-03-26T15:28:45Z" + content=""" +Well, you've found an edge case here. + +It behaves as documented as long as the file being imported is located in some +repository know to git-annex. The file content does not have to be present in +the local repository for it to behave as documented. + +In your case, the file being imported has a symlink in the git repo, but +git-annex knows about 0 annexed copies of the file, so it's treated as +if it's a new file and not a duplicate. + +Since import is working at the key level, there's not a good way to look up +that there are some symlinks in the git repo even though the content is +gone. And even if there was, I think I'd be uncomfortable with it deleting +the file as "duplicate" when its content is not available in any known +repository. The only behavior improvement might be to import the content +but not make a redundant symlink in this case. + +I think it's best to change the documentation. I've added a new +paragraph that more exactly and clearly explains what duplicate files +are for the purposes of importing. +"""]] |