-
-
Notifications
You must be signed in to change notification settings - Fork 21
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
ENH CI setup and an example #2
Conversation
WIP code to generate and build the CDTs
ok @conda-forge/core @conda-forge/help-cdts This PR is ready for review! |
Very nice! |
): | ||
cdt_name = distro | ||
bits = "32" if architecture in ("armv6", "armv7a", "i686", "i386") else "64" | ||
base_architectures = dict({"i686": "i386"}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the significance of base_architectures
? Would it make sense to add a comment above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was in the upstream code, so IDK. Maybe @mingwandroid can comment?
Thanks for the comments @jakirkham! I made two issues for the ones I'd like to address later. |
any more comments here @conda-forge/core ? |
Thanks Matt! 😀 |
This PR contains all of the initial code needed for the CDT builds.
Note that it won't actually upload anything yet as I have not put in the proper token on azure. Once we are confident the CDT recipes are correct, I will add the token.
cc @conda-forge/help-cdts @conda-forge/core