-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add support for Plan 9 acme #25
Comments
Hey Fazlul, I'd like to help making this work in 9front. Did you have any ideas already on how to tackle this? Is the only problem the way mountAcme works? |
There is a PR open for
PRs are welcome. |
In #34 I think I addressed your points about integration with acme and plumber (while for the moment for the proxy I'm just reverting to tcp via CLI flags). Made some progress debugging the PR; right now my issue is that |
I tried applying the patch by hand on 9front in order to add the log support to acme; it worked fine, as far as I could tell. Backporting that patch should do the trick. I have no idea how to do that, but if you need some help on that I wouldn't mind figuring it out. |
Hey @otremblay I was waiting for some feedback from Fazlul either here or on the PR before taking the next step. I'm working on something else right now and I'll stick with *BSD + p9p for the time being. FWIW, I'd also considered applying the patch to 9front's acme, but didn't get around to it. |
Another option is to use Edwood on Plan 9, which has log support. The issue again comes down to |
I've created an importable 9fans.net/go fork with plan9 support:
This should give us a good starting point for acme-lsp plan9 support. If you want to try out Edwood for log support, you'll need to change all the import paths first:
There is a small issue with accessing the plan9 plumber. Edwood tries to mount the plumb service when it's usually already mounted at /mnt/plumb on plan9. You can comment it out to get it started. |
Update: I've also created a plan9 branch, which has some of the commands already working. |
The acme log patch has been merged into 9front and I fixed the remaining issues. acme-lsp on master supports plan9 now. Work remaining:
|
Indeed it works, thanks Fazlul. |
Thank you for uploading this fork. It's generally extremely useful. I hope your PR in the original package will be merged someday! |
Thanks for verifying it works. I wasn't able to set up CI for plan9 -- trying to run tests in qemu. It would've been easier if kvm was supported. |
acme-lsp should be made to work on Plan 9 (or 9front). Currently, it doesn't work because of
9fans.net/go
packages and because we use unix domain sockets.The text was updated successfully, but these errors were encountered: