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

fix: Pass pkg_format to container-based build #994

Merged

Conversation

mbargull
Copy link
Member

@mbargull mbargull commented Jun 1, 2024

This way we should be able to switch to pkg_format: 2 (i.e., build .conda packages) via config from bioconda-common.

Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
@mbargull mbargull force-pushed the fix-pkg_format-build_script_template branch from ed81dee to 20a732e Compare June 1, 2024 22:05
# v0.16.2: this used to have a side effect, calling conda build purge
# hopefully, it's not actually needed.
build_conf = utils.load_conda_build_config()
return build_conf.build_folder
Copy link
Member Author

Choose a reason for hiding this comment

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

.build_folder is actually the .../conda-bld/pkg_.... path if the Config would be initialized for some package pkg -- it just happens to be .../conda-bld/ since the pkg_... part is empty for a Config not created for a certain package/recipe.
The new code has the correct config.croot (which is always only the .../conda-bld part.

@mbargull mbargull merged commit e3f23ed into bioconda:master Jun 2, 2024
9 checks passed
mbargull pushed a commit that referenced this pull request Jun 2, 2024
🤖 I have created a release \*beep\* \*boop\*
---
##
[3.2.0](https://www.github.com/bioconda/bioconda-utils/compare/v3.1.0...v3.2.0)
(2024-06-02)


### Features

* Update base-glibc-*-bash images to 3.1
([#996](https://www.github.com/bioconda/bioconda-utils/issues/996))
([e7c6ca0](https://www.github.com/bioconda/bioconda-utils/commit/e7c6ca0bf7bf2a0815c694634e45d7dff0aa483a))


### Bug Fixes

* Pass pkg_format to container-based build
([#994](https://www.github.com/bioconda/bioconda-utils/issues/994))
([e3f23ed](https://www.github.com/bioconda/bioconda-utils/commit/e3f23ed70300d97a8f99c5bace7dbba0f559bd19))
---


This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

1 participant