jeremyletang / rust-sfml

SFML bindings for Rust
Other
631 stars 88 forks source link

What exactly is wrong with the wiki document for windows? #320

Closed TheFern2 closed 10 months ago

TheFern2 commented 10 months ago

I have no idea where else to discuss this, I know wikis are part of the community but there is no way to discuss this anywhere else.

If you compare my edits with newer edits there are claims about being outdated and everything below is old, and @crumblingstatue rather than fixing whatever was outdated, just made an entire new paragraph. This can lead to confusion for new rust-sfml users.

https://github.com/jeremyletang/rust-sfml/wiki/Windows/_history image

Nearly this entire addition just copies what was already in the MSVC section, but is worded differently.

image

Everything works for me in the msvc section, so exactly what is old and outdated?

crumblingstatue commented 10 months ago

I can't really remember what prompted me to make that edit, I think it was a user being confused from reading it.

As for why I didn't delete the old content, I guess I felt bad about completely undoing the work of other wiki contributors.

As for what's wrong with them:

Fell free to redo the wiki page based on this information.

TheFern2 commented 10 months ago

I can't speak for the GNU section the outdated banner can stay for that, I only work with MSVC, but if my memory serves me right and it was a while ago when I edited the wiki I was getting an error when running cargo build and the error was pointing to needing llvm (based on the error and some googling) maybe just a misconfiguration on my end before switching toolchains can't really remember 100%. But whatever isn't needed then def can be deleted, is there because that's the only way I could compile the examples at the time. I just think having two parts addressing msvc is probably not needed. I had tested with rust-sfml 0.11.2 at the time.

I will setup a brand new windows vm, and retest the setup to remove unnecessary items. I'll edit the wiki then. I'll add a specific rust-sfml version to the wiki, really the best way to handle this would be instructions in the repo so that those can be referenced within each release.

TheFern2 commented 10 months ago

@crumblingstatue Okay, removed duplicated instructions for msvc which is the default install for rust. I also tried the gnu toolchain instructions and I could not compile it to save my life. I added a small note so that if someone does get it or have it working they can chime in here or edit the wiki themselves.