Re: [PATCH v6 2/5] git-remote-mediawiki: new git bin-wrapper for developement

2013-07-01 Thread Matthieu Moy
Benoît Person benoit.per...@ensimag.fr writes: Another idea crossed my mind: for now the test suite creates a symlink of git-remote-mediawiki in the toplevel if it's not installed. It would be better to use the bin-wrapper in the testsuite I think ? Absolutely. The symlink was a dirty hack

Re: [PATCH v6 2/5] git-remote-mediawiki: new git bin-wrapper for developement

2013-06-29 Thread Benoît Person
GIT_ROOT_DIR=../../.. GIT_EXEC_PATH=$(cd $(dirname $0) cd ${GIT_ROOT_DIR} pwd) GIT_MEDIAWIKI=$GIT_EXEC_PATH/contrib/mw-to-git PATH=$GIT_MEDIAWIKI/contrib/mw-to-git:$PATH GPLEXTRA=$GIT_MEDIAWIKI/contrib/mw-to-git exec ${GIT_EXEC_PATH}/bin-wrappers/git $@ Should I do that in a reroll ?

Re: [PATCH v6 2/5] git-remote-mediawiki: new git bin-wrapper for developement

2013-06-27 Thread Junio C Hamano
benoit.per...@ensimag.fr writes: diff --git a/contrib/mw-to-git/bin-wrapper/git b/contrib/mw-to-git/bin-wrapper/git new file mode 100755 index 000..aa714a5 --- /dev/null +++ b/contrib/mw-to-git/bin-wrapper/git @@ -0,0 +1,27 @@ +#!/bin/sh + +# git executable wrapper script for

Re: [PATCH v6 2/5] git-remote-mediawiki: new git bin-wrapper for developement

2013-06-27 Thread Matthieu Moy
Junio C Hamano gits...@pobox.com writes: As far as I can tell, the only real reason why you need this and cannot use ../../bin-wrappers/git directly is because the GITPERLLIB it gives you only points at ../../perl/blib/lib and not this directory. Not only. You also need to have

Re: [PATCH v6 2/5] git-remote-mediawiki: new git bin-wrapper for developement

2013-06-27 Thread Benoît Person
As far as I can tell, the only real reason why you need this and cannot use ../../bin-wrappers/git directly is because the GITPERLLIB it gives you only points at ../../perl/blib/lib and not this directory. Plus (forgot to mention it in the other mail :/ ) it enables us to not copy git-mw and