From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 14 Jun 2013 18:09:27 GMT
Subject: Re: [PATCH V2] git-remote-mediawiki: remove hardcoded version number in the test suite
Message-ID: <7vsj0ko97s.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1371205151-13549-1-git-send-email-benoit.person@ensimag.fr>

benoit.person@ensimag.fr writes:

> From: Benoit Person <benoit.person@ensimag.fr>
>
> Updates the code to make it more easy to switch mediawiki version when
> testing. Before that, the version number was partly hardcoded, partly
> in a var.

This definitely goes in a good direction; you no longer have to
touch random lines in test-gitmw-lib.sh and what you have to edit 
is very much localized.

It seems t/test.config that is tracked is also designed to be
randomly edited to suit user's needs.

The patch does not make things any worse, but I would imagine that a
better longer term structure would be for contrib/mw-to-git/Makefile
to create contrib/mw-to-git/t/test.config by replacing variables in
contrib/mw-to-git/t/test.config-template, so that the user could say

	cd contrib/mw-to-git
        make MW_VERSION_MAJOR=1.20 MW_VERSION_MINOR=0 test

or something like that.

But of course, that is a totally unrelated issue.

Will apply.  Thanks.