Skip to content

Add stack file and fix build #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TristanCacqueray
Copy link

This change adds a stack file to build the project with lts-11.22 and
a compatible vulkan-api version.

This change also fixes the defaultWindow record for VulkanContext and
update the shader path to be relative.

This change adds a stack file to build the project with lts-11.22 and
a compatible vulkan-api version.

This change also fixes the defaultWindow record for VulkanContext and
update the shader path to be relative.
@TristanCacqueray
Copy link
Author

Not sure if that is correct, the extra deps versions was picked randomly until the project built. In particular, I couldn't find the library that provided the SDL.windowVulkan defaultWindow constructor. This PR let me build and run the project using stack build.

@dpwiz
Copy link

dpwiz commented May 1, 2020

I'm too using SDL.windowGraphicsContext = SDL.VulkanContext for this.

@dpwiz
Copy link

dpwiz commented May 1, 2020

BTW, I can run it with LTS-14. Pushing into LTS-15 should require only minor tweaks here and there.

@ocharles
Copy link
Owner

ocharles commented May 1, 2020

Thanks for the feedback folks! I'm trying to keep all dev on this strictly on stream, and will leave the PR for that too. Maybe this is motivation to crack on with the project again! 😆

@TristanCacqueray
Copy link
Author

Thank you for putting that project together @ocharles, it's very instructive. I find nix difficult to use for OpenGL outside of nixos, even with nixgl, thus having a cabal or stack build would be even better :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants