Update Process.dependencies
to allow more flexibility when setting envs
#124
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.
This PR updates how the
dependencies
field is use when aprocess
recipe is baked. Each dependency recipe can contain abrioche-env.d/env
directory to set env vars on the process, but previously this was limited to appending$PATH
-like env vars: each entry inbrioche-env.d/env
had to be a directory (whose name was an env var), where each entry within was a symlink, whose target would be appended to the env var (separated by:
).This PR lifts the restriction, and now each env var within
brioche-env.d/env
can either be a file, directory, or symlink:So, this is now a valid structure for a dependency:
When used as a dependency, it will append
bin
to$PATH
, set$CC
to the path tobin/cc
(if unset), and set$DEBUG
to1
(if unset)