HomePhabricator
Diffusion Comm 4ff2d64d1f10

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

Description

[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:
ShellCheck

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.

Reviewers: atul, jon, tomek, ashoat

Reviewed By: atul, jon

Subscribers: adrian, karol

Differential Revision: https://phab.comm.dev/D4720

Details

Provenance
aboshAuthored on Aug 2 2022, 10:02 AM
Reviewer
atul
Differential Revision
D4720: [services] Clean up `build_sources.sh` using ShellCheck
Parents
rCOMM1e62e4dff75c: [services] Clean up `build_service.sh` using ShellCheck
Branches
Unknown
Tags
Unknown