[U-Boot] [PATCH] checkpatch.pl: Do not hardcode perl path

[Joel A Fernandes]

On Sunday, June 16, 2013, Jagannadha Sutradharudu Teki wrote:

> From: Jagannadha Sutradharudu Teki <jagannadh.teki at gmail.com<javascript:;>
> >
>
> checkpatch.pl requires perl v5.10.0 to run but it
> doesn't require to place in /usr/bin/perl
> Use env to ensure that the interpreter used is the
> first one on environment's $PATH on system with
> several versions of perl installed.
>
> Signed-off-by: Jagannadha Sutradharudu Teki <jagannadh.teki at gmail.com<javascript:;>
> >
> ---
>  tools/checkpatch.pl | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/checkpatch.pl b/tools/checkpatch.pl
> index 896e2bc..8dc5b9c 100755
> --- a/tools/checkpatch.pl
> +++ b/tools/checkpatch.pl
> @@ -1,10 +1,11 @@
> -#!/usr/bin/perl -w
> +#!/usr/bin/env perl


Would it not work to pass in the -w here?

Joel


>  # (c) 2001, Dave Jones. (the file handling bit)
>  # (c) 2005, Joel Schopp <jschopp at austin.ibm.com <javascript:;>> (the
> ugly bit)
>  # (c) 2007,2008, Andy Whitcroft <apw at uk.ibm.com <javascript:;>> (new
> conditions, test suite)
>  # (c) 2008-2010 Andy Whitcroft <apw at canonical.com <javascript:;>>
>  # Licensed under the terms of the GNU GPL License version 2
>
> +use warnings;
>  use strict;
>
>  my $P = $0;
> --
> 1.8.3
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de <javascript:;>
> http://lists.denx.de/mailman/listinfo/u-boot
>