Skip to content

Added premake support for better Visual Studio and other build systems (like Makefile) support.#1456

Open
AashiqMeow12 wants to merge 7 commits into
jbeder:masterfrom
AashiqMeow12:master
Open

Added premake support for better Visual Studio and other build systems (like Makefile) support.#1456
AashiqMeow12 wants to merge 7 commits into
jbeder:masterfrom
AashiqMeow12:master

Conversation

@AashiqMeow12

Copy link
Copy Markdown

No description provided.

AashiqMeow12 and others added 7 commits June 27, 2026 15:38
As otherwise, cannot link this library with other dependencies of Hazel engine.
So that buidling Hazel (which includes this repo as a submodule) does not make git think the submodule has changes to commit.
Note tha the Hazel build system should really be using "out-of-source" builds instead.
Maybe it isn't needed, we will update this in the future,
for now it is okay.

@SGSSGene SGSSGene left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@AashiqMeow12 thank you for your PR.

Could you change everything to two-spaces indentation instead of tabs. This makes it inline with all the other files in this project.

@jbeder : Do we want or are willing to accept more build systems?

Comment thread premake5.lua
kind "StaticLib"
systemversion "latest"
language "C++"
cppdialect "C++17"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

C++11 should be enough as a requirement.

Comment thread .gitignore
Comment on lines +4 to +7
bin
bin-int
yaml-cpp.vcxproj
yaml-cpp.vcxproj.filters No newline at end of file

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please do not change the .gitignore list. I do not think these changes are necessary.

Comment thread premake5.lua

filter "configurations:Dist"
runtime "Release"
optimize "on" No newline at end of file

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I believe there is a new line missing at the end of the file.

Comment thread premake5.lua
{
"src/**.h",
"src/**.cpp",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There are two tabs not are not needed.

@jbeder

jbeder commented Jul 7, 2026

Copy link
Copy Markdown
Owner

I'm pretty sure CMake generates these. If so, we should just defer to CMake. I made the exception for bazel because it really is fundamentally different and I actually like its fundamental difference.

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