schellingb / wajic

WebAssembly JavaScript Interface Creator
zlib License
194 stars 5 forks source link

Fix samples/ path #2

Closed nsauzede closed 2 years ago

schellingb commented 2 years ago

Thanks for catching this! Did you found this by trying it on a non-Windows system? Did everything else work fine? I actually only ever tested it on Windows (also the reason for the spelling mistake...).

nsauzede commented 2 years ago

Yup, on Linux Ubuntu 21.10 The pre-built "./system" didn't work for me Trying to build it manually rn...

nsauzede commented 2 years ago

Thanks for catching this! Did you found this by trying it on a non-Windows system? Did everything else work fine? I actually only ever tested it on Windows (also the reason for the spelling mistake...).

The basic sample worked fine! Which made me want to test the advanced one :-)

nsauzede commented 2 years ago

Yup, on Linux Ubuntu 21.10 The pre-built "./system" didn't work for me Trying to build it manually rn...

Actually I failed building "system/" manually.. See issue https://github.com/schellingb/wajic/issues/1

schellingb commented 2 years ago

Hey there I don't think issue #1 is about system.bc but thank you for posting the error you get in there. I just committed 3f4de6b which should hopefully fix the issues for you. Please check it out and report back. Thanks.

nsauzede commented 2 years ago

Hey there I don't think issue #1 is about system.bc but thank you for posting the error you get in there. I just committed 3f4de6b which should hopefully fix the issues for you. Please check it out and report back. Thanks.

Yes, it fixed the error, thanks! But then I get another error; I created issue #3 and cleaned up issue #1