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

mesonlsp: init at 4.1.8 #19733

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

mesonlsp: init at 4.1.8 #19733

wants to merge 2 commits into from

Conversation

Freed-Wu
Copy link
Contributor

Fix #19630

I met https://todo.sr.ht/~lattis/muon/116 , which looks like the bug of upstream.

@truboxl
Copy link
Contributor

truboxl commented Jun 6, 2024

Again abandoned and not building. Will be closed if not handled.

@Freed-Wu
Copy link
Contributor Author

Freed-Wu commented Jun 6, 2024

I think it's an issue with the upstream muon buildsystem.

What is the result? @JCWasmx86

@JCWasmx86
Copy link
Contributor

It's fixed upstream in muon, so if there are compilation errors, these are within the scope of mesonlsp

@Freed-Wu Freed-Wu force-pushed the mesonlsp branch 7 times, most recently from 68e3fdd to 142f526 Compare June 6, 2024 06:19
@JCWasmx86
Copy link
Contributor

Revisiting this, I doubt this has a future here, unless you are open to modifying a lot of code due to this reasons:

  • I never even thought of running this on Android systems
  • I use the most recent C++ versions (C++23) with GNU extensions and nearly the most recent compilers
  • I never tested outside of macOS/Windows/Linux

If you get it running here, feel free to add it to CI upstream, so it can be somewhat more tested, but I think this needs a lot of work to be viable

@licy183
Copy link
Member

licy183 commented Oct 22, 2024

@Freed-Wu Hello, could you please test whether this package works fine on Termux? Thanks!

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.

[Package]: mesonlsp
4 participants