threads / discuss / 1030

'dotest' script broken

Subject: 'dotest' script broken

## tl;dr

3 messages between Jun 26, 2005 and Jun 26, 2005.

replies: 2people: 2as markdown or json

Jeff Garzik· Jun 26, 2005, 21:43 UTC · lore

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· Jun 26, 2005, 21:51 UTC · re: Jeff Garzik · lore

Re: 'dotest' script broken

On Sun, 26 Jun 2005, Jeff Garzik wrote:

Show 12 quoted lines

> 
> 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· Jun 26, 2005, 22:39 UTC · re: Linus Torvalds · lore

Re: 'dotest' script broken

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