use lookupKeyStaged in --batch code paths
authorJoey Hess <joeyh@joeyh.name>
Wed, 26 Oct 2022 18:23:06 +0000 (14:23 -0400)
committerJoey Hess <joeyh@joeyh.name>
Wed, 26 Oct 2022 18:43:06 +0000 (14:43 -0400)
commit731e806c9638f4f5852b965a2e3430d3df83d3c1
tree4afa00075610d65543d93a67d22c7cc96029f0f5
parentb2ee2496ee9b22df6d61b73e71ff09e17267e98c
use lookupKeyStaged in --batch code paths

Make --batch mode handle unstaged annexed files consistently whether the
file is unlocked or not. Before this, a unstaged locked file
would have the symlink on disk examined and operated on in --batch mode,
while an unstaged unlocked file would be skipped.

Note that, when not in batch mode, unstaged files are skipped over too.
That is actually somewhat new behavior; as late as 7.20191114 a
command like `git-annex whereis .` would operate on unstaged locked
files and skip over unstaged unlocked files. That changed during
optimisation of CmdLine.Seek with apparently little fanfare or notice.

Turns out that rmurl still behaved that way when given an unstaged file
on the command line. It was changed to use lookupKeyStaged to
handle its --batch mode. That also affected its non-batch mode, but
since that's just catching up to the change earlier made to most
other commands, I have not mentioed that in the changelog.

It may be that other uses of lookupKey should also change to
lookupKeyStaged. But it may also be that would slow down some things,
or lead to unwanted behavior changes, so I've kept the changes minimal
for now.

An example of a place where the use of lookupKey is better than
lookupKeyStaged is in Command.AddUrl, where it looks to see if the file
already exists, and adds the url to the file when so. It does not matter
there whether the file is staged or not (when it's locked). The use of
lookupKey in Command.Unused likewise seems good (and faster).

Sponsored-by: Nicholas Golder-Manning on Patreon
Annex/WorkTree.hs
CHANGELOG
CmdLine/Batch.hs
Command/MetaData.hs
Command/RmUrl.hs
doc/bugs/addurl_+_metadata_on_Windows_doesn__39__t_work.mdwn
doc/bugs/addurl_+_metadata_on_Windows_doesn__39__t_work/comment_2_88b7db5434a56c25c75772caa37bc14a._comment [new file with mode: 0644]