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

aqua gr generates arm64 replacements even though windows_arm_emulation is enabled #3336

Closed
suzuki-shunsuke opened this issue Dec 5, 2024 · 1 comment · Fixed by #3337
Closed
Labels
bug Something isn't working command:generate-registry
Milestone

Comments

@suzuki-shunsuke
Copy link
Member

suzuki-shunsuke commented Dec 5, 2024

aqua info

$ aqua info
{
  "version": "2.38.3",
  "commit_hash": "53d9f3361b6b64e36d06689524a96e0048a348db",
  "os": "darwin",
  "arch": "arm64",
  "pwd": "/Users/(USER)/Documents/aqua/aqua",
  "root_dir": "/Users/(USER)/.local/share/aquaproj-aqua",
  "env": {
    "AQUA_GLOBAL_CONFIG": ":/Users/(USER)/repos/src/github.com/aquaproj/aqua-registry/aqua-all.yaml",
    "AQUA_PROGRESS_BAR": "true"
  },
  "config_files": [
    {
      "path": "/Users/(USER)/Documents/aqua/aqua/aqua/aqua.yaml"
    }
  ]
}

Overview

aqua gr generates arm64 replacements even though windows_arm_emulation is enabled.

How to reproduce

aqua gr -l 1 coder3101/protols

Expected behaviour

packages:
  - type: github_release
    repo_owner: coder3101
    repo_name: protols
    description: Language Server for protocol buffers
    asset: protols-{{.Arch}}-{{.OS}}.{{.Format}}
    format: tar.gz
    windows_arm_emulation: true
    overrides:
      - goos: windows
        format: zip
    replacements:
      amd64: x86_64
      arm64: aarch64
      darwin: apple-darwin
      linux: unknown-linux-gnu
      windows: pc-windows-msvc

Actual behaviour

packages:
  - type: github_release
    repo_owner: coder3101
    repo_name: protols
    description: Language Server for protocol buffers
    asset: protols-{{.Arch}}-{{.OS}}.{{.Format}}
    format: tar.gz
    windows_arm_emulation: true
    overrides:
      - goos: windows
        format: zip
        replacements:
          arm64: arm64
    replacements:
      amd64: x86_64
      arm64: aarch64
      darwin: apple-darwin
      linux: unknown-linux-gnu
      windows: pc-windows-msvc

Note

We can reproduce the issue using old aqua (v2.30.0) too.

aqua upa v2.30.0

@suzuki-shunsuke suzuki-shunsuke added the bug Something isn't working label Dec 5, 2024
@suzuki-shunsuke
Copy link
Member Author

If I comment out this code, the bug is fixed.

override.Replacements[k] = k

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working command:generate-registry
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant