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

Add support for overriding built-in bind modules #1619

Conversation

Pantsworth
Copy link
Contributor

@Pantsworth Pantsworth commented Jan 17, 2024

Fixes #1557

This PR allows modules defined in config.bind_module_path to override builtin bind modules (and adds a test to verify the behavior).

This enables users to customize how Rez creates bind packages (like os and arch) to match their needs.

@Pantsworth Pantsworth requested a review from a team as a code owner January 17, 2024 21:22
Copy link

linux-foundation-easycla bot commented Jan 17, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: Pantsworth / name: Michael Nowakowski (980eab9)
  • ✅ login: JeanChristopheMorinPerso / name: Jean-Christophe Morin (0fd6818, 88bcf9d)

…lt-in bind modules.

Signed-off-by: Michael Nowakowski <[email protected]>
@Pantsworth Pantsworth marked this pull request as draft January 17, 2024 21:52
@Pantsworth Pantsworth force-pushed the issue_1557-package-bind-override branch from fd9242b to 980eab9 Compare January 17, 2024 22:32
@Pantsworth Pantsworth marked this pull request as ready for review January 17, 2024 22:37
@JeanChristopheMorinPerso JeanChristopheMorinPerso changed the title Issue #1557 - Add support for overriding built-in bind modules Add support for overriding built-in bind modules Jan 27, 2024
Signed-off-by: Jean-Christophe Morin <[email protected]>
Signed-off-by: Jean-Christophe Morin <[email protected]>
Copy link
Member

@JeanChristopheMorinPerso JeanChristopheMorinPerso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again!

@JeanChristopheMorinPerso JeanChristopheMorinPerso merged commit 082ce72 into AcademySoftwareFoundation:main Jan 27, 2024
55 checks passed
@JeanChristopheMorinPerso JeanChristopheMorinPerso added this to the 3.0.0 milestone Jan 27, 2024
Pixel-Minions pushed a commit to Pixel-Minions/rez that referenced this pull request Feb 14, 2024
…dation#1619)

* Issue AcademySoftwareFoundation#1557 - Add support for overriding built-in bind modules.

Signed-off-by: Michael Nowakowski <[email protected]>

* Add copyrights

Signed-off-by: Jean-Christophe Morin <[email protected]>

* Add test for get_bind_modules

Signed-off-by: Jean-Christophe Morin <[email protected]>

---------

Signed-off-by: Michael Nowakowski <[email protected]>
Signed-off-by: Jean-Christophe Morin <[email protected]>
Co-authored-by: Jean-Christophe Morin <[email protected]>
Signed-off-by: Jose Enriquez <[email protected]>
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

Successfully merging this pull request may close these issues.

Rez-Bind: Override Built-in Bind Modules
2 participants