stanoddly / DotnetRaylibWasm

Prototype for .NET 7 with Raylib compiled into WebAssembly (wasm)
MIT License
60 stars 8 forks source link

Raylib-cs can't be used because of the hyphen in the name #5

Closed stanoddly closed 8 months ago

stanoddly commented 1 year ago

[!IMPORTANT]
UPDATE, the hyphen works with the latest .NET version, feel free to make a pull request.

Apparently .NET wasm tools create some C++ wrapper code for nuget packages, which sadly doesn't handle well packages with hyphen in the name. C++ compiler rightfully complains it's not valid C++ code.

That's one of the reasons why the nuget package isn't used.

For whatever reason I took a screenshot instead of coping & pasting the error and I already changed the setup,

image

stanoddly commented 1 year ago

I reported a bug in dotnet/runtime: https://github.com/dotnet/runtime/issues/78992

stanoddly commented 1 year ago

It was fixed in dotnet/runtime and backported to 7.0x. However I think it will take a while before we see that. The workaround is to either build from dotnet/runtime main or copy Raylib-cs or find some alternatives.

graphnode commented 1 year ago

Seems to have been fixed for dotnet 7.0.400. I removed the Raylib_cs project and added the nuget Raylib-cs 4.2.0.2 and seems to work fine on that version.

stanoddly commented 1 year ago

@graphnode that's great news! If you happen to have the code, please make a pull request with your changes and I will merge it. :heart:

leftbones commented 8 months ago

@graphnode @stanoddly Still no pull request :(

stanoddly commented 8 months ago

@leftbones feel free to make a pull request too. It should be basically just about removing the Raylib-cs project in the repository, replacing it with a nuget package and it should work.

I at least updated the first post's description to highlight it's been updated. EDIT: report -> repository