From: Michael J Gruber <michaeljgruber+gmane@fastmail.fm>
To: git@vger.kernel.org
Subject: Re: Test breakage [Re: [ANNOUNCE] GIT 1.5.6.2]
Date: Tue, 08 Jul 2008 11:47:45 +0200 [thread overview]
Message-ID: <g4vd41$psa$1@ger.gmane.org> (raw)
In-Reply-To: <20080708051622.GC2037@sigill.intra.peff.net>
Jeff King venit, vidit, dixit 08.07.2008 07:16:
> On Mon, Jul 07, 2008 at 02:44:49PM +0200, Michael J Gruber wrote:
>
>> On Kubuntu 7.10 (not my choice, this is at work...), "make test" gives:
>>
>> *** t9600-cvsimport.sh ***
>> [...]
>> * FAIL 6: update git module
>> cd module-git &&
>> git cvsimport -a -z 0 module &&
>> git merge origin &&
>> cd .. &&
>> test_cmp module-cvs/o_fortuna module-git/o_fortuna
>>
>> On the second (and subsequent) run, "make test" succeeds. I tried again
>> with a fresh copy from the tarball, same effect: failure on 1, success on
>> 2 and following.
>
> I tried reproducing on my Debian box with various weaks of $LANG, but I
> couldn't. Can you please try running "./t9600-cvsimport.sh -v -i" and
> send us the output of a failing instance. It's not clear which step in
> failing test is causing the problem, or if cvsps is giving off any
> useful errors.
>
>> This is with LANG=de_DE.UTF-8, whereas with LANG=C the tests succeed the
>> first time already. Are tests supposed to be done in C locale only? Do
>> they reuse data from a previous test run? I didn't notice this with
>> 1.5.6.1.
>
> They should be perfectly repeatable, but I have run into problems in the
> past with cruft in my ~/.cvsps directory. However, we should be setting
> $HOME properly in t9600 to avoid this, so I'm not sure what the culprit
> is.
Sure enough, I can't reproduce it today. I reproduced that effect
several times yesterday, or else I would not have reported anything.
Trying the very same steps today the tests succeed on the first run.
Only differences between yesterday and today are be:
- I installed git 1.5.6.2 now (had 1.5.6.1 before) under $HOME, with
PATH, PERL5LIB, LD_LIBRARY_PATH set accordingly; but tests should pick
up git stuff from the local build dir, right?
- There may have been an automated update to kubuntu meanwhile, but I
don't think so; cvsps is under $HOME anyways, and what else would
matter? Perl version maybe?
So it's WORKSFORME now. Sorry for the noise. Next time I'll not only try
to reproduce (as I did) but wait a day also!
Best
Michael
next prev parent reply other threads:[~2008-07-08 9:48 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-06 5:34 [ANNOUNCE] GIT 1.5.6.2 Junio C Hamano
2008-07-06 8:09 ` Jakub Narebski
2008-07-06 9:30 ` Junio C Hamano
2008-07-06 12:44 ` Jan Engelhardt
2008-07-06 14:54 ` Mikael Magnusson
2008-07-07 12:44 ` Test breakage [Re: [ANNOUNCE] GIT 1.5.6.2] Michael J Gruber
2008-07-08 5:16 ` Jeff King
2008-07-08 9:47 ` Michael J Gruber [this message]
2008-07-14 3:13 ` [ANNOUNCE] GIT 1.5.6.3 Junio C Hamano
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='g4vd41$psa$1@ger.gmane.org' \
--to=michaeljgruber+gmane@fastmail.fm \
--cc=git@vger.kernel.org \
/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).