All of lore.kernel.org
 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.