reports.mk: streamline basedir(1) equivalent

That "clever hack" with two rev(1) invocations seems pretty stupid to me: too much processes and processing instead of simple prefix removal. One can also have that as ${var##*/} when dealing with a shell variable and avoiding extra forks, BTW.

reports.mk: streamline basedir(1) equivalent
4c701def · Michael Shigorin · Anton Midyukov · 4a419e6b · 4c701def
Commit 4c701def authored Apr 28, 2021 by Michael Shigorin Committed by Anton Midyukov Apr 29, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 2 deletions

reports.mk reports.mk +1 -2

No files found.
--- a/reports.mk
+++ b/reports.mk
@@ -79,8 +79,7 @@ reports/contents: reports/prep

 reports/packages: reports/prep
 	@grep -E 'chroot/.in/[^/]*.rpm' < $(BUILDLOG) | \
-		cut -d' ' -f 1 | tr -d "'"'`' | \
-		rev | cut -d'/' -f 1 | rev | \
+		cut -d' ' -f 1 | tr -d "'"'`' | sed 's,^.*/,,' | \
 		sort -u > "$(REPORTDIR)/list-rpms.txt"
 	@grep -E 'chroot/.in/[^/]*.rpm' < $(BUILDLOG) | \
 		cut -d' ' -f 1 | tr -d "'"'`' | \