search

open-dis / open-dis-cpp

C++ implementation of the IEEE-1278.1 Distributed Interactive Simulation (DIS) application protocol v6 and v7
BSD 2-Clause "Simplified" License
92 stars 66 forks source link

CMake Build System #37

Closed rodneyp290 closed 4 years ago

rodneyp290 commented 4 years ago

This is my attempt at a CMake Build for Open DIS C++.

This CMake File (CMakeLists.txt):

I tried to update the README to cover all the step, although I highly recommend someone else try following the instructions before the Pull Request is merged, to make sure they actually makes sense and does skip anything.

During the update I figured out how to Travis-CI to install dependencies with apt, rather than making them from scratch.

leif81 commented 4 years ago

This looks great @rodneyp290

One day we can look at hooking up AppVeyor CI for the Windows build, just like we do with Travis for Linux.

I'm going to try the build on Windows soon. Might not be today, but should be this week.

leif81 commented 4 years ago

This is pretty great @rodneyp290

I get some build errors with OpenDIS7 when doing a full build. Do you know why? I haven't had an opportunity to investigate yet. From what I recall it built with Premake.

rodneyp290 commented 4 years ago

Sorry, been a bit busy this week. I will try have a look on the weekend. Admittedly, I was focusing on OpenDIS 6 for Windows, because I thought 7 didn't build on Windows.

rodneyp290 commented 4 years ago

Hi @leif81, I finally got around to looking at today. I checked out both the master branch, and cmake branches on a MS provided Development Windows VM (Link Below). Building results in what looks like the same 97 error for both OpenDIS7 projects - a bunch of LNK2019 & LNK2001 errors. Is that the same for you?

MS WIndows VM - https://developer.microsoft.com/en-us/windows/downloads/virtual-machines/

leif81 commented 4 years ago

Ok lets merge this and deal with the existing DIS7 build issue on Windows in #38

rodneyp290 commented 4 years ago

Cool thanks for merging. I just figured out why the error and it is is related to #18, I see you've created a issue so I'll continue there, as my current solution to build it is a work around, that may have unintended consequences for users of OpenDIS7. A proper solution will be a bit more involved.