git-arc: Fix fetching of reviewers
AbandonedPublic
Actions

Authored by markj on Dec 4 2024, 1:18 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Nov 8, 2:02 AM

	Unknown Object (File)
	Fri, Nov 7, 7:17 AM

	Unknown Object (File)
	Wed, Nov 5, 10:48 AM

	Unknown Object (File)
	Sep 30 2025, 8:58 AM

	Unknown Object (File)
	Sep 25 2025, 2:26 AM

	Unknown Object (File)
	Sep 8 2025, 7:31 PM

	Unknown Object (File)
	Sep 6 2025, 7:02 PM

	Unknown Object (File)
	Aug 29 2025, 3:14 AM

View All 33 Files

Subscribers

imp

Details

Reviewers

jlduran
des

Summary

Let the shell normalize whitespace (i.e., convert newlines to spaces).
Otherwise we end up generating invalid JSON.

Fixes: 9e84289f2c7e ("git-arc: Prefer echo over printf")

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 60940
Build 57824: arc lint + arc unit

Event Timeline

markj created this revision.Dec 4 2024, 1:18 AM

Herald added a subscriber: imp. · View Herald TranscriptDec 4 2024, 1:18 AM

markj requested review of this revision.Dec 4 2024, 1:18 AM

Harbormaster completed remote builds in B60940: Diff 147436.Dec 4 2024, 1:18 AM

Is this the same as D47889?

This revision is now accepted and ready to land.Dec 4 2024, 1:38 AM

In D47893#1092074, @jlduran wrote:

Is this the same as D47889?

Yep, Gleb beat me to it. :)

Revision Contents
Changeset List

Path

Size

tools/

git/

git-arc.sh

2 lines

Diff 147436

View Options

tools/tools/git/git-arc.sh

git-arc: Fix fetching of reviewersAbandonedPublicActions