Skip to content

fix: update apple/container asset name for versions 0.7.0 and later#46443

Closed
preinpost wants to merge 1 commit into
aquaproj:mainfrom
preinpost:fix/apple/container
Closed

fix: update apple/container asset name for versions 0.7.0 and later#46443
preinpost wants to merge 1 commit into
aquaproj:mainfrom
preinpost:fix/apple/container

Conversation

@preinpost
Copy link
Copy Markdown

@preinpost preinpost commented Dec 31, 2025

apple/container - A tool for creating and running Linux containers using lightweight virtual machines on a Mac. It is written in Swift, and optimized for Apple silicon

Check List

Summary

Starting from version 0.7.0, the asset naming convention changed:

  • Before (≤0.6.0): container-{version}-installer-signed.pkg

  • After (≥0.7.0): container-installer-signed.pkg
    This change adds version_overrides to support both naming conventions.

  • Install and execute the package and confirm if the package works well

    • Tested with 0.6.0 (old format) 0.7.0 and 0.7.1 (new format)

@suzuki-shunsuke
Copy link
Copy Markdown
Member

suzuki-shunsuke commented Dec 31, 2025

version_constraint: "false"
version_overrides:
- version_constraint: semver("<= 0.6.0")
- version_constraint: semver("< 0.7.0")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think this change has meaning.

https://github.com/apple/container/releases

Versions:

  • 0.7.1
  • 0.7.0
  • 0.6.0

The evaluation result of <= 0.6.0 and < 0.7.0 are same in this case.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I verified that it installs correctly using aqua directly (using ref: v4.450.0).

My apologies. The root cause was actually my outdated local environment (I was using mise v2025.8.21). After upgrading mise to the latest version, everything works perfectly.

I saw the error log and hastily assumed the config was broken without checking the existing logic carefully.
Thanks for the review. I'll close this PR.

@preinpost preinpost closed this Dec 31, 2025
@github-project-automation github-project-automation Bot moved this to Done in main Dec 31, 2025
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