search

MelonSpeedruns / BotwRandomizer

GNU General Public License v3.0
56 stars 15 forks source link

Use os.path.normpath to stay platform agnostic #13

Closed nalyDylan closed 2 years ago

nalyDylan commented 2 years ago

A commit to switch to os.path.normpath to fix issues with Linux

nalyDylan commented 2 years ago

Someone should probably confirm this works on Windows, but the documentation says the function converts / to \ on Windows also

nalyDylan commented 2 years ago

Nvm I ran it in a VM and it produces bad builds, any thoughts @MelonSpeedruns ?

nalyDylan commented 2 years ago

I solved the issue with the above commit, the aoc folder was no longer being named to Aoc, I cleaned that up and some other places where it didn't matter on Windows that the folder wasn't case-sensitive.

The script now makes graphicsPacks that run on my Linux Cemu when generated on either Windows or Linux

MelonSpeedruns commented 2 years ago

While I super appreciate the pull request, the Randomizer is now C# based, meaning that the code you wrote won't work anymore. Thank you so much for your interest!