Fix apply configuration generation on macOS #248
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I noticed this while looking into #240
Commit 1
On at least macos 14.1 (sonoma) the paste utility seems to require a filename to be specified; before this change, the
API_DIRS
variable fails with the following error:Commit 2
This avoids us spinning up a separate shell when evaluating
API_DIRS
, sinceGO_MODULE
already contains the value we want.This change also changes the variable assignments; make variables are very confusing, but
:=
should only evaluate at startup while=
will evaluate every time the variable is used.See these docs for how
=
works, and these for:=
. From the first link: