git: 793745fad866 - main - mfc-candidates: move pretty printing into lua

From: Ed Maste <emaste_at_FreeBSD.org>
Date: Sun, 21 Apr 2024 14:11:48 UTC
The branch main has been updated by emaste:

URL: https://cgit.FreeBSD.org/src/commit/?id=793745fad866f65863ab3b16a17bdf18dc779efd

commit 793745fad866f65863ab3b16a17bdf18dc779efd
Author:     Ed Maste <emaste@FreeBSD.org>
AuthorDate: 2024-04-16 16:49:05 +0000
Commit:     Ed Maste <emaste@FreeBSD.org>
CommitDate: 2024-04-21 14:10:57 +0000

    mfc-candidates: move pretty printing into lua
    
    d51c59002367 moved the MFC hash matching logic into a lua utility
    script but left the output formatting in the shell script. Simplify this
    slightly by just printing the formatted output from lua.
    
    Sponsored by:   The FreeBSD Foundation
    Differential Revision: https://reviews.freebsd.org/D44836
---
 tools/tools/git/candidatematch.lua | 10 +++++++++-
 tools/tools/git/mfc-candidates.sh  |  7 +------
 2 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/tools/tools/git/candidatematch.lua b/tools/tools/git/candidatematch.lua
index 98c247fca339..481c1f38fea1 100755
--- a/tools/tools/git/candidatematch.lua
+++ b/tools/tools/git/candidatematch.lua
@@ -36,6 +36,14 @@ local function set_difference(set1, set2)
 	return result
 end
 
+-- Execute a command and print to stdout
+local function exec_command(command)
+	local handle = io.popen(command)
+	local output = handle:read("a")
+	handle:close()
+	io.write(output)
+end
+
 -- Main function
 local function main()
 	local from_file = arg[1]
@@ -59,7 +67,7 @@ local function main()
 
 	-- Print the result
 	for _, hash in ipairs(result_hashes) do
-		print(hash)
+		exec_command("git show --pretty='%h %s' --no-patch " .. hash)
 	end
 end
 
diff --git a/tools/tools/git/mfc-candidates.sh b/tools/tools/git/mfc-candidates.sh
index 0787e1278991..cf71c314cb95 100644
--- a/tools/tools/git/mfc-candidates.sh
+++ b/tools/tools/git/mfc-candidates.sh
@@ -152,7 +152,6 @@ canonicalize_hashes()
 workdir=$(mktemp -d /tmp/find-mfc.XXXXXXXXXX)
 from_list=$workdir/commits-from
 to_list=$workdir/commits-to
-candidate_list=$workdir/candidates
 
 if [ -n "$exclude_file" ]; then
 	exclude_list=$workdir/commits-exclude
@@ -163,10 +162,6 @@ commits_from "$@" > $from_list
 commits_to "$@" > $to_list
 
 /usr/libexec/flua $(dirname $0)/candidatematch.lua \
-    $from_list $to_list $exclude_list > $candidate_list
-
-while read hash; do
-	git show --pretty='%h %s' --no-patch $hash
-done < $candidate_list
+    $from_list $to_list $exclude_list
 
 rm -rf "$workdir"