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

Update for conda/conda-build deprecations #395

Merged
merged 9 commits into from
Mar 20, 2024

Conversation

mbargull
Copy link
Collaborator

@mbargull mbargull commented Mar 14, 2024

This updates boa for compatibility with current/upcoming conda-build versions.
Overview:

Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
@mbargull mbargull force-pushed the update-for-conda-build-deprecations branch 4 times, most recently from 76ebc17 to d6eae80 Compare March 14, 2024 12:38
conda._vendor.boltons has been marked for deprecation in conda=23.3 and
will be removed in conda=23.4.

refs:
- https://github.com/conda/conda/pull/12453/files

Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
@mbargull mbargull force-pushed the update-for-conda-build-deprecations branch from 95fd1b8 to b6572ec Compare March 16, 2024 08:31
Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
@mbargull mbargull force-pushed the update-for-conda-build-deprecations branch 2 times, most recently from 8e23ad8 to 689bf8e Compare March 16, 2024 09:01
@mbargull mbargull force-pushed the update-for-conda-build-deprecations branch from 689bf8e to 1aa8d1f Compare March 16, 2024 10:33
Signed-off-by: Marcel Bargull <marcel.bargull@udo.edu>
@mbargull mbargull marked this pull request as ready for review March 16, 2024 15:01
@@ -149,7 +149,7 @@ def get_value(self, in_key: str, default: Any = None, autotype=True) -> Any:
if autotype and default is None and FIELDS.get(section, {}).get(key):
default = FIELDS[section][key]()

section = self.output.sections.get(section, {})
section = self.meta.get(section, {})
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This caused gh-388 since .get_value is called from conda-build code which expects return types from original conda_build.metadata.MetaData (e.g., str, not CondaBuildSpec).
From a quick glance, I couldn't make out any calls from within boa that expect boa-specific types here; would be good to have another pair of eyes confirming this..

@jakirkham
Copy link

cc @SylvainCorlay (for awareness)

@wolfv wolfv merged commit e18263a into mamba-org:main Mar 20, 2024
7 checks passed
@wolfv
Copy link
Member

wolfv commented Mar 20, 2024

Terrific job, thanks!

@dhvu
Copy link

dhvu commented Mar 20, 2024

Nice. Thank you @mbargull

@jakirkham
Copy link

Thanks all! 🙏

Think we will want a release with these changes as well. Not sure what the process is there

@wolfv
Copy link
Member

wolfv commented Mar 21, 2024

Just made a PR @jakirkham #396

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.

AttributeError: 'CondaBuildSpec' object has no attribute 'split'
4 participants