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

Build .conda artifacts by default #5527

Merged
merged 10 commits into from
Nov 27, 2024
Merged

Conversation

jaimergp
Copy link
Contributor

@jaimergp jaimergp commented Nov 6, 2024

Description

Closes #5183

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Nov 6, 2024
@jakirkham
Copy link
Member

Thanks Jaime! 🙏

Copy link

codspeed-hq bot commented Nov 6, 2024

CodSpeed Performance Report

Merging #5527 will not alter performance

Comparing jaimergp:default-dot-conda (25c4f5d) with main (4260236)

Summary

✅ 5 untouched benchmarks

@beeankha
Copy link
Member

beeankha commented Nov 6, 2024

It looks like there is a similar/related issue that was filed here previously: #4890

There's currently an open PR that addresses this issue by creating a new command line argument --package-format and may potentially be the option we want to go with to include in the November 2024 conda-build release. @jaimergp @jakirkham please let me know your thoughts on this.

@jakirkham
Copy link
Member

Thanks Bianca! 🙏

Think these are different changes

This change is making the default package type .conda (instead of .tar.bz2) as requested in issue: #5183

That change is allowing users to specify the package type via CLI

These are both good changes. They just serve different purposes. Ideally we get them both in

With this change, we are thinking of having a deprecation (or future) warning in 24.11 to note the default would change in 25.1. Then include this change after the 24.11 branch (or tag)

@jaimergp
Copy link
Contributor Author

jaimergp commented Nov 7, 2024

Yea maybe #5209 should go in first (that refactor is useful here too). It's also a good opportunity to add a warning there about the future change in e.g. 25.1. In other words:

For 24.11:

For 25.1:

  • Rebase this PR and flip the default. Some tests need adjustment.

@jezdez
Copy link
Member

jezdez commented Nov 7, 2024

Yea maybe #5209 should go in first (that refactor is useful here too). It's also a good opportunity to add a warning there about the future change in e.g. 25.1. In other words:

For 24.11:

  • Review and merge Issue/4890 #5209
  • A new PR adds a warning about the change of the default in 25.1

For 25.1:

  • Rebase this PR and flip the default. Some tests need adjustment.

I agree with that timeline, just reviewed #5209

@jaimergp jaimergp marked this pull request as ready for review November 26, 2024 15:25
@jaimergp jaimergp requested a review from a team as a code owner November 26, 2024 15:25
Copy link
Contributor

@beckermr beckermr left a comment

Choose a reason for hiding this comment

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

LGTM! I wonder if in the future we need to add to the test suite to cover both formats or possibly move all of the tests to .conda and remove support to build the old format all together.

@@ -101,7 +101,7 @@ def ext(self):
no_rewrite_stdout_env_default = "false"
ignore_verify_codes_default = []
exit_on_verify_error_default = False
conda_pkg_format_default = CondaPkgFormat.V1
conda_pkg_format_default = CondaPkgFormat.V2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is the main change. Everything else is to fix assumptions about tar.bz2 being always default.

@beckermr beckermr merged commit 3cf75b6 into conda:main Nov 27, 2024
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed [bot] added once the contributor has signed the CLA
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Make .conda the default format for future conda-build versions
6 participants