From: "Simon.Cathebras" <Simon.Cathebras@ensimag.imag.fr>
To: konglu@minatec.inpg.fr
Cc: git@vger.kernel.org, Matthieu Moy <matthieu.moy@imag.fr>,
Guillaume Sasdy <guillaume.sasdy@ensimag.imag.fr>,
Julien Khayat <julien.khayat@ensimag.imag.fr>,
charles.roussel@ensimag.imag.fr, gitster@pobox.com,
peff@peff.net, Simon Perrat <simon.perrat@ensimag.imag.fr>
Subject: Re: [PATCHv3 3/6] Test file for git-remote-mediawiki clone
Date: Tue, 12 Jun 2012 15:58:48 +0200 [thread overview]
Message-ID: <4FD74B18.7080205@ensimag.imag.fr> (raw)
In-Reply-To: <20120611230720.Horde.Xx4mVHwdC4BP1l4IcI6RWrA@webmail.minatec.grenoble-inp.fr>
On 11/06/2012 23:07, konglu@minatec.inpg.fr wrote:
>
> Simon Cathebras <simon.cathebras@ensimag.imag.fr> a écrit :
>
>
>> +test_expect_success 'Git clone creates the git log expected with one
>> file' '
>> + wiki_reset &&
>> + cd '"$TRASH_DIR"' &&
>> + rm -rf mw_dir &&
>> + wiki_editpage foo "this is not important" false -c cat -s "this
>> must be the same" &&
>> + git clone mediawiki::http://localhost/wiki mw_dir &&
>> + cd mw_dir &&
>> + git log --format=%s HEAD^..HEAD > log.tmp &&
>> + echo "this must be the same" > msg.tmp &&
>> + diff -b log.tmp msg.tmp &&
>> + cd .. &&
>> + rm -rf mw_dir
>> +'
>> +
>
> It's best not to chdir around outside a subshell (same comment for all
> the
> other tests)
Thanks, we managed to change these problems of chdir.
>
> BTW, please be aware of the whitespaces (the first 5 parts of your
> patch are
> full of it - couldn't apply the last one because of the special
> characters, maybe
> something that i haven't configure, i don't know).
>
>
Got it. We modified this issue with whitespaces.
In addition, we know what prevented you to install the last patch:
encoding problem. It will be fixed in the next update.
Simon
--
CATHEBRAS Simon
2A-ENSIMAG
Filière Ingéniérie des Systèmes d'Information
Membre Bug-Buster
next prev parent reply other threads:[~2012-06-12 13:59 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-11 20:27 [PATCH]Test environment for git-remote-mediawiki Simon.Cathebras
2012-06-11 20:28 ` [PATCHv3 1/6] Scripts to install, delete and clear a MediaWiki Simon Cathebras
2012-06-11 20:28 ` [PATCHv3 2/6] Test environment of git-remote-mediawiki Simon Cathebras
2012-06-11 20:28 ` [PATCHv3 3/6] Test file for git-remote-mediawiki clone Simon Cathebras
2012-06-11 21:07 ` konglu
2012-06-12 13:58 ` Simon.Cathebras [this message]
2012-06-11 20:28 ` [PATCHv3 4/6] Tests for git-remote-mediawiki pull Simon Cathebras
2012-06-11 21:09 ` konglu
2012-06-12 13:58 ` Simon Perrat
2012-06-12 21:12 ` [PATCH 1/6] Scripts to install, delete and clear a MediaWiki Simon Cathebras
2012-06-12 21:12 ` [PATCH 2/6] Test environment of git-remote-mediawiki Simon Cathebras
2012-06-13 7:56 ` Matthieu Moy
2012-06-13 8:10 ` Simon.Cathebras
2012-06-13 10:14 ` Ævar Arnfjörð Bjarmason
2012-06-13 17:00 ` Simon.Cathebras
2012-06-13 17:03 ` [PATCHv5 1/5] Scripts to install, delete and clear a MediaWiki Simon Cathebras
2012-06-13 17:03 ` [PATCHv5 2/5] Test environment of git-remote-mediawiki Simon Cathebras
2012-06-13 17:03 ` [PATCHv5 3/5] Test file for git-remote-mediawiki clone Simon Cathebras
2012-06-13 17:03 ` [PATCHv5 4/5] Tests for git-remote-mediawiki pull and push Simon Cathebras
2012-06-13 17:03 ` [PATCHv5 5/5] Tests of UTF8 character with git-remote-mediawiki Simon Cathebras
2012-06-14 8:57 ` [PATCHv5 1/5] Scripts to install, delete and clear a MediaWiki Matthieu Moy
2012-06-14 8:57 ` [PATCH 1/3] chmod -x test-gitmw-lib.sh Matthieu Moy
2012-06-14 8:57 ` [PATCH 2/3] Coding style Matthieu Moy
2012-06-14 8:57 ` [PATCH 3/3] Explicit error when curl_exec() fails Matthieu Moy
2012-06-14 9:23 ` Simon.Cathebras
2012-06-14 12:45 ` Matthieu Moy
2012-06-14 9:20 ` [PATCHv5 1/5] Scripts to install, delete and clear a MediaWiki Simon.Cathebras
2012-06-14 16:17 ` Matthieu Moy
2012-06-14 8:58 ` Matthieu Moy
2012-06-12 21:12 ` [PATCH 3/6] Test file for git-remote-mediawiki clone Simon Cathebras
2012-06-12 21:34 ` konglu
2012-06-13 7:20 ` Simon.Cathebras
2012-06-12 21:12 ` [PATCH 4/6] Tests for git-remote-mediawiki pull and push Simon Cathebras
2012-06-12 21:12 ` [PATCH 5/6] Tests of UTF8 character with git-remote-mediawiki Simon Cathebras
2012-06-12 21:18 ` Simon.Cathebras
2012-06-12 21:52 ` konglu
2012-06-13 7:30 ` Simon.Cathebras
2012-06-12 21:45 ` konglu
2012-06-11 20:28 ` [PATCHv3 5/6] Test file for git-remote-mediawiki push Simon Cathebras
2012-06-11 20:28 ` =?y?q?=5BPATCHv3=206/6=5D=20Tests=20of=20UTF8=20character=20with=20git-remote-mediawiki?= Simon Cathebras
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4FD74B18.7080205@ensimag.imag.fr \
--to=simon.cathebras@ensimag.imag.fr \
--cc=charles.roussel@ensimag.imag.fr \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=guillaume.sasdy@ensimag.imag.fr \
--cc=julien.khayat@ensimag.imag.fr \
--cc=konglu@minatec.inpg.fr \
--cc=matthieu.moy@imag.fr \
--cc=peff@peff.net \
--cc=simon.perrat@ensimag.imag.fr \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).