# 'dotest' script broken

3 messages from 2005-06-26 to 2005-06-26. Participants: Jeff Garzik, Linus Torvalds.
Thread: https://gitlist.dev/t/1030

## Jeff Garzik, 2005-06-26 21:43

Subject: 'dotest' script broken
Message-ID: <42BF2167.9030404@pobox.com>
URL: https://gitlist.dev/e/42BF2167.9030404%40pobox.com

```

When I run 'dotest /g/tmp/mbox' I receive an error message

	[jgarzik@pretzel netdev-2.6]$ dotest /g/tmp/mbox
	mailinfo msg-file path-file filelist-file < email

and if you look at the dotest script, it only passes 2 args to mailinfo, 
not the 3 args that the above usage message appears to require:

for i in .dotest/*
do
         mailinfo .dotest/msg .dotest/patch < $i > .dotest/info || exit 1
	...

```

## Linus Torvalds, 2005-06-26 21:51

Subject: Re: 'dotest' script broken
Message-ID: <Pine.LNX.4.58.0506261450000.19755@ppc970.osdl.org>
URL: https://gitlist.dev/e/Pine.LNX.4.58.0506261450000.19755%40ppc970.osdl.org
In-Reply-To: <42BF2167.9030404@pobox.com>

```


On Sun, 26 Jun 2005, Jeff Garzik wrote:
> 
> When I run 'dotest /g/tmp/mbox' I receive an error message
> 
> 	[jgarzik@pretzel netdev-2.6]$ dotest /g/tmp/mbox
> 	mailinfo msg-file path-file filelist-file < email
> 
> and if you look at the dotest script, it only passes 2 args to mailinfo, 
> not the 3 args that the above usage message appears to require:
> 
> for i in .dotest/*
> do
>          mailinfo .dotest/msg .dotest/patch < $i > .dotest/info || exit 1

Looks like you updated "dotest", but didn't update "mailinfo"? Why? 
Especially as they come in the same package..

The new mailinfo wants just two arguments to match the new dotest.

			Linus

```

## Jeff Garzik, 2005-06-26 22:39

Subject: Re: 'dotest' script broken
Message-ID: <42BF2EA4.4070207@pobox.com>
URL: https://gitlist.dev/e/42BF2EA4.4070207%40pobox.com
In-Reply-To: <Pine.LNX.4.58.0506261450000.19755@ppc970.osdl.org>

```
Linus Torvalds wrote:
> Looks like you updated "dotest", but didn't update "mailinfo"? Why? 


Duh.  I was so used to just updating git-tools without 'make install'

Sorry for the noise.

	Jeff

```
