I played with GIT_ASKPASS and managed to get a fully automated behavior without any user prompt. I guess that the patches are not necessary as we can provide the same , it is just less straightforward than direct parameters but I understand the security concerns.
I dont know if you plan to keep GIT_ASKPASS as is, but this environment variable is not mentioned on git svn documentation and I guess just a few people know about it. Moreover, the name is misleading as it handles all the prompts, not only the password one. Arnaud Le 15 oct. 2013 à 01:35, Eric Wong <normalper...@yhbt.net> a écrit : > Jeff King <p...@peff.net> wrote: >> On Mon, Oct 14, 2013 at 06:40:05PM +0000, Eric Wong wrote: >> >>>> arnaud.brej...@gmail.com wrote: >>>> >>>>> Signed-off-by: Arnaud Brejeon <arnaud.brejeon <at> gmail.com> >>>> >>>> Thanks. >>>> >>>> Can you say a little more about the context? Do you run a script that >>>> wants to pass a password to 'git svn', do you type it each time on the >>>> command line, or something else? Is it ok that the password would >>>> show up in "ps" output? Would the platform's keyring or netrc be >>>> usable here, or is there something in the context that avoids that? >>> >>> I think using keyring or netrc is more appropriate. Having a password >>> on the command-line and visible to all via ps doesn't seem like >>> something git should support. >> >> Agreed. We have ready-made git-credential helpers to handle this exact >> problem. We would need to convert SVN::Prompt to use git-credential >> rather than prompting itself, though. One of the things that held me >> back from writing such a patch is that I thought libsvn already handled >> things like keychain integration, and it was better for git-svn to be >> more svn-like than git-like in its access of SVN repos. >> >> Are those already supported out of the box by libsvn? If git's >> credential helpers are significantly more featureful, it might be worth >> converting, but if not, I think it makes sense to stay with svn's >> existing code. > > I looks like this patch was forgotten once again: > > http://mid.gmane.org/1371573490-21973-1-git-send-email-matth...@stdin.nl > > Matthijs: can you add a Signed-off-by for your patch? I'm inclined to > push it to Junio as-is since it looks reasonable. > I admit I don't know SVN callbacks anymore well enough and don't have > time to test with GNOME. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html