On Wed, 26 Aug 2009, Junio C Hamano wrote: > * db/vcs-helper (2009-08-09) 17 commits > - Allow helpers to request marks for fast-import > - Allow helpers to report in "list" command that the ref is unchanged > - Add support for "import" helper command > - transport-helper_init(): fix a memory leak in error path > - Add a config option for remotes to specify a foreign vcs > - Allow programs to not depend on remotes having urls > - Allow fetch to modify refs > - Use a function to determine whether a remote is valid > - Use a clearer style to issue commands to remote helpers > (merged to 'next' on 2009-08-07 at f3533ba) > + Makefile: install hardlinks for git-remote-<scheme> supported by libcurl if possible > + Makefile: do not link three copies of git-remote-* programs > + Makefile: git-http-fetch does not need expat > (merged to 'next' on 2009-08-06 at 15da79d) > + http-fetch: Fix Makefile dependancies > + Add transport native helper executables to .gitignore > (merged to 'next' on 2009-08-05 at 33d491e) > + git-http-fetch: not a builtin > + Use an external program to implement fetching with curl > + Add support for external programs for handling native fetches > (this branch is used by jh/cvs-helper.) > > There was a discussion that suggests that the use of colon ':' before vcs > helper name needs to be corrected. Nothing happened since. I believe the outcome of that discussion was: - We want to keep supporting using regular location URLs that are URLs of git repositories (e.g., http://git.savannah.gnu.org/cgit/xboard.git), and we probably want to do it with a helper which runs when run_command() is given "remote-<scheme>". I think installing hardlinks in EXECPATH ended up being the best implementation here. This is currently a special case, because these URLs have push handled internally (or, rather, with internal code calling a different external program), but I think we want to make it no longer special at all, so that people can install the handling for access to native repos via dumb filesystem-like protocols separately. This is in next. - We want to support a separate "vcs" option for cases where repositories in the foreign system need to be addressed through the combination of a bunch of options, which will be read from the configuration by the helper. The helper which gets run is "remote-<value of vcs option>". This is in pu. - We want to support URLs of some sort leading to using helpers appropriate for foreign systems that use URLs and are most conveniently located this way. We didn't come to any concensus on what this should do, but we also haven't had any helpers proposed yet that would use it, and the series doesn't include any code that would lead to running a helper other than in one of the above two cases. (jh's cvs helper clearly wants "cvsroot" and "cvsmodule" options, and my p4 helper wants "port" and "codeline" options; SVN is the big use cases for URLs, but nobody's tackled that as a helper) So I think this issue is squarely in "future work" anyway, and the current part of the series should be able to move forward, unless I've missed some other issue. Of course, there are a bunch of things that are beyond the present contents of the series, but I think there's nothing wrong in the series as it is, and it's worthwhile without any further patches. -Daniel *This .sig left intentionally blank* -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html