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
next prev parent 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