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

Musl detection broken #629

Closed
1 task done
dafu opened this issue Nov 5, 2022 · 1 comment · Fixed by #630
Closed
1 task done

Musl detection broken #629

dafu opened this issue Nov 5, 2022 · 1 comment · Fixed by #630

Comments

@dafu
Copy link

dafu commented Nov 5, 2022

I've searched open issues for similar requests

Yes

I've recently downloaded the latest plugin version of mason.nvim

  • Yes

Problem description

Hi!

"feat(platform): better glibc detection #537 " breaks musl detection on alpine.

ldd --version returns 1 which will set vim.v.shell_error to 1 and cause the new system() function to fail.

get_libc returns nothing resulting in just Linux_x64 instead of Linux_x64_musl

Detection works in the previous commit.

Expected behavior

detect musl with Linux_x64_musl

Affected packages

*_musl

Mason output

No response

Installation log

[ERROR Sat Nov  5 18:47:01 2022] .../paqs/start/mason.nvim/lua/mason-core/installer/init.lua:154: Installation failed for Package(name=rust-analyzer) error=".../paqs/start/mason.nvim/lua/mason-core/installer/init.lua:121: Could not find which release file to download.\nMost likely the current operating system or architecture is not supported (Linux_x64).

Neovim version (>= 0.7)

NVIM v0.8.0
Build type: MinSizeRel
LuaJIT 2.1.0-beta3
Compiled by buildozer@build-edge-x86_64

Operating system/version

Linux 6b15fbca47d8 5.15.41-0-lts #1-Alpine 3.16.2

Healthcheck

mason: require("mason.health").check()    
  ========================================================================    
  ## mason.nvim report          
    - OK: neovim version >= 0.7.0    
    - OK: Go: `go version go1.19.2 linux/amd64`    
    - OK: cargo: `cargo 1.64.0 (387270bc7 2022-09-16)`    
    - WARNING: luarocks: not available    
    - WARNING: Ruby: not available    
    - WARNING: RubyGem: not available    
    - WARNING: Composer: not available    
    - WARNING: PHP: not available    
    - OK: npm: `8.19.1`         
    - OK: node: `v19.0.0`       
    - OK: python3: `Python 3.10.8`    
    - WARNING: pip3: not available    
    - WARNING: javac: not available    
    - WARNING: java: not available    
    - WARNING: julia: not available    
    - OK: wget: `GNU Wget 1.21.3 built on linux-musl.`    
    - OK: curl: `curl 7.85.0 (x86_64-alpine-linux-musl) libcurl/7.85.0 OpenSSL/3.0.5 zlib/1.2.13 brotli/1.0.9 nghttp2/1.50.0`    
    - OK: gzip: `gzip 1.12`     
    - OK: tar: `tar (GNU tar) 1.34`    
    - OK: bash: `GNU bash, version 5.2.2(1)-release (x86_64-alpine-linux-musl)`    
    - OK: sh: `Ok`              
    - OK: GitHub API rate limit. Used: 5. Remaining: 55. Limit: 60. Reset: Sat Nov  5 20:10:37 2022.    
                                
~

Screenshots

No response

@williamboman
Copy link
Owner

Ah, it doesn't respect exit code appropriately. #630 should fix this, please reopen if it doesn't!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants