Re: [PATCH 1/5] Tests defs: don't let useless variables leak in test scripts.

2011-01-15 Thread Ralf Wildenhues
Hi Stefano, and apologies for the long delay,

there were three patches in this series that haven't been reviewed yet:

* Stefano Lattarini wrote on Mon, Nov 15, 2010 at 06:21:16PM CET:
 * tests/defs ($priv_check_temp, $overwrite_status, $ro_dir_temp,
 $create_status, $r2h, $libtool_found, $gettext_found, $aclocaldir,
 $extra_includes): Unset once they've served their purpose.

This is OK; all of the variables are set in all code paths.

Thanks,
Ralf



Re: [PATCH 1/5] Tests defs: don't let useless variables leak in test scripts.

2011-01-15 Thread Stefano Lattarini
On Saturday 15 January 2011, Ralf Wildenhues wrote:
 Hi Stefano, and apologies for the long delay,
 
 there were three patches in this series that haven't been reviewed yet:
 
 * Stefano Lattarini wrote on Mon, Nov 15, 2010 at 06:21:16PM CET:
  * tests/defs ($priv_check_temp, $overwrite_status, $ro_dir_temp,
  $create_status, $r2h, $libtool_found, $gettext_found, $aclocaldir,
  $extra_includes): Unset once they've served their purpose.
 
 This is OK; all of the variables are set in all code paths.
 
I've applied this (fixing the date in the ChangeLog entry, and bumping
copyright years in tests/defs), and pushed to the 'tests-init' branch.

Thanks,
  Stefano