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 command line option to specify the wanted package format #5209

Merged
merged 12 commits into from
Nov 11, 2024

Conversation

ryanskeith
Copy link
Contributor

@ryanskeith ryanskeith commented Mar 2, 2024

Description

Fix for issue #4890.

  • Created enum class CondaPkgFormat to replace string usage of types
  • --package-format introduced as command line argument
    • This takes precedence over default value and .condarc
    • Normalization occurs so 1,tar.bz2,.tar.bz2,2,conda,.conda are all recognized and mapped appropriately
    • Other options are rejected
    • Action created so in future, multiple outputs can be explored

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?

@ryanskeith ryanskeith requested a review from a team as a code owner March 2, 2024 03:14
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Mar 2, 2024
Copy link

codspeed-hq bot commented Jun 7, 2024

CodSpeed Performance Report

Merging #5209 will not alter performance

Comparing ryanskeith:issue/4890 (6c05920) with main (d587da5)

Summary

✅ 5 untouched benchmarks

@ryanskeith ryanskeith closed this Jun 7, 2024
@ryanskeith ryanskeith deleted the issue/4890 branch June 7, 2024 22:56
@ryanskeith ryanskeith restored the issue/4890 branch June 7, 2024 22:57
@ryanskeith ryanskeith changed the title [DRAFT] Issue/4890 Issue/4890 Aug 2, 2024
@ryanskeith ryanskeith reopened this Aug 2, 2024
@@ -58,7 +99,7 @@ def set_invocation_time():
no_rewrite_stdout_env_default = "false"
ignore_verify_codes_default = []
exit_on_verify_error_default = False
conda_pkg_format_default = None
conda_pkg_format_default = CondaPkgFormat.V1
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
conda_pkg_format_default = CondaPkgFormat.V1
conda_pkg_format_default = CondaPkgFormat.V2

Copy link
Contributor

Choose a reason for hiding this comment

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

I think this change should be postponed to 25.1.

conda_build/metadata.py Outdated Show resolved Hide resolved
@beeankha beeankha mentioned this pull request Nov 6, 2024
3 tasks
Co-authored-by: jaimergp <jaimergp@users.noreply.github.com>
@jezdez jezdez changed the title Issue/4890 Add command line option to specify the wanted package format Nov 7, 2024
jezdez
jezdez previously approved these changes Nov 7, 2024
@dholth
Copy link
Contributor

dholth commented Nov 8, 2024

Isn't there also an environment variable for this

Co-authored-by: Daniel Holth <dholth@anaconda.com>
@jezdez jezdez merged commit d6c792c into conda:main Nov 11, 2024
28 checks passed
jaimergp added a commit to jaimergp/conda-build that referenced this pull request Nov 12, 2024
@jezdez jezdez mentioned this pull request Nov 12, 2024
3 tasks
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.

6 participants