From: Eric Biggers <ebiggers@kernel.org>
To: Theodore Ts'o <tytso@mit.edu>
Cc: fstests@vger.kernel.org
Subject: [xfstests-bld PATCH] Use grep -E instead of egrep
Date: Tue, 18 Oct 2022 11:16:26 -0700 [thread overview]
Message-ID: <20221018181626.154383-1-ebiggers@kernel.org> (raw)
From: Eric Biggers <ebiggers@google.com>
Avoid some of the "egrep is obsolescent" warnings that the latest
version of GNU grep prints.
Signed-off-by: Eric Biggers <ebiggers@google.com>
---
run-fstests/get-results | 8 ++++----
setup-buildchroot | 2 +-
test-appliance/android-setup-partitions | 10 +++++-----
test-appliance/files/usr/local/sbin/gce-shutdown | 16 ++++++++--------
test-appliance/files/usr/local/sbin/get-results | 8 ++++----
5 files changed, 22 insertions(+), 22 deletions(-)
diff --git a/run-fstests/get-results b/run-fstests/get-results
index ebaf701..d899c33 100755
--- a/run-fstests/get-results
+++ b/run-fstests/get-results
@@ -21,12 +21,12 @@ then
IN=$(/bin/ls -t logs/log.* | head -1)
fi
-egrep "$REGEXP" "$IN"
+grep -E "$REGEXP" "$IN"
if test -n "$do_failure"; then
- b=$(egrep ^BEGIN "$IN" | wc -l)
- e=$(egrep ^END "$IN" | wc -l)
+ b=$(grep -E ^BEGIN "$IN" | wc -l)
+ e=$(grep -E ^END "$IN" | wc -l)
if test "$b" -gt "$e" ; then
- l=$(egrep ^BEGIN "$IN" | tail -n 1 | sed -e 's/BEGIN //')
+ l=$(grep -E ^BEGIN "$IN" | tail -n 1 | sed -e 's/BEGIN //')
echo " "
echo Missing END "$l"
fi
diff --git a/setup-buildchroot b/setup-buildchroot
index 6cf14b3..ff2b01f 100755
--- a/setup-buildchroot
+++ b/setup-buildchroot
@@ -421,7 +421,7 @@ get_schroot_config()
then
(cd $SCHROOT_CHROOT_D ;
find . -maxdepth 1 -type f |
- egrep '^./[a-ZA-Z0-9_-][a-ZA-Z0-9_.-]*$' |
+ grep -E '^./[a-ZA-Z0-9_-][a-ZA-Z0-9_.-]*$' |
xargs cat)
fi
}
diff --git a/test-appliance/android-setup-partitions b/test-appliance/android-setup-partitions
index 2eb5fca..1f4bb7e 100755
--- a/test-appliance/android-setup-partitions
+++ b/test-appliance/android-setup-partitions
@@ -57,7 +57,7 @@ pprint_bytes()
get_partition_number()
{
local dev=$1
- echo $dev | egrep -o '[0-9]+$'
+ echo $dev | grep -E -o '[0-9]+$'
}
# Get the size in bytes of the given partition, as stored in the partition table
@@ -108,7 +108,7 @@ find_userdata_partition()
if [ ! -b $dev ]; then
die "Unable to find the userdata partition"
fi
- if ! echo $dev | egrep -q '[0-9]+$'; then
+ if ! echo $dev | grep -E -q '[0-9]+$'; then
die "Name of userdata device node has an unexpected format: \"$dev\""
fi
echo $dev
@@ -118,7 +118,7 @@ find_userdata_partition()
find_dm_device_by_name()
{
local dm_device_name=$1
- if ls /sys/class/block/ | egrep -q 'dm-[0-9]+$'; then
+ if ls /sys/class/block/ | grep -E -q 'dm-[0-9]+$'; then
for dir in /sys/class/block/dm-*; do
if [ $dm_device_name = $(< $dir/dm/name) ]; then
local dev=/dev/block/$(basename $dir)
@@ -173,8 +173,8 @@ validate_dm_device()
;;
esac
- if ! echo "$start_sector" | egrep -q '^[0-9]+$' ||
- ! echo "$raw_devno" | egrep -q '^[0-9]+:[0-9]+$'; then
+ if ! echo "$start_sector" | grep -E -q '^[0-9]+$' ||
+ ! echo "$raw_devno" | grep -E -q '^[0-9]+:[0-9]+$'; then
die "device-mapper device \"$dm_devname\" uses target with" \
"unsupported table format: ${table[@]}"
fi
diff --git a/test-appliance/files/usr/local/sbin/gce-shutdown b/test-appliance/files/usr/local/sbin/gce-shutdown
index 9ebe008..0ca1d32 100755
--- a/test-appliance/files/usr/local/sbin/gce-shutdown
+++ b/test-appliance/files/usr/local/sbin/gce-shutdown
@@ -24,13 +24,13 @@ REGEXP_FAILURE="(^CMDLINE)|(^FSTEST)|(^MNTOPTS)|(^CPUS:)|(^MEM:)|(^BEGIN)|(^Fail
function gen_xfstests_summary()
{
- egrep "$REGEXP" < /results/runtests.log > /results/summary
+ grep -E "$REGEXP" < /results/runtests.log > /results/summary
- egrep "$REGEXP_FAILURE" < /results/runtests.log > /results/failures
+ grep -E "$REGEXP_FAILURE" < /results/runtests.log > /results/failures
printf "\n" >> /results/failures
sed -n -f /usr/local/lib/get-check-failures.sed < /results/runtests.log >> /results/failures
printf "\n" >> /results/failures
- egrep "$SYSLOG_RE_FAILURE" < /results/syslog >> /results/failures
+ grep -E "$SYSLOG_RE_FAILURE" < /results/syslog >> /results/failures
}
BLK_REGEXP="( Linux version )|(^CMDLINE)|(^FSTEST)|(^MNTOPTS)|(^CPUS:)|(^MEM:)|(^BEGIN)|(^MOUNT_OPTIONS)|(^MKFS_OPTIONS)|(^EXT4-fs error)|(WARNING)|(^_check_)|(inconsistent)|(^Shutdown reason)"
@@ -38,24 +38,24 @@ BLK_REGEXP_FAILURE="( Linux version )|(^CMDLINE)|(^FSTEST)|(^MNTOPTS)|(^CPUS:)|(
function gen_blktests_summary()
{
- run_tests=$(egrep "(\[failed\])|(\[passed\])" < /results/runtests.log | awk '{print $1}')
- failed_tests=$(egrep "\[failed\]" < /results/runtests.log | awk '{print $1}')
+ run_tests=$(grep -E "(\[failed\])|(\[passed\])" < /results/runtests.log | awk '{print $1}')
+ failed_tests=$(grep -E "\[failed\]" < /results/runtests.log | awk '{print $1}')
nr_run="$(echo $run_tests | wc -w)"
nr_failed="$(echo $failed_tests | wc -w)"
- egrep "$BLK_REGEXP" < /results/runtests.log > /results/summary
+ grep -E "$BLK_REGEXP" < /results/runtests.log > /results/summary
echo "Run: $(echo $run_tests)" >> /results/summary
echo "Failures: $(echo $failed_tests)" >> /results/summary
echo "Failed $nr_failed of $nr_run tests" >> /results/summary
grep ^END < /results/runtests.log >> /results/summary
- egrep "$BLK_REGEXP_FAILURE" < /results/runtests.log >> /results/failures
+ grep -E "$BLK_REGEXP_FAILURE" < /results/runtests.log >> /results/failures
echo "Run: $(echo $run_tests)" >> /results/failures
echo "Failures: $(echo $failed_tests)" >> /results/failures
echo "Failed $nr_failed of $nr_run tests" >> /results/failures
grep ^END < /results/runtests.log >> /results/failures
- egrep "$SYSLOG_RE_FAILURE" < /results/syslog >> /results/failures
+ grep -E "$SYSLOG_RE_FAILURE" < /results/syslog >> /results/failures
}
reason=$(gce_attribute shutdown_reason)
diff --git a/test-appliance/files/usr/local/sbin/get-results b/test-appliance/files/usr/local/sbin/get-results
index 7169b12..053c45c 100755
--- a/test-appliance/files/usr/local/sbin/get-results
+++ b/test-appliance/files/usr/local/sbin/get-results
@@ -21,12 +21,12 @@ then
IN=-
fi
-egrep "$REGEXP" "$IN"
+grep -E "$REGEXP" "$IN"
if test -n "$do_failure"; then
- b=$(egrep ^BEGIN "$IN" | wc -l)
- e=$(egrep ^END "$IN" | wc -l)
+ b=$(grep -E ^BEGIN "$IN" | wc -l)
+ e=$(grep -E ^END "$IN" | wc -l)
if test "$b" -gt "$e" ; then
- l=$(egrep ^BEGIN "$IN" | tail -n 1 | sed -e 's/BEGIN //')
+ l=$(grep -E ^BEGIN "$IN" | tail -n 1 | sed -e 's/BEGIN //')
echo " "
echo Missing END "$l"
fi
--
2.38.0
next reply other threads:[~2022-10-18 18:16 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-18 18:16 Eric Biggers [this message]
2022-10-19 2:28 ` [xfstests-bld PATCH] Use grep -E instead of egrep Theodore Ts'o
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=20221018181626.154383-1-ebiggers@kernel.org \
--to=ebiggers@kernel.org \
--cc=fstests@vger.kernel.org \
--cc=tytso@mit.edu \
/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