public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Cyril Hrubis <chrubis@suse.cz>
To: Petr Vorel <pvorel@suse.cz>
Cc: ltp@lists.linux.it
Subject: Re: [LTP] [PATCH 1/1] file01.sh: Drop csh and ksh detection
Date: Fri, 24 Mar 2023 10:29:10 +0100	[thread overview]
Message-ID: <ZB1tZkIeqbDN9cz5@yuki> (raw)
In-Reply-To: <20230324081744.539431-1-pvorel@suse.cz>

Hi!
Maybe we should remove the whole file tests, but this is the least
controversial change.

Reviewed-by: Cyril Hrubis <chrubis@suse.cz>

> Reported-by: Cyril Hrubis <chrubis@suse.cz>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
>  testcases/commands/file/datafiles/in.csh |  4 ---
>  testcases/commands/file/datafiles/in.ksh |  4 ---
>  testcases/commands/file/file01.sh        | 35 ++++++++++++------------
>  3 files changed, 17 insertions(+), 26 deletions(-)
>  delete mode 100644 testcases/commands/file/datafiles/in.csh
>  delete mode 100644 testcases/commands/file/datafiles/in.ksh
> 
> diff --git a/testcases/commands/file/datafiles/in.csh b/testcases/commands/file/datafiles/in.csh
> deleted file mode 100644
> index c475aef9c..000000000
> --- a/testcases/commands/file/datafiles/in.csh
> +++ /dev/null
> @@ -1,4 +0,0 @@
> -#! /bin/csh
> -
> -echo "this is a shell script"
> -echo "used to test file command"
> diff --git a/testcases/commands/file/datafiles/in.ksh b/testcases/commands/file/datafiles/in.ksh
> deleted file mode 100644
> index 3ae45125c..000000000
> --- a/testcases/commands/file/datafiles/in.ksh
> +++ /dev/null
> @@ -1,4 +0,0 @@
> -#! /bin/ksh
> -
> -echo "this is a shell script"
> -echo "used to test file command"
> diff --git a/testcases/commands/file/file01.sh b/testcases/commands/file/file01.sh
> index df152b656..f0c129f33 100755
> --- a/testcases/commands/file/file01.sh
> +++ b/testcases/commands/file/file01.sh
> @@ -2,12 +2,13 @@
>  # SPDX-License-Identifier: GPL-2.0-or-later
>  # Copyright (c) International Business Machines Corp., 2001
>  # Copyright (c) 2016 Cyril Hrubis <chrubis@suse.cz>
> +# Copyright (c) Linux Test Project, 2017-2023
>  #
>  # This program tests the file command. The tests are aimed at
>  # testing if the file command can recognize some of the commonly
>  # used file formats like, tar, tar.gz, rpm, C, ASCII, ELF etc.
>  
> -TST_CNT=20
> +TST_CNT=18
>  TST_SETUP=setup
>  TST_TESTFUNC=do_test
>  TST_NEEDS_TMPDIR=1
> @@ -61,30 +62,28 @@ do_test()
>  			    "POSIX shell script text executable" \
>  			    "POSIX shell script text" \
>  			    "Bourne shell script text executable";;
> -	 4) file_test in.ksh "Korn shell script";;
> -	 5) file_test in.csh "C shell script";;
> -	 6) file_test in.c "ASCII C program text" "C source, ASCII text";;
> -	 7) file_test in.pl "[pP]erl script, ASCII text executable" \
> +	 4) file_test in.c "ASCII C program text" "C source, ASCII text";;
> +	 5) file_test in.pl "[pP]erl script, ASCII text executable" \
>  			    "[pP]erl script text executable" \
>  			    "a /usr/bin/perl script text";;
> -	 8) file_test in.py "[pP]ython3\{0,1\} script, ASCII text executable" \
> +	 6) file_test in.py "[pP]ython3\{0,1\} script, ASCII text executable" \
>  			    "[pP]ython3\{0,1\} script text executable";;
> -	 9) file_test in.m4 "M4 macro processor script, ASCII text" \
> +	 7) file_test in.m4 "M4 macro processor script, ASCII text" \
>  			    "ASCII M4 macro language pre-processor text";;
> -	10) file_test in "ELF .*-bit $TEST_ARCH executable, .*" \
> +	 8) file_test in "ELF .*-bit $TEST_ARCH executable, .*" \
>  			 "ELF .*-bit $TEST_ARCH shared object, .*" \
>  			 "ELF .*-bit $TEST_ARCH pie executable, .*" \
>  			 "ELF .*-bit $TEST_ARCH pie shared object, .*";;
> -	11) file_test in.ar "current ar archive";;
> -	12) file_test in.tar "tar archive";;
> -	13) file_test in.tar.gz "gzip compressed data, .*";;
> -	14) file_test in.tar.bz2 "bzip2 compressed data, .*";;
> -	15) file_test in.src.rpm "RPM v3 src" "RPM v3.0 src";;
> -	16) file_test in.jpg "JPEG image data";;
> -	17) file_test in.png "PNG image data";;
> -	18) file_test in.wav "RIFF (little-endian) data, WAVE audio, Microsoft PCM";;
> -	19) file_test in.mp3 "MPEG ADTS, layer III";;
> -	20) file_test in.zip "Zip archive data";;
> +	 9) file_test in.ar "current ar archive";;
> +	10) file_test in.tar "tar archive";;
> +	11) file_test in.tar.gz "gzip compressed data, .*";;
> +	12) file_test in.tar.bz2 "bzip2 compressed data, .*";;
> +	13) file_test in.src.rpm "RPM v3 src" "RPM v3.0 src";;
> +	14) file_test in.jpg "JPEG image data";;
> +	15) file_test in.png "PNG image data";;
> +	16) file_test in.wav "RIFF (little-endian) data, WAVE audio, Microsoft PCM";;
> +	17) file_test in.mp3 "MPEG ADTS, layer III";;
> +	18) file_test in.zip "Zip archive data";;
>  	esac
>  }
>  
> -- 
> 2.40.0
> 

-- 
Cyril Hrubis
chrubis@suse.cz

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

  reply	other threads:[~2023-03-24  9:28 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-24  8:17 [LTP] [PATCH 1/1] file01.sh: Drop csh and ksh detection Petr Vorel
2023-03-24  9:29 ` Cyril Hrubis [this message]
2023-03-24  9:36   ` Petr Vorel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZB1tZkIeqbDN9cz5@yuki \
    --to=chrubis@suse.cz \
    --cc=ltp@lists.linux.it \
    --cc=pvorel@suse.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox