summaryrefslogtreecommitdiff
path: root/doc/bugs/git_annex_sync_can_commit_files_to_.git__47__objects_instead_of_.git__47__annex.mdwn
blob: 76260cb5cfc4a61f47b9f84279d9611806671606 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
### Please describe the problem.

If you `git annex edit FILE`, an already-committed file, then do `git annex sync`, FILE will be added to .git/objects, but can be cleaned up with `git prune`. An annoyance, but not a huge problem.

> This is why you're recommended to use `git add` after you're done
> changing unlocked files. This avoids the git commit staging the file
> in the index, even though the pre-commit hook fixes that up. It does
> indeed leave the file contents in .git/objects, although with no refs
> pointing to it, `git prune` will delete it sooner or later. --[[Joey]]

If, on the other hand, you do git annex add, then edit, then sync, it will actually be committed to the git repository. Fixing that is a lot less trivial than `git prune`.

> This is indeed a problem..

### What steps will reproduce the problem?

    anthony@Watt:/tmp/test$ du -sh .git/objects/
    24K     .git/objects/
    anthony@Watt:/tmp/test$ dd if=/dev/urandom bs=$[1024*1024] count=100 of=100mb-3
    100+0 records in
    100+0 records out
    104857600 bytes (105 MB) copied, 6.08718 s, 17.2 MB/s
    anthony@Watt:/tmp/test$ git annex add 100mb-3
    add 100mb-3 ok
    (Recording state in git...)
    anthony@Watt:/tmp/test$ git annex edit 100mb-3
    unlock 100mb-3 (copying...) ok
    anthony@Watt:/tmp/test$ git annex sync
    commit  ok
    anthony@Watt:/tmp/test [$?=130]$ git prune
    anthony@Watt:/tmp/test$ du -sh .git/objects/
    101M    .git/objects/
    anthony@Watt:/tmp/test$ ls -l 100mb-3 
    -rw-r--r-- 1 anthony anthony 104857600 Jan  1 13:41 100mb-3


### What version of git-annex are you using? On what operating system?

5.20141125 on Debian testing/unstable

> Analysis: This only happens when a file is added and then unlocked
> without first being committed.
> 
> In this situation, the file has a typechange between the index and
> the working tree. However, by the time the pre-commit hook gets
> run, git commit has already staged the unlocked file content into the
> index. So, there is nolonger a typechange between the index and the work
> tree. And, since the file is new, there is no typechange between the
> index and last commit either. The latter is what the pre-commit hook
> looks for.
> 
> Conclusion: The pre-commit hook cannot possibly detect this situation.
> Instead, it seems the only way to block the problem is to prevent
> unlocking a file that does not have any git history.
> 
> And I've [[done]] that now. --[[Joey]]