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 conda-build #1065

Merged
merged 1 commit into from
Aug 1, 2016
Merged

Add conda-build #1065

merged 1 commit into from
Aug 1, 2016

Conversation

jakirkham
Copy link
Member

@jakirkham jakirkham commented Jul 16, 2016

Adds a recipe for conda-build. Written from scratch. Please let me know if you have any thoughts or suggestions

cc @conda-forge/core @msarahan @ilanschnell @kalefranz @mcg1969 @mingwandroid

@conda-forge-linter
Copy link

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 (recipes/conda-build) and found it was in an excellent condition.

@jakirkham jakirkham mentioned this pull request Jul 16, 2016
31 tasks

extra:
recipe-maintainers:
- jakirkham
Copy link
Member

Choose a reason for hiding this comment

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

add me please

Copy link
Member Author

Choose a reason for hiding this comment

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

Of course. 😄

Copy link
Member Author

Choose a reason for hiding this comment

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

See below.

@patricksnape
Copy link
Contributor

Weird windows error? C:\Miniconda\conda-bld\test-tmp_dir>call activate _test could not find environment: _test

@jakirkham
Copy link
Member Author

jakirkham commented Jul 17, 2016

@msarahan, any idea what is going on with Windows? As @patricksnape points out, it appears to not be able to activate the environment.

@msarahan
Copy link
Member

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.

@jakirkham
Copy link
Member Author

The OS X failure is a different story. Are we missing a dependency maybe? Or is this conda inspect still failing in tests on OS X?

+conda inspect -h
Traceback (most recent call last):
  File "/Users/travis/miniconda/bin/conda-inspect", line 3, in <module>
    from conda_build.main_inspect import main
  File "/Users/travis/miniconda/lib/python3.5/site-packages/conda_build/main_inspect.py", line 22, in <module>
    from conda.cli.install import check_install
ImportError: cannot import name 'check_install'

@msarahan
Copy link
Member

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?

@jakirkham
Copy link
Member Author

Though we do have conda as a dependency.

@msarahan msarahan mentioned this pull request Aug 1, 2016
@msarahan msarahan merged commit 487b785 into conda-forge:master Aug 1, 2016
@pelson pelson changed the title WIP: Add conda-build Add conda-build Aug 4, 2016
@jakirkham jakirkham deleted the add_conda-build branch August 4, 2016 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants