On Sat, Aug 14, 2010 at 4:53 AM, Eric Li(李咏竹) <[email protected]> wrote:
> Thanks for the quick response. Please take another look.
>
> Index: client/common_lib/utils.py
> ===================================================================
> --- client/common_lib/utils.py (revision 4747)
> +++ client/common_lib/utils.py (working copy)
> @@ -1165,6 +1165,16 @@
>      system('%s %s' % (configure, ' '.join(args)))
>
>
> +def make(extra='', make='make', timeout=None, ignore_status=False):
> +    """
> +    Run make, adding MAKEOPTS to the list of options.
> +
> +   �...@param extra: extra command line arguments to pass to make.
> +    """
> +    cmd = '%s %s %s' % (make, os.environ.get('MAKEOPTS', ''), extra)
> +    return system(cmd, timeout=timeout, ignore_status=ignore_status)
> +
> +
>  def compare_versions(ver1, ver2):
>      """Version number comparison between ver1 and ver2 strings.

Looks good for me.
ACK
_______________________________________________
Autotest mailing list
[email protected]
http://test.kernel.org/cgi-bin/mailman/listinfo/autotest

Reply via email to