diff options
author | Joey Hess <joeyh@joeyh.name> | 2016-10-05 13:30:46 -0400 |
---|---|---|
committer | Joey Hess <joeyh@joeyh.name> | 2016-10-05 13:30:46 -0400 |
commit | 6666de1f505b1929f80992fe953f739c6d02d9c1 (patch) | |
tree | 4dc9b99124048076ef96ad879fbc446ca708b6e0 | |
parent | 4826c53703f82a3d3fb049e29d22eea4705c67f5 (diff) |
comment
-rw-r--r-- | doc/bugs/committing_an_edited_file_fails_with___34__hGetBuf__58___Invalid_argument__34__/comment_1_e2bc83fc04641c3547aeb1830da0d947._comment | 27 |
1 files changed, 27 insertions, 0 deletions
diff --git a/doc/bugs/committing_an_edited_file_fails_with___34__hGetBuf__58___Invalid_argument__34__/comment_1_e2bc83fc04641c3547aeb1830da0d947._comment b/doc/bugs/committing_an_edited_file_fails_with___34__hGetBuf__58___Invalid_argument__34__/comment_1_e2bc83fc04641c3547aeb1830da0d947._comment new file mode 100644 index 000000000..4046ad7bb --- /dev/null +++ b/doc/bugs/committing_an_edited_file_fails_with___34__hGetBuf__58___Invalid_argument__34__/comment_1_e2bc83fc04641c3547aeb1830da0d947._comment @@ -0,0 +1,27 @@ +[[!comment format=mdwn + username="joey" + subject="""comment 1""" + date="2016-10-05T17:02:12Z" + content=""" +By using git commit -a to commit changes to a large annexed file, you +are causing git to first add the file to the git repository, and then +git-annex has to go fix up and convert it back to an annexed file. + +So, you probably don't want to be doing that, irrespective of this bug. +Storing the file content in the git repository will waste disk space until +git gc gets around to cleaning it up. + +Instead, use `git annex add` on the file after editing it, and then commit +the result. As well as not cluttering up git with large unused objects, +that will be generally faster, and will probably avoid this bug. + +---- + +I've reproduced on linux a behavior that probably has the same root cause. +It looks like git-annex pre-commit is reading the whole content of the +large file from git cat-file, and buffering it in memory. Of course +this uses a lot of memory and will fail for some size files, and it +should definitely not be doing this. + +Suspect this is a reversion caused by the changes to support v6 mode. +"""]] |