Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Subject: Re: [PATCH] ktest: Fix typos in ktest.pl
To:     Masanari Iida <standby24x7@gmail.com>,
        linux-kernel@vger.kernel.org, rostedt@goodmis.org
References: <20200309115430.57540-1-standby24x7@gmail.com>
From:   Randy Dunlap <rdunlap@infradead.org>
Message-ID: <097cdf11-33b5-45be-6a41-0e5354276bad@infradead.org>
Date:   Mon, 9 Mar 2020 07:59:52 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.5.0
MIME-Version: 1.0
In-Reply-To: <20200309115430.57540-1-standby24x7@gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 3/9/20 4:54 AM, Masanari Iida wrote:
> This patch fixes multipe spelling typo found in ktest.pl.
> 
> Signed-off-by: Masanari Iida <standby24x7@gmail.com>

Looks good.  Thanks.

Acked-by: Randy Dunlap <rdunlap@infradead.org>

> ---
>  tools/testing/ktest/ktest.pl | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/tools/testing/ktest/ktest.pl b/tools/testing/ktest/ktest.pl
> index 6a605ba75dd6..dab03d80c902 100755
> --- a/tools/testing/ktest/ktest.pl
> +++ b/tools/testing/ktest/ktest.pl
> @@ -1030,7 +1030,7 @@ sub __read_config {
>  	    }
>  
>  	    if (!$skip && $rest !~ /^\s*$/) {
> -		die "$name: $.: Gargbage found after $type\n$_";
> +		die "$name: $.: Garbage found after $type\n$_";
>  	    }
>  
>  	    if ($skip && $type eq "TEST_START") {
> @@ -1063,7 +1063,7 @@ sub __read_config {
>  	    }
>  
>  	    if ($rest !~ /^\s*$/) {
> -		die "$name: $.: Gargbage found after DEFAULTS\n$_";
> +		die "$name: $.: Garbage found after DEFAULTS\n$_";
>  	    }
>  
>  	} elsif (/^\s*INCLUDE\s+(\S+)/) {
> @@ -1154,7 +1154,7 @@ sub __read_config {
>  	    # on of these sections that have SKIP defined.
>  	    # The save variable can be
>  	    # defined multiple times and the new one simply overrides
> -	    # the prevous one.
> +	    # the previous one.
>  	    set_variable($lvalue, $rvalue);
>  
>  	} else {
> @@ -1234,7 +1234,7 @@ sub read_config {
>  	foreach my $option (keys %not_used) {
>  	    print "$option\n";
>  	}
> -	print "Set IGRNORE_UNUSED = 1 to have ktest ignore unused variables\n";
> +	print "Set IGNORE_UNUSED = 1 to have ktest ignore unused variables\n";
>  	if (!read_yn "Do you want to continue?") {
>  	    exit -1;
>  	}
> @@ -1345,7 +1345,7 @@ sub eval_option {
>  	# Check for recursive evaluations.
>  	# 100 deep should be more than enough.
>  	if ($r++ > 100) {
> -	    die "Over 100 evaluations accurred with $option\n" .
> +	    die "Over 100 evaluations occurred with $option\n" .
>  		"Check for recursive variables\n";
>  	}
>  	$prev = $option;
> @@ -1461,7 +1461,7 @@ sub get_test_name() {
>  
>  sub dodie {
>  
> -    # avoid recusion
> +    # avoid recursion
>      return if ($in_die);
>      $in_die = 1;
>  
> 


-- 
~Randy