comment
authorJoey Hess <joeyh@joeyh.name>
Fri, 9 Dec 2022 16:18:37 +0000 (12:18 -0400)
committerJoey Hess <joeyh@joeyh.name>
Fri, 9 Dec 2022 16:18:37 +0000 (12:18 -0400)
doc/bugs/git_annex_test_never_exits__63__/comment_10_d258114e6e8f8db28b65a39cb13f2a04._comment [new file with mode: 0644]

diff --git a/doc/bugs/git_annex_test_never_exits__63__/comment_10_d258114e6e8f8db28b65a39cb13f2a04._comment b/doc/bugs/git_annex_test_never_exits__63__/comment_10_d258114e6e8f8db28b65a39cb13f2a04._comment
new file mode 100644 (file)
index 0000000..4a2a692
--- /dev/null
@@ -0,0 +1,29 @@
+[[!comment format=mdwn
+ username="joey"
+ subject="""comment 10"""
+ date="2022-12-09T16:16:10Z"
+ content="""
+The retest seemed to hang again, but I'm not sure I was testing the right
+binary. Running it again.
+
+This patch exposes when the lock is held and so illustrates the bug
+reliably and quickly:
+
+       diff --git a/Command/Smudge.hs b/Command/Smudge.hs
+       index a5d8871998..cf6e2973f8 100644
+       --- a/Command/Smudge.hs
+       +++ b/Command/Smudge.hs
+       @@ -127,7 +129,8 @@ clean'
+               -> (Key -> Annex ())
+               -- ^ emitpointer: Emit a pointer file for the key.
+               -> Annex ()
+       -clean' file mk passthrough discardreststdin emitpointer =
+       +clean' file mk passthrough discardreststdin emitpointer = do
+       +       calcRestageLog () (\_ _ -> ())
+               ifM (fileOutsideRepo file)
+                       ( passthrough
+                       , inSmudgeCleanFilter go
+
+With that patch, the unfixed git-annex hangs immediately. With
+my fix, it does not hang, which seems to show I fixed it correctly..
+"""]]