Skip to content
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

Linux port WIP #811

Merged
merged 4 commits into from
Apr 19, 2016
Merged

Linux port WIP #811

merged 4 commits into from
Apr 19, 2016

Conversation

MAnyKey
Copy link
Contributor

@MAnyKey MAnyKey commented Apr 19, 2016

I decided to do it iteratively instead of having one huge PR. Please, take a look.

@@ -87,16 +87,19 @@ function defaultConfigurations()
configuration "Debug"
targetdir(BINARY_DIR .. "Debug")
defines { "DEBUG" }
buildoptions { "-std=c++11" }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not very sure about these options on MSVS in particular and about build configuration for Linux in general.

Copy link
Owner

Choose a reason for hiding this comment

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

It seems to be ignored on MSVS

Copy link
Owner

@nem0 nem0 Apr 19, 2016

Choose a reason for hiding this comment

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

I tried to compile it with MSVS, and even though -std=c++11 is ignored, it causes warnings

2>cl : Command line warning D9002: ignoring unknown option '-std=c++11'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, I'm removing it for now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@nem0
Copy link
Owner

nem0 commented Apr 19, 2016

I guess it is better to do it iteratively, github has problems with large PR anyway. There is just one issue before this can be merged, see my comment about #include <cstdlib>

@nem0 nem0 merged commit 21d4139 into nem0:master Apr 19, 2016
@nem0
Copy link
Owner

nem0 commented Apr 19, 2016

Good job

@nem0
Copy link
Owner

nem0 commented Apr 21, 2016

#655

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.

2 participants