GNU bug report logs -
#60764
[PATCH] tests: change `sort|uniq` to `sort -u`
Previous Next
Reported by: Mike Frysinger <vapier <at> gentoo.org>
Date: Fri, 13 Jan 2023 01:31:02 UTC
Severity: normal
Tags: patch
Merged with 60763
Done: Mike Frysinger <vapier <at> gentoo.org>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 60764 in the body.
You can then email your comments to 60764 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
automake-patches <at> gnu.org
:
bug#60764
; Package
automake-patches
.
(Fri, 13 Jan 2023 01:31:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Mike Frysinger <vapier <at> gentoo.org>
:
New bug report received and forwarded. Copy sent to
automake-patches <at> gnu.org
.
(Fri, 13 Jan 2023 01:31:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
These are equivalent, but more performant. We already use `sort -u`
elsewhere in the codebase, so nothing new here really.
* t/add-missing.tap: Change `sort|uniq` to `sort -u`.
* t/CheckListOfTests.am: Likewise.
---
t/CheckListOfTests.am | 2 +-
t/add-missing.tap | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/t/CheckListOfTests.am b/t/CheckListOfTests.am
index e17083a62d47..672af700e88b 100644
--- a/t/CheckListOfTests.am
+++ b/t/CheckListOfTests.am
@@ -49,7 +49,7 @@ maintainer-check-list-of-tests:
($(am__cd) $(srcdir) && ls $$dir/*$$ext 2>/dev/null); \
fi; \
done; \
- done | sed 's,^\./,,' | sort | uniq >$(am__tfs); \
+ done | sed 's,^\./,,' | sort -u >$(am__tfs); \
## Compare the two lists, complain if they differ.
if $$diff $(am__tmk) $(am__tfs) >$(am__tdf); then \
result=0; \
diff --git a/t/add-missing.tap b/t/add-missing.tap
index 010c03ec5864..ea42bc9cda05 100644
--- a/t/add-missing.tap
+++ b/t/add-missing.tap
@@ -196,7 +196,7 @@ check_ ()
# Only the expected files should be installed. But automake always
# require 'missing' and 'install-sh', so account for them.
all_files="install-sh missing $files"
- for f in $all_files; do echo $f; done | sort | uniq > files.exp
+ for f in $all_files; do echo $f; done | sort -u > files.exp
(cd $build_aux && ls) | sort > files.got
cat files.exp
cat files.got
--
2.39.0
Merged 60763 60764.
Request was from
Mike Frysinger <vapier <at> gentoo.org>
to
control <at> debbugs.gnu.org
.
(Fri, 13 Jan 2023 04:08:02 GMT)
Full text and
rfc822 format available.
bug closed, send any further explanations to
60763 <at> debbugs.gnu.org and Mike Frysinger <vapier <at> gentoo.org>
Request was from
Mike Frysinger <vapier <at> gentoo.org>
to
control <at> debbugs.gnu.org
.
(Sat, 14 Jan 2023 00:08:03 GMT)
Full text and
rfc822 format available.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Sat, 11 Feb 2023 12:24:09 GMT)
Full text and
rfc822 format available.
This bug report was last modified 1 year and 67 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.