bugfix: Pass --full-tree when using git ls-files to get a list of files on the git-annex branch, so it works when run in a subdirectory. This bug affected git-annex unused, and potentially also transitions running code and other things.

This commit is contained in:
Joey Hess 2015-07-06 14:09:24 -04:00
parent c9c63b092e
commit 174da80ddc
4 changed files with 23 additions and 0 deletions

View file

@ -316,6 +316,7 @@ files = do
branchFiles :: Annex [FilePath]
branchFiles = withIndex $ inRepo $ Git.Command.pipeNullSplitZombie
[ Param "ls-tree"
, Param "--full-tree"
, Param "--name-only"
, Param "-r"
, Param "-z"

4
debian/changelog vendored
View file

@ -12,6 +12,10 @@ git-annex (5.20150618) UNRELEASED; urgency=medium
* Brought back the setkey plumbing command that was removed in 2011, since
we found a use case for it. Note that the command's syntax was changed
for consistency.
* bugfix: Pass --full-tree when using git ls-files to get a list of files
on the git-annex branch, so it works when run in a subdirectory.
This bug affected git-annex unused, and potentially also transitions
running code and other things.
-- Joey Hess <id@joeyh.name> Thu, 02 Jul 2015 12:31:14 -0400

View file

@ -27,3 +27,5 @@ arch linux x86_64
### Please provide any additional information below.
The S3 remote is encrypted with the default "hybrid" method
> [[fixed|done]] --[[Joey]]

View file

@ -0,0 +1,16 @@
[[!comment format=mdwn
username="joey"
subject="""comment 4"""
date="2015-07-06T18:00:53Z"
content="""
Analysis: To find the keys, it's using git ls-tree git-annex. When this is
run in a subdirectory, it only lists files in the branch located under that
subdirectory. Which is no files. It should pass --full-tree to avoid that
problem.
Pretty much everywhere git-annex uses ls-tree, it does pass --full-tree.
But not in this particular case. Anything using Annex.Branch.branchFiles or
Annex.Branch.files would be affected; that's limited to git-annex unused
and code that runs transitions. git-annex unused w/o --from could also be
affected, I think, perhaps it's harder to trigger that though.
"""]]