-
Notifications
You must be signed in to change notification settings - Fork 28
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
Building with VS2022 under Windows 11 #21
Comments
@suchja , may I suggest that we get initially your requirements straight?
|
Absolutely! I currently have only Win11 for development. The release version should run on Win10. I wasn't aware that there are such differences. Is it because building the addon.cc involves the WinSDK? Sorry, but I'm not aware of any other requirements. Ado you need to know something else? |
This comment was marked as outdated.
This comment was marked as outdated.
@suchja , is the anonymous backer on bountysource you? |
This comment was marked as outdated.
This comment was marked as outdated.
My tech skills are not on par for this. But maybe I can give some pointers..
The code in this repo is from @cmidgley, which also has containts the commits of the InfernoEmbedded fork.
Another fork which might be wortwhile looking into is this one. The changes from that repo have not been commited here. |
This comment was marked as outdated.
This comment was marked as outdated.
@suchja , I can eliminate the error (you have) on my side simply by falling back to VS2019 Can you try this: Build with VS2019
VS2022
after doing some research, this seems to be true. Not sure how trivial it is to update (you should avoid this, use the existent code of this project until it is updated officially) |
Is If VS2019 is ok, then the issue is solved (I would update the doc here a little bit to make things clearer, thus others do not fall into the same trap) |
draft revision of the build instructions: |
I attempted to upgrade to the latest everything earlier (including the latest Node and therefore node-gyp), but there were too many changes to Electron involved so I moved it as far forward as possible without breaking that. Somebody experienced with Electron likely could do those changes efficiently but it's not my expertise. There may be more involved, but that seemed to be the bulk of the lift. |
Thank you all for your immense effort and the high quality and fast feedback! I'll start from scratch with a new VM and give the different provided options a try. Hope that I'll be able to do so on this weekend. |
Hey, @suchja, i got involved here in context of the bounty, see #21 (comment). Ignoring my concrete questions and results is not the way to go.
You do not need a new vm. |
Can someone please clarify who backed the bounty? https://app.bountysource.com/issues/120217906-building-with-vs2022-under-windows-11 |
@cmidgley , I've added your info re updating to |
I've added a basic CI here, and it builds fine, just by falling back to a windows-2019 runner (uses VS2019). https://github.com/deepnest-io/Deepnest/actions/runs/4899908971/jobs/8750207768 So, @suchja, this issue is solved, once you've verified on your side, please close thus I can process the bounty and go on with the other tasks here. EDIT: see updated readme for instructions: https://github.com/deepnest-io/Deepnest/blob/master/README.md#prerequisites |
Did some additional work to get the project running, see APEX issues and the MAJOR issues/sub-issues: |
Closing, in order to speedup things and go on with: |
@suchja , hi there! As you possibly know, bountysource stopped paying out bounty-workers since early this year The bountysource app is down since mid Sep. 2023 https://app.bountysource.com/issues/120217906-building-with-vs2022-under-windows-11 Paypal Buyer ProtectionThough you got the service from me, I did not receive the payment, bountysource has kept the money. You are now eligible for paypal buyer protection (others have gone successfully this path) : bountysource/core#1586 (comment) If you could do this, and pay me directly, you could keep the 10% for your efforts. Thank you. (EDIT: notified bounty-issuer additionally via email) |
I'm trying to get this (as well as the InfernoEmbedded fork) to build for several hours now. Unfortunately the node-gyp is not working. Most likely it is not an issue at all with this repo, but with my configuration. However, as you guys figured out how to build, I hope someone can point me into the right direction. I'm willing to create some enhanced installation / build instructions, if you think it is required (I do ;).
I'm using Win11 (in a VM) and VS2022 (including the desktop C++ workload). I've installed NVM for windows and I am using node 8.17.0 as suggested. Also I finally installed pyenv with Python 2.7.18.
The problem I'm mainly facing is this:
I've added the following lines to my .npmrc and I would think, that it is working:
However, it looks like it is not able to use my VS2022. Is somebody using VS2022 for build deepnest? Otherwise I would assume that node-gyp is simply not compatible with VS2022 (or at least not the old node-gyp version used here).
The text was updated successfully, but these errors were encountered: