Continuous Integration about recastnavigation HOT 6 CLOSED

With Travis you can run pre-build scripts to do things like install dependencies. I'm not as familiar with appveyor, but I'd imagine it's similar. Is this something we can do on there as well? If so, we can use it to run premake to generate the sln, rather than keeping it in the repo. We'd also need to set it up to download SDL and unpackage it in the right place.

I'm reluctant to start checking in dependencies and sln files, just because as a platform-independent project, things can get out of sync. I imagine most people are using Windows, so if new dependencies or build settings are tweaked in the sln, there's a chance they wouldn't get carried over to linux or OSX.

from recastnavigation.

Already done in my pull request :) yep, appveyor can do the same. You're
right, we definitely shouldn't have project files checked in.
On 9 Dec 2015 7:37 pm, "Graham Pentheny" [email protected] wrote:

With Travis you can run pre-build scripts to do things like install
dependencies. I'm not as familiar with appveyor, but I'd imagine it's
similar. Is this something we can do on there as well? If so, we can use it
to run premake to generate the sln, rather than keeping it in the repo.
We'd also need to set it up to download SDL and unpackage it in the right
place.

I'm reluctant to start checking in dependencies and sln files, just
because as a platform-independent project, things can get out of sync. I
imagine most people are using Windows, so if new dependencies or build
settings are tweaked in the sln, there's a chance they wouldn't get carried
over to linux or OSX.

—
Reply to this email directly or view it on GitHub
#126 (comment)
.

from recastnavigation.

For reference, my PR is #89

from recastnavigation.

@rwindegger, I'm going to update my PR to remove the prebuilt premake and get that committed both for Travis and AppVeyor, since any CI is better than nothing! When you've separated out what you need to separate out, just replace my AppVeyor config with your own.

from recastnavigation.

@rwindegger - oh sorry, I misunderstood what you said about VS solutions - you'll need to do a little merging actually, shouldn't be hard though!

from recastnavigation.

Merged in with #131

from recastnavigation.