git-arc: Fix fetching of reviewers
AbandonedPublic
Actions

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

Tags

None

Referenced Files

	F144400260: D47893.diff
	Sun, Feb 8, 6:09 AM

	Unknown Object (File)
	Dec 21 2025, 10:52 PM

	Unknown Object (File)
	Nov 25 2025, 9:20 AM

	Unknown Object (File)
	Nov 23 2025, 11:37 PM

	Unknown Object (File)
	Nov 20 2025, 5:53 PM

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

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

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

View All 38 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