git-arc: Quote echoed variables
AbandonedPublic
Actions

Authored by 0mp on Oct 24 2024, 8:42 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, May 16, 3:58 AM

	Unknown Object (File)
	Tue, Apr 28, 5:56 AM

	Unknown Object (File)
	Apr 27 2026, 5:11 AM

	Unknown Object (File)
	Apr 22 2026, 12:38 PM

	Unknown Object (File)
	Apr 21 2026, 6:29 PM

	Unknown Object (File)
	Apr 21 2026, 11:14 AM

	Unknown Object (File)
	Apr 19 2026, 7:35 PM

	Unknown Object (File)
	Apr 19 2026, 6:32 PM

View All 56 Files

Subscribers

imp

jlduran

Details

Reviewers

bnovkov
christos
markj

Summary

This is a preparation for an eventual transition to printfs. Let's make
sure that we do not depend on globbing and word splitting.

MFC after: 2 weeks

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 60180
Build 57064: arc lint + arc unit

Event Timeline

0mp created this revision.Oct 24 2024, 8:42 AM

Herald added a subscriber: imp. · View Herald TranscriptOct 24 2024, 8:42 AM

0mp requested review of this revision.Oct 24 2024, 8:42 AM

Harbormaster completed remote builds in B60180: Diff 145415.Oct 24 2024, 8:42 AM

0mp added a child revision: D47274: git-arc: Use printf instead of echo.Oct 24 2024, 8:43 AM

0mp added reviewers: bnovkov, christos, markj.

christos accepted this revision.Oct 24 2024, 10:40 AM

This revision is now accepted and ready to land.Oct 24 2024, 10:40 AM

Seems fine, but it's gratuitous in my opinion. This could be done at the same time as converting to printf.

Considering that git-arc is now more portable thanks to the patches by @jlduran_gmail.com, I'm closing this revision. If we decide to start quoting variables in git-arc for the sake of defensive programming, we should review the whole file instead of just the echo cases.

Revision Contents
Changeset List

Path

Size

tools/

git/

git-arc.sh

8 lines

Diff 145415

View Options

tools/tools/git/git-arc.sh

git-arc: Quote echoed variablesAbandonedPublicActions