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

Added a comment

Browse source
This commit is contained in:
https://www.google.com/accounts/o8/id?id=AItOawmkBwMWvNKZZCge_YqobCSILPMeK6xbFw8 2013-12-31 18:20:32 +00:00 committed by admin
parent 9aac2d54ff
commit 66dd1c3151
1 changed files with 16 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
doc/design/external_special_remote_protocol/comment_7_edb649d0019a061ef7bf6534a444429d._comment Normal file
View file

@ -0,0 +1,16 @@
[[!comment format=mdwn
username="https://www.google.com/accounts/o8/id?id=AItOawmkBwMWvNKZZCge_YqobCSILPMeK6xbFw8"
nickname="TobiasTheViking"
subject="comment 7"
date="2013-12-31T18:20:32Z"
content="""
I think the hook running anything in shell, to interact with git annex, is a mistake.
I see a lot more potential pitfalls and mistakes(especially crossplatform).
It should be the existing git annex plumbing (preferred content) as you say. I just really think it should be configurable in the protocol, instead of a having to run a shell command.
Since you have made this advanced protocol i really see it as a mistake to do anything between the hook and git-annex outside of the protocol, it makes much more sense to have all their interactions happen within the protocol.
IMO anyways.
"""]]