search

cps-org / cps

Common Package Specification — A cross-tool mechanism for locating software dependencies
https://cps-org.github.io/cps/
Other
99 stars 8 forks source link

Add Runtime-Environment #17

Open mwoehlke opened 1 year ago

mwoehlke commented 1 year ago

Add an attribute specifying environment variables to set when executing a binary that uses a component. This is useful as components that do not live in system default search paths may need to supplement PATH, LD_LIBRARY_PATH or similar in order to be found at execution time.

COMMENTS REQUESTED

This is being submitted as a pull request rather than simply merged in order to solicit community feedback to the proposed change. If you have suggestions or concerns regarding this proposed change, please leave a comment!

bretbrownjr commented 5 months ago

@dcbaker @mwoehlke Just noticed this is a PR that has been hanging around for a while. What's the next step here?

mwoehlke commented 5 months ago

@bretbrownjr, mostly it's looking for buy-in; is this something we want in the spec, especially as it's likely not 1.0 MVP? Or should we leave it on hold?

If we want to work on it, I can rebase it and dig into Dylan's comments. Last I'd heard, however, there seemed to be some agreement to back-burner it.

bretbrownjr commented 5 months ago

It makes sense as a feature.

I personally would consider it not essential to an initial release, but I'm happy to be educated otherwise.