Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rez variant environment var during build #304

Closed
maxnbk opened this issue Mar 3, 2016 · 1 comment
Closed

rez variant environment var during build #304

maxnbk opened this issue Mar 3, 2016 · 1 comment

Comments

@maxnbk
Copy link
Contributor

maxnbk commented Mar 3, 2016

Multi-issue Issue, but all in the same ballpark so keeping collected in one item.

First, introduce REZ_BUILD_VARIANT env var during rez-build process which is the value of the REZ_VARIANT_INDEX -ed variant string (so for variant ['platform-linux','arch-x86_64','os-Ubuntu16'], give back "platform-linux,arch-x86_64,os-Ubuntu16".

Second, Additional REZ_VARIANT_PATH which is just the same thing, with slashes instead, to be modified in next item.

Third, add ability to specify custom rez-variant paths as dictionary key in variant list. This would override #2 item variant path var appropriately, while leaving build variant var as the key.

@nerdvegas
Copy link
Contributor

REZ_BUILD_VARIANT_REQUIRES was added in https://github.com/nerdvegas/rez/releases/tag/2.25.0

Will add REZ_BUILD_VARIANT_SUBPATH in upcoming release.

Dict-based variants are a heavy enough feature that they deserve a separate ticket. Note however that a coming feature (hashed variants) will alleviate the need for this to a fair degree.

Will close this ticket with next release as it addresses these points.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants