On Wed, Jul 21, 2010 at 00:01, Jonathan Nieder <jrnieder@xxxxxxxxx> wrote: > Ævar Arnfjörð Bjarmason wrote: > >> You must use "export PAGER;", not "export PAGER &&". export doesn't >> return zero on all systems when exporting, see previous changes in >> this regard in t/. > > Nope. Sorry I missed this before. > > diff --git a/t/README b/t/README > index b906ceb..f81998b 100644 > --- a/t/README > +++ b/t/README > @@ -259,11 +259,11 @@ Do: > test ... > > That way all of the commands in your tests will succeed or fail. If > - you must ignore the return value of something (e.g. the return > - value of export is unportable) it's best to indicate so explicitly > - with a semicolon: > + you must ignore the return value of something (e.g., the return > + after unsetting a variable that was already unset is unportable) it's > + best to indicate so explicitly with a semicolon: We should have examples for both export and unset, but the prose should mention both IMO -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html