Skip to content
This repository has been archived by the owner on Aug 19, 2018. It is now read-only.

Ci integration #388

Merged
merged 10 commits into from Jun 20, 2017
Merged

Ci integration #388

merged 10 commits into from Jun 20, 2017

Conversation

kf6kjg
Copy link
Contributor

@kf6kjg kf6kjg commented May 14, 2017

How would you like your compilation served? This iteration is just Windows .NET AppVeyor builds and works with runprebuild.bat if it exists, and skips it if it does not.

The version is pulled from the resulting OpenSim.Framework.Servers.dll, so should reflect real-world versioning of Halcyon.

It is set to build on pushes to every branch. This can be restricted to any specific branch or set thereof.

I can also set it up to push the result to a draft GitHub release if it is a tagged commit.

See example build here: https://ci.appveyor.com/project/kf6kjg/halcyon
And see example resulting artifact ZIP at: https://ci.appveyor.com/project/kf6kjg/halcyon/build/artifacts

kf6kjg added 10 commits May 14, 2017 11:50
No deployment rules, just build & package the bin folder.
That's not what one support guy had posted. :/
Not sure there are any ATM.
Choose a syntax and stick with it.
Seems it wants everything to be unique.  Ok.

Also seems to want env vars, ok, so here's the env vars instead of the exampled values.
More specifically, a .NET build, but meh, it'll have a lot of Windows-specific code.  Mono builds will be done elsehow.
@appurist
Copy link
Member

I love seeing progress like this, on the goals that we all know we want to make progress on. CI and CD are clear goals here.

I also like this commit because it is confined to a single file, that has no impact on anything else in the project.

One concern I have is how out of date it could get very quickly with other commits. We had a full nuget-based build in a branch but we had to postpone that merge due to a couple of issues that popped up with limited time. Once the missing prim fix goes out in 0.9.35, I think we need to return to the nuget work before the branches get too far out of sync and we loose that work. At that time, the .sln file will be the "hot" original, and runprebuild will be gone. I think this PR is nicely set up for that being optional so we're good, but any commits related to the build process will need to be aware of this file and the CI process. We can discuss the setup for that separately.

I think this PR is good to go, at least in terms of it not having impact on anything done elsewhere, but may need additional commits in the future as we update the build process (or may not!).

@kf6kjg
Copy link
Contributor Author

kf6kjg commented May 14, 2017

Actually I designed it to seamlessly handle that branch. :) Didn't test, so I'm not 100% confident, but it should.

@appurist
Copy link
Member

"it is confined to a single file, that has no impact on anything else in the project."

Merging. ;)

@appurist appurist merged commit 32a90e2 into IslandzVW:master Jun 20, 2017
@kf6kjg kf6kjg deleted the ci_integration branch July 25, 2017 23:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants