aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar Joey Hess <joey@kitenet.net>2012-09-16 23:09:08 -0400
committerGravatar Joey Hess <joey@kitenet.net>2012-09-16 23:09:08 -0400
commitba744c84a4f683e50bf4c9b8c388e3a611f7fb91 (patch)
tree5f2e3a717ae629c9cb1ca14da59d97e7e8aae366
parent2081a31fdaf8b76df6bdee82f48ec1e3105f5abe (diff)
better name for fallback sync refs
Don't expose these as branches in refs/heads/. Instead hide them away in refs/synced/ where only show-ref will find them. Make unused only look at branches and tags, not these other things, so it won't care if some stale sync ref used to use a file. This means they don't need to be deleted, which could have led to an incoming sync being missed.
-rw-r--r--Assistant/Sync.hs8
-rw-r--r--Assistant/Threads/Merger.hs18
-rw-r--r--Command/Unused.hs4
-rw-r--r--Git/Ref.hs5
4 files changed, 15 insertions, 20 deletions
diff --git a/Assistant/Sync.hs b/Assistant/Sync.hs
index b78a2a5a4..326baff78 100644
--- a/Assistant/Sync.hs
+++ b/Assistant/Sync.hs
@@ -90,7 +90,7 @@ pushToRemotes threadname now st mpushmap remotes = do
go True branch g u remotes
where
go _ Nothing _ _ _ = return True -- no branch, so nothing to do
- go shouldretry (Just branch) g u rs = do
+ go shouldretry (Just branch) g u rs = do
debug threadname
[ "pushing to"
, show rs
@@ -133,12 +133,12 @@ pushToRemotes threadname now st mpushmap remotes = do
, Param $ refspec branch
] g
where
+ {- Push to refs/synced/uuid/branch; this
+ - avoids cluttering up the branch display. -}
refspec b = concat
[ s
, ":"
- , show $ Git.Ref.base $
- Command.Sync.syncBranch $ Git.Ref $
- "fallback" </> fromUUID u </> s
+ , "refs" </> "synced" </> fromUUID u </> s
]
where s = show $ Git.Ref.base b
diff --git a/Assistant/Threads/Merger.hs b/Assistant/Threads/Merger.hs
index 52db625cd..ce1ff1c7b 100644
--- a/Assistant/Threads/Merger.hs
+++ b/Assistant/Threads/Merger.hs
@@ -15,18 +15,17 @@ import qualified Annex.Branch
import qualified Git
import qualified Git.Merge
import qualified Git.Branch
-import qualified Git.Command as Git
thisThread :: ThreadName
thisThread = "Merger"
-{- This thread watches for changes to .git/refs/heads/synced/,
- - which indicate incoming pushes. It merges those pushes into the
- - currently checked out branch. -}
+{- This thread watches for changes to .git/refs/, looking for
+ - incoming pushes. It merges those pushes into the currently
+ - checked out branch. -}
mergeThread :: ThreadState -> NamedThread
mergeThread st = thread $ do
g <- runThreadState st $ fromRepo id
- let dir = Git.localGitDir g </> "refs" </> "heads" </> "synced"
+ let dir = Git.localGitDir g </> "refs"
createDirectoryIfMissing True dir
let hook a = Just $ runHandler g a
let hooks = mkWatchHooks
@@ -82,11 +81,6 @@ onAdd g file _
, show current
]
void $ Git.Merge.mergeNonInteractive changedbranch g
- when ("fallback/" `isInfixOf` (show changedbranch)) $
- void $ Git.runBool "branch"
- [ Param "-D"
- , Param $ show changedbranch
- ] g
go _ = noop
equivBranches :: Git.Ref -> Git.Ref -> Bool
@@ -100,6 +94,6 @@ isAnnexBranch f = n `isSuffixOf` f
n = "/" ++ show Annex.Branch.name
fileToBranch :: FilePath -> Git.Ref
-fileToBranch f = Git.Ref $ "refs" </> "heads" </> base
+fileToBranch f = Git.Ref $ "refs" </> base
where
- base = Prelude.last $ split "/refs/heads/" f
+ base = Prelude.last $ split "/refs/" f
diff --git a/Command/Unused.hs b/Command/Unused.hs
index 39a7a59cf..f5eb4b7dd 100644
--- a/Command/Unused.hs
+++ b/Command/Unused.hs
@@ -251,7 +251,9 @@ withKeysReferencedInGit a = do
rs <- relevantrefs <$> showref
forM_ rs (withKeysReferencedInGitRef a)
where
- showref = inRepo $ Git.Command.pipeRead [Param "show-ref"]
+ {- List heads and tags, but not other refs used in syncing. -}
+ showref = inRepo $ Git.Command.pipeRead
+ [Param "show-ref", Param "--heads", Param "--tags"]
relevantrefs = map (Git.Ref . snd) .
nubBy uniqref .
filter ourbranches .
diff --git a/Git/Ref.hs b/Git/Ref.hs
index 021dbe6cc..cfaafacef 100644
--- a/Git/Ref.hs
+++ b/Git/Ref.hs
@@ -49,9 +49,8 @@ sha branch repo = process <$> showref repo
{- List of (refs, branches) matching a given ref spec. -}
matching :: Ref -> Repo -> IO [(Ref, Branch)]
-matching ref repo = do
- r <- pipeRead [Param "show-ref", Param $ show ref] repo
- return $ map gen (lines r)
+matching ref repo = map gen . lines <$>
+ pipeRead [Param "show-ref", Param $ show ref] repo
where
gen l = let (r, b) = separate (== ' ') l in
(Ref r, Ref b)