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

Added a comment: partial implementation plan

Browse source
This commit is contained in:
http://joeyh.name/ 2014-03-19 16:37:09 +00:00 committed by admin
parent e5b0ba4d0b
commit 84f894bc3c
1 changed files with 14 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
doc/todo/required_content/comment_2_132ec6378db63af6281569cf5748b9d3._comment Normal file
View file

@ -0,0 +1,14 @@
[[!comment format=mdwn
username="http://joeyh.name/"
ip="209.250.56.154"
subject="partial implementation plan"
date="2014-03-19T16:37:05Z"
content="""
* When there is a required content expression, OR it with the preferred content expression. This will make the assistant, --auto etc want to get required content.
* When checking if something can be dropped, check the required content expression. Only Command.Drop does this so nicely centralized in one place.
* When checking required content expression for drop, must do active verification of terminals that relate to other copies, for the same reason the numcopies check when dropping does active validation. This includes `copies=`, `lackingcopies=`, `approxlackingcopies=`, `inallgroup=`.
The last is where the complication comes in. Seems to need a cache of places the key was just now verified to be present, which can be used to avoid unnecessary redundant active verification (for example \"not (copies=2 and inallgroup=bar)\" would otherwise redundantly check some remotes). The numcopies checking code should use the same cache too.
* Finally, if the required content cannot be satisfied, it would be nice to know which terminal failed in order to show the user a useful error message. The current Matcher does not provide a way to do that though. Or could just say, \"unable to satisfy required content: $expression\"
"""]]