Skip to content

build: add entt, vulkan_sdk and glm to project dependencies - #18

Merged
ripel2 merged 2 commits into
mainfrom
4-project-dependencies
Oct 4, 2024
Merged

build: add entt, vulkan_sdk and glm to project dependencies#18
ripel2 merged 2 commits into
mainfrom
4-project-dependencies

Conversation

@Miou-zora

Copy link
Copy Markdown
Contributor

Linked to:

I've added required dependencies of the issue and some other rules for compilation like define DEBUG when compiling in debug mode or set EngineSquared as the default target.

@Miou-zoraMiou-zora added the enhancement New feature or request label Oct 4, 2024
@Miou-zora
Miou-zora requested a review from a teamOctober 4, 2024 14:04
@Miou-zoraMiou-zora self-assigned this Oct 4, 2024
@Miou-zoraMiou-zora linked an issue Oct 4, 2024 that may be closed by this pull request
Comment threadsrc/hw.cpp
@@ -1,6 +1,20 @@
#include <iostream>
#include <entt/entt.hpp>
#define GLM_ENABLE_EXPERIMENTAL

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what does this do ?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This enable glm::to_string. Without this, we can't "stringify" vectors

@ripel2ripel2 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@ripel2
ripel2 merged commit 7502030 into mainOct 4, 2024
@ripel2
ripel2 deleted the 4-project-dependencies branch October 4, 2024 15:29
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Project dependencies

2 participants

@Miou-zora@ripel2