mirrors/git-annex
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

analysis

Browse source
This commit is contained in:
Joey Hess 2018-06-04 13:14:20 -04:00
parent 2e6a6024c2
commit dc5550a54e
No known key found for this signature in database
GPG key ID: DB12DB0FF05F8F38
1 changed files with 28 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
doc/bugs/annex-checkuuid_renders_remotes_inaccessible/comment_1_451498fab01ae745eb33d42da8682ad3._comment Normal file
View file

@ -0,0 +1,28 @@
[[!comment format=mdwn
username="joey"
subject="""comment 1"""
date="2018-06-04T16:44:39Z"
content="""
I think checkuuid=false has always had this problem; it's never been
actually usable.
Problem is that checkuuid=false prevents reading the git config, so
the Repo is LocalUnknown, and such repositories are assumed to not be
currently available to use.
To get from LocalUnknown to Local, it would have to read the repo's
config to determine if the repo is bare or not. But reading the repo's
config every time is exactly what checkuuid=false is intended to prevent.
Only path I can see is to make the DeferredUUIDCheck that's done
with checkuuid=false return the Local Repo. That could be done in
Remote.Git. But, other modules use Remote.repo to look at the Repo too,
including Command.Sync, and Assistant.TransferSlots. Remote.repo can't
be updated after it's constructed without some mess of updating the remote
list, which does not seem like a good idea. Seems that Remote.repo would
need to be converted to an IO action.
(So would Remote.gitconfig, which incidentally doesn't contain the config
of the remote when checkuuid=false, which could also be considered a bug
with checkuuid=false.)
"""]]