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-bind os requires arch #1718

Open
cfxegbert opened this issue Apr 11, 2024 · 0 comments
Open

rez-bind os requires arch #1718

cfxegbert opened this issue Apr 11, 2024 · 0 comments
Labels
bug rez-bind rez-bind related issues

Comments

@cfxegbert
Copy link
Contributor

cfxegbert commented Apr 11, 2024

When running rez-bind os the resulting package.py includes a requirement on arch-amd64. A operating system can run on multiple architectures. The requirement on arch should be relaxed.

Environment

  • windows-10.0.19044.SP0
  • rez 3.1.0
  • Rez python version 3.10.10

To Reproduce

  1. My rezconfig.py contains the following mapping
     platform_map = {
         "os": {
             r"(osx|windows)-(\d+)\..*": r"\1-\2",
         },
         "arch": {"AMD64": "amd64"},
     }
    
  2. rez-bind os

Expected behavior
In the os/windows-10/package.py

requires = [
    'platform-windows'
]

Actual behavior
In the os/windows-10/package.py

requires = [
    'platform-windows',
    'arch-amd64'
]
@cfxegbert cfxegbert added the bug label Apr 11, 2024
@JeanChristopheMorinPerso JeanChristopheMorinPerso added the rez-bind rez-bind related issues label Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug rez-bind rez-bind related issues
Projects
None yet
Development

No branches or pull requests

2 participants