Paths

Table of Contentst

Differential D4720

[services] Clean up `build_sources.sh` using ShellCheck
ClosedPublic
Actions

Authored by • abosh on Aug 2 2022, 10:02 AM.

Tags

None

Referenced Files

	F3398111: D4720.diff
	Sun, Dec 1, 9:04 PM

	Unknown Object (File)
	Sat, Nov 9, 11:38 AM

	Unknown Object (File)
	Fri, Nov 8, 8:36 PM

	Unknown Object (File)
	Fri, Nov 8, 8:36 PM

	Unknown Object (File)
	Tue, Nov 5, 2:06 AM

	Unknown Object (File)
	Oct 5 2024, 3:07 AM

	Unknown Object (File)
	Sep 13 2024, 1:45 AM

	Unknown Object (File)
	Sep 12 2024, 11:47 AM

View All 67 Files

Subscribers

Details

Reviewers

atul
• jon
tomek
ashoat

Commits

rCOMM4ff2d64d1f10: [services] Clean up `build_sources.sh` using ShellCheck

Summary

Related Linear issue here. This is part of a set of diffs that will allow ShellCheck to be added to the CI. See inline comments for specific details of the ShellCheck error/warning output.

Test Plan

Additionally, this script is run by build_service.sh, which is run during the Backup, Blob, and Tunnelbroker builds so if those workflows continue to pass CI we can reasonably assume that the script continues to work as expected.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• abosh created this revision.Aug 2 2022, 10:02 AM

Herald added subscribers: • karol, • adrian. · View Herald TranscriptAug 2 2022, 10:02 AM

• abosh added inline comments.Aug 2 2022, 10:03 AM

services/lib/docker/build_sources.sh
16 ↗	(On Diff #15228)	Double quote to prevent globbing and word splitting.

Harbormaster completed remote builds in B11063: Diff 15228.Aug 2 2022, 10:15 AM

• abosh requested review of this revision.Aug 2 2022, 10:15 AM

• abosh edited the test plan for this revision. (Show Details)Aug 2 2022, 10:51 AM

I don't have much context on this script I think, and I'm a bad choice for a first-pass reviewer except for cases listed here

• jon accepted this revision.Aug 2 2022, 11:54 AM

This revision is now accepted and ready to land.Aug 2 2022, 11:54 AM

atul accepted this revision.Aug 2 2022, 11:58 AM

Closed by commit rCOMM4ff2d64d1f10: [services] Clean up `build_sources.sh` using ShellCheck (authored by • abosh). · Explain WhyAug 2 2022, 2:23 PM

This revision was automatically updated to reflect the committed changes.

• abosh added a commit: rCOMM4ff2d64d1f10: [services] Clean up `build_sources.sh` using ShellCheck.

Revision Contents
Changeset List

Path

Size

services/

lib/

docker/

build_sources.sh

2 lines

Diff 15247

services/lib/docker/build_sources.sh

Loading...