pasteofexile icon indicating copy to clipboard operation
pasteofexile copied to clipboard

readme: update instructions for Windows

Open ocombe opened this issue 1 year ago • 7 comments

Hello ! Installing Wrangler on Windows only works if Perl has been installed (should be installed by default on Unix systems). I added a link to a working Windows Perl distribution in the readme.

ocombe avatar Feb 06 '24 16:02 ocombe

I am not aware of a component which needs Perl, can you tell me what failed for you without Perl, so I can see if we just cant get rid of that all together?

Dav1dde avatar Feb 06 '24 17:02 Dav1dde

It is required for wrangler, you cannot run cargo install wrangler without perl, it fails during compilation

More info: https://github.com/sfackler/rust-openssl/issues/1086#issuecomment-846160769

ocombe avatar Feb 06 '24 17:02 ocombe

Thanks, that's annoying.

Btw that reminds me you might have to install worker-build from my fork, if you encounter some bundling errors. Not sure if they ever fixed the bug I had with the assets manifest.

Dav1dde avatar Feb 06 '24 17:02 Dav1dde

I didn't go that far yet, I tried yarn start and it fails already :D I'll look into it later

ocombe avatar Feb 06 '24 18:02 ocombe

@ocombe did you figure out the issues? I think you'd definitely have to use WSL, the setup is very fragile and held together with a bash script, which is probably the issue.

Dav1dde avatar Feb 11 '24 17:02 Dav1dde

I didn't retry, like you said the scripts won't work on windows. I just wanted to check the sources (especially the part used to create the passive skill tree), so for now it doesn't really matter

ocombe avatar Feb 12 '24 20:02 ocombe

The skilltree is in poe-tmm.

I recently reworked this, it used to be a service which generated SVGs on the fly, now it's just a single SVG per game version with embedded JS. If you're interested in the service you can go a few commits back.

Dav1dde avatar Feb 12 '24 21:02 Dav1dde