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

Enable easy local testing of mulled-build #8537

Merged
merged 7 commits into from
Apr 5, 2018
Merged

Enable easy local testing of mulled-build #8537

merged 7 commits into from
Apr 5, 2018

Conversation

daler
Copy link
Member

@daler daler commented Apr 5, 2018

  • I have read the guidelines for bioconda recipes.
  • This PR adds a new recipe.
  • AFAIK, this recipe is directly relevant to the biological sciences (otherwise, please submit to the more general purpose conda-forge channel).
  • This PR updates an existing recipe.
  • This PR does something else (explain below).

After we moved to CircleCI, it got harder to do local tests with mulled-build due to docker not running nicely from within docker using circleci build. This PR makes some changes to .circleci/setup.sh to make it play nicer with the new driver script bootstrap.py. The driver script creates a custom activate script to activate the new conda installation.

Example usage:

$ ./bootstrap.py /tmp/miniconda
$ source activate ~/.config/bioconda/activate
(BIOCONDA-UTILS) $ bioconda-utils build recipes config.yml --mulled-test --git-range master
(BIOCONDA-UTILS) $ source deactivate

@bgruening
Copy link
Member

@daler out of interest what was the error which you are seeing. It seemed to work for me really nicely.

@daler
Copy link
Member Author

daler commented Apr 5, 2018

@bgruening no error, it's just that I don't think circleci build runs the mulled-build tests (https://github.com/bioconda/bioconda-recipes/blob/master/.circleci/config.yml#L79). So there's not really a good way of locally debugging failures from the mulled-build stage using just the CircleCI client. Also I know folks have complained in the past about messing up their existing root env when installing bioconda-utils, so this tries to take care of all of those points.

@daler
Copy link
Member Author

daler commented Apr 5, 2018

(making sure a recipe builds correctly, will revert build number once it does)

@daler daler merged commit 063d3d1 into master Apr 5, 2018
@daler daler deleted the bootstrap branch April 5, 2018 22:26
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.

2 participants