Skip to content

New package: neovide-0.15.0 #55149

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

theoheretic
Copy link

@theoheretic theoheretic commented Apr 22, 2025

Closes #43688

Testing the changes

  • I tested the changes in this PR: YES

New package

Local build testing

  • I built this PR locally for my native architecture, x86_64-glibc
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • x86_64-musl
    • aarch64
    • aarch64-musl

Notes

Currently figuring out how to get aarch64{,"-musl"} to build.

@classabbyamp classabbyamp added the new-package This PR adds a new package label Apr 22, 2025
@theoheretic theoheretic force-pushed the init-neovide branch 5 times, most recently from 2b12e58 to caff021 Compare April 23, 2025 21:49
@classabbyamp
Copy link
Member

it's definitely possible to build on musl: https://gitlab.alpinelinux.org/alpine/aports/-/blob/master/community/neovide/APKBUILD

@tranzystorekk
Copy link
Contributor

There's a lot of work-in-progress bits to this PR that makes me not want to give it a green light over the prior work in #53515

@theoheretic
Copy link
Author

theoheretic commented Apr 25, 2025

@classabbyamp: Good to know. I'll look at that for inspiration.
@tranzystorekk: Oh, interesting. I wasn't aware of the previous PR for Neovide. I'm up for drafting it until I feel as if it is complete.

@theoheretic theoheretic marked this pull request as draft April 25, 2025 06:02
@theoheretic

This comment was marked as resolved.

@theoheretic
Copy link
Author

theoheretic commented Apr 26, 2025

Resolved the comment above by using xbps-src -A <arch> instead of xbps-src -a <arch>.

Now the build failure happens during install phase when it can't find gn... despite it being available as a host dependency? Some further looking is needed.

It now builds successfully on musl after hacking around a small bit.

@theoheretic theoheretic marked this pull request as ready for review April 26, 2025 19:29
@theoheretic theoheretic marked this pull request as draft April 26, 2025 19:41
@theoheretic theoheretic marked this pull request as ready for review April 26, 2025 19:42
@theoheretic theoheretic marked this pull request as draft April 27, 2025 09:15
@theoheretic
Copy link
Author

Cross-building is now fixed and as such, the PR is ready to be merged.

@theoheretic theoheretic marked this pull request as ready for review April 27, 2025 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Package request: NeoVide
3 participants