-
Notifications
You must be signed in to change notification settings - Fork 25
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
Window support (depends on llbpg_query) #22
Comments
Hi! I don't use windows so would have a hard time testing, reproducing or fixing this. Happy to accept a PR. |
Hello, Thanks for your reply. I would like to look into how I can do this but I am going to need a lot of help from you. I have not done anything with |
Hello all involved, My team member is also affected by this. Same 404 error reported by npm, on the following resource: I went a few directories up on the URI and: It doesn't look like there are any windows packages on the S3 bucket. Only linux and darwin versions. |
Same here, fails to install ERROR Stack Trace
|
The underlying issue is in the library that we depend on, libpq_query? Either way, all of the contributors use either mac or linux, and would need your help to make a PR to fix this. So if there is anyone out there (on Windows) that needs this feature, please dig in and help your community! We'll gladly merge a PR that is proven to fix this issue. |
@pyramation If I'd know or work with Thanks! |
Here is the relevant issue, thanks for note @nichita-pasecinic |
Hm, wonder if using WebAssembly in Wonder if an unconventional alternative would be to compile the C code to WebAssembly (Wasm) using something like Of course, ideally Also, if the compiled Wasm could be published to npm, then it may offer a way to avoid the compilation step every time the @RReverser recently ported Sharp to WebAssembly, and used the |
@gregnr amazing, thanks so much for taking the time to look at this!! 🔥🔥 I'll be watching that PR closely, and will help with testing where I can! |
same problem here. |
Oh nice, it looks like @lfittl added support for compiling on Windows to @pyramation @darora would it be possible to get a new release with these changes? Going to be great to install SafeQL by @Newbie012 for our students with Windows 👍 |
I don't have a windows machine to build it, but would be happy to find a way to get this in |
Could it be built on CI somehow? (eg. GitHub Actions) Could also be hosted on GitHub Releases maybe, as mentioned by @maschwenk in #35 |
@pyramation or if running the Windows builds in CI is not an option, could a community member with a Windows machine help get this released? What needs to be done here? |
We're talking about building binaries, so my first question is how can we securely verify it? Someone that is trusted could manually send a binary over. Then we can publish a new version, and then publish all three together for the new npm version |
Good point. There is something in
Ok, makes sense. @lfittl would you or someone on your team be able to build a Windows binary and send it to @pyramation ? I think that I personally shouldn't be considered as a trusted person yet. I also asked whether
Nice, I reached out there too, in case we need to do more back and forth. |
I'm afraid I can be of very little help here, because I'm just a plain end user of |
Yes, we can set that up. The question I have is ... is it possible to have a CI env that somehow spins up all three operating system environments (windows, osx, and linux) and can get all the output files together before publishing — it may take some wrangling to get it to work |
Should be possible on GitHub Actions, yeah. There are other projects where I've seen GitHub Actions used to automatically build and publish binaries from multiple architectures. For example:
|
WOW! thanks @karlhorky — trying something out, stay tuned! |
https://github.com/launchql/libpg-query-node/actions/runs/7984103879/job/21800377394 I started working on a workflow, need to sort out a mac issue — I think it's similar to the M1/docker issue that typically pops up windows also fails, but at least it's reproducible in CI! |
@pyramation any update, do we support windows? |
@Eprince-hub Looks like Windows support has landed as of Can you confirm? And if it's working for you, then this issue can be closed? |
closed with #46 |
libpg-query is now supported on Windows launchql/libpg-query-node#22 (comment)
libpg-query is supported now on Windows launchql/libpg-query-node#22 (comment)
libpg-query compatible with Windows: launchql/libpg-query-node#22 (comment)
libpg-query is now installable on Windows launchql/libpg-query-node#22
The latest release of the libpg-query now supports Windows; installed successfully in a project on a Windows machine after I installed a fresh Windows 11 on my Windows machine and followed the UpLeveled System Setup Guide
|
libpg-query is now installable on Windows launchql/libpg-query-node#22
Installing libpg-query on windows 10/11 failed with this long errors below
I think libpg-query doesn't support windows machines as pointed out in this issue here
launchql/pgsql-parser#24
OS: Windows 10 and Windows 11
Node Version: 18
I have tried the installation in the following command lines
VSCode Terminal
Hyper
Windows command prompt
Windows PowerShell
In every instance, the same error was returned
Steps to reproduce
You can try this installation on this repo
A computer running windows OS is needed to reproduce this error
clone
projectcd
into projectcd libpg-query-installation-error
install
libpg-queryThe text was updated successfully, but these errors were encountered: