search

patriciogonzalezvivo / glslViewer

Console-based GLSL Sandbox for 2D/3D shaders
BSD 3-Clause "New" or "Revised" License
4.57k stars 352 forks source link

glslViewer does not compile. #342

Closed MubinMuhammad closed 1 year ago

MubinMuhammad commented 1 year ago

Why you all haven't included the 'stdint.h' header file in the 'deps/vera/include/vera/ops/pixel.h' I had do to it by myself.

droshux commented 1 year ago

I also had this issue, I don't know what the pull request permissions are but if I could I might make this fix tonight.

patriciogonzalezvivo commented 1 year ago

Oh, please submit a PR for it. Just for curiosity, what OS are you using?On Jun 8, 2023, at 3:40 AM, droshux @.***> wrote: I also had this issue, I don't know what the pull request permissions are but if I could I might make this fix tonight.

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

MubinMuhammad commented 1 year ago

Ok, I use Arch Linux, and used CMake to compile it.

f-and commented 1 year ago

I'm in the same boat, tried with both the yay install and the wiki instruction and both come to the same error about cstdint.

droshux commented 1 year ago

Oh, please submit a PR for it. Just for curiosity, what OS are you using?On Jun 8, 2023, at 3:40 AM, droshux @.> wrote: I also had this issue, I don't know what the pull request permissions are but if I could I might make this fix tonight. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.>

Hi, sorry for the VERY late reply: There is already a pull request on the vera repository that appears to fix this issue. All checks have passed and there are no conflicts but it doesn't look like it has been merged yet. https://github.com/patriciogonzalezvivo/vera/pull/4

patriciogonzalezvivo commented 1 year ago

Sorry, I miss out that one. Approving that PR and upgrading glslViewer to look at main. Again, Sorry for the delay. Thanks @leo1mml for that PR

patriciogonzalezvivo commented 1 year ago

OK, I just push the change to the latest VERA. Do you mind?

git pull
git submodule update
cd build
make
MubinMuhammad commented 1 year ago

Ok, thank you very much for this change. Really appreciate the response from the owner!