Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 19 Jun 1997 10:56:12 -0700
From:      "Jordan K. Hubbard" <jkh@time.cdrom.com>
To:        Bruce Evans <bde@zeta.org.au>
Cc:        cvs-all@FreeBSD.ORG, cvs-bin@FreeBSD.ORG, cvs-committers@FreeBSD.ORG, jkh@FreeBSD.ORG
Subject:   Re: cvs commit: src/bin/sh expand.c 
Message-ID:  <23917.866742972@time.cdrom.com>
In-Reply-To: Your message of "Fri, 20 Jun 1997 01:30:14 %2B1000." <199706191530.BAA13078@godzilla.zeta.org.au> 

next in thread | previous in thread | raw e-mail | index | archive | help
> The submitted patch misses the point and should never have been committed.

C'mon, Bruce, don't beat around the bush - what are you trying to say? :-)

> See followup mail in the PR about what the problem actually is.

OK, sorry, I missed this somehow.

> Submitted patches are almost never suitable for committing directly.

Well, I did look at it first, sheesh! ;) Given the mastications that
sh was doing on the value of the integer there, the fix seemed
perfectly correct so I committed it.  I didn't know I was walking into
a POSIX war in the process, the kind where you generally tend to note
27 different things which are wrong in copious detail and thus scare
everyone away from actually trying to treat the affected areas. :-)

Been there, done that already, am backing out the change right now. :)

					Jordan



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?23917.866742972>