-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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 conda-build #1065
Add conda-build #1065
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
|
||
extra: | ||
recipe-maintainers: | ||
- jakirkham |
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.
add me please
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.
Of course. 😄
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.
See below.
71c9124
to
a1baa29
Compare
a1baa29
to
487b785
Compare
Weird windows error? |
@msarahan, any idea what is going on with Windows? As @patricksnape points out, it appears to not be able to activate the environment. |
Not sure if I have a fix in above PR. Possibly. I'm going to tag 1.21.6 tomorrow - let's rebuild this when that is available. |
The OS X failure is a different story. Are we missing a dependency maybe? Or is this
|
That should come from conda, obviously. If we're not seeing it here, then I think the reason might be that conda is in the root environment, but that's not on sys.path? |
Though we do have |
Adds a recipe for
conda-build
. Written from scratch. Please let me know if you have any thoughts or suggestionscc @conda-forge/core @msarahan @ilanschnell @kalefranz @mcg1969 @mingwandroid