Skip to content

Conversation

@tlaferriere
Copy link

Added tests and found some issues.

@tlaferriere tlaferriere marked this pull request as draft March 30, 2025 13:25
@tlaferriere tlaferriere changed the title Draft: Add tests for anyzls Add tests for anyzls Mar 30, 2025
@marler8997
Copy link
Owner

Looks like zls 0.9.0 and earlier may not have supported the --version flag?

zls 0.9.0 --version failure
anyzig: zig version '0.9.0' pulled from 'C:\Users\runneradmin\AppData\Local\zig\o\20be8a7d44a547184df2cc35e2c47416\out\build.zig.zon'
anyzig: appdata 'C:\Users\runneradmin\AppData\Local\anyzig'
anyzig: zig '0.9.0' already exists at 'C:\Users\runneradmin\AppData\Local\zig\p\N-V-__8AAEeRXQAh8Tv1e53XQunUhPMVhmsW8XRiUXpRHZIh'
Unrecognized argument --version
error: 
========= expected to find in stdout: =========
0.9.0
========= but stdout does not contain it: =====

@tlaferriere
Copy link
Author

There is odd behavior in the CI job for macos-x86_64, it tries to compile and run the macos-aarch64, which is obviously wrong... I'm trying to understand why it's trying to run arm artifacts on an x86_64 platform. It's either our code not reading platform information properly or they are using an arm Mac expecting the rosetta compatibility layer to take care of running x86_64 compiled applications. In which case, we will have to consider macos-x86_64-compiled binaries compatible with the platform macos-aarch64. On the other hand, it seems ZLS 0.9.0 doesn't support macos-aarch64 based on their installation guide, and the 404 error we get in CI.

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