MatejSloboda / Dijkstra_map_for_Godot

MIT License
77 stars 13 forks source link

Windows Binaries for 3.5.1 #115

Closed goblinJoel closed 1 year ago

goblinJoel commented 1 year ago

Thanks for getting the repo code updated to support Godot 3.5!

Would it be possible to get Windows binaries for 3.5.1 updated and/or updated in the asset library?

I've never attempted to work with rust or gdnative, but if it's actually pretty straightforward to set up enough to compile this, I could try it and make a PR. (I'm not sure how much the instructions are glossing over or if you have to do things a certain way to make it work on other peoples' OS versions / CPU brands.)


If not, maybe we can update the instructions -- they are a little misleading:

The instructions recommend the asset library, but the github method notes you have to use github for 3.5.1. It says you can skip the rust install, gdnative dependencies, and compilation if you're on Windows, but the existing files might be "slightly outdated" (which in this case means they just won't work). So for 3.5.1, you have to skip the recommended route to see the github version's note, then notice that the binaries in the repo are pretty old, then figure out how to set up rust and gdnative.

goblinJoel commented 1 year ago

Messed with it myself to see if I could build the Windows dll based on the instructions in the readme, but after installing various things and trying to get it to cargo build --release, I'm hitting errors about libclang being missing. I didn't understand the instructions for godot-rust, so I probably missed something there. Fun stuff. I'll wait to see if anyone who already has this set up on Windows is able to do the updated build.

astrale-sharp commented 1 year ago

Hey! Do you have clang installed? If you don't that'll probably fix it ☺️

I actually tried compiling for Windows (from linux) but it doesn't work at the moment and i don't think it's been fixed! If it doesn't work we can still compile with GitHub actions so just ask !


From: Joel @.> Sent: Sunday, October 23, 2022 12:03:36 AM To: MatejSloboda/Dijkstra_map_for_Godot @.> Cc: Subscribed @.***> Subject: Re: [MatejSloboda/Dijkstra_map_for_Godot] Windows Binaries for 3.5.1 (Issue #115)

Messed with it myself to see if I could build the Windows dll based on the instructions in the readme, but after installing various things and trying to get it to cargo build --release, I'm hitting errors about libclang being missing. I didn't understand the instructions for godot-rust, so I probably missed something there. Fun stuff. I'll wait to see if anyone who already has this set up on Windows is able to do the updated build.

— Reply to this email directly, view it on GitHubhttps://github.com/MatejSloboda/Dijkstra_map_for_Godot/issues/115#issuecomment-1287929581, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMZTDKR646MKWEXTFXQKFEDWERQDRANCNFSM6AAAAAARL67TMM. You are receiving this because you are subscribed to this thread.Message ID: @.***>

goblinJoel commented 1 year ago

Oh, do you have a github action set up to generate these? Please do, that'd definitely be great!

Not sure what's involved in updating things in the repo itself or for the asset library, but it seems like those are the remaining steps for the 3.5.1 upgrade to be wrapped up. (At least for Windows users. I see there's another issue in this repo for Linux still.)

astrale-sharp commented 1 year ago

Not yet but that could be done, it would up the repositary size tho which i don't like... for now have you tried installing clang before building? https://www.wikihow.com/Install-Clang-on-Windows

astrale-sharp commented 1 year ago

The asset library release is pending for approval ;)

astrale-sharp commented 1 year ago

Closing as the release is available on the asset store ! <3

goblinJoel commented 1 year ago

Confirming I was able to update to Godot 3.5.1 and update the addon via the AssetLib. Looks like it's working! Thanks, guys!

astrale-sharp commented 1 year ago

😇❤️


From: Joel @.> Sent: Sunday, October 30, 2022 2:13:42 AM To: MatejSloboda/Dijkstra_map_for_Godot @.> Cc: astrale-sharp @.>; State change @.> Subject: Re: [MatejSloboda/Dijkstra_map_for_Godot] Windows Binaries for 3.5.1 (Issue #115)

Confirming I was able to update to Godot 3.5.1 and update the addon via the AssetLib. Looks like it's working! Thanks, guys!

— Reply to this email directly, view it on GitHubhttps://github.com/MatejSloboda/Dijkstra_map_for_Godot/issues/115#issuecomment-1296048995, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMZTDKRJ4UR7Q7LTFOAMKOLWFXDUNANCNFSM6AAAAAARL67TMM. You are receiving this because you modified the open/close state.Message ID: @.***>