Initial Cake Build Script #898

gep13 · 2016-06-09T12:46:07Z

Lets see how well this works
Thrown together over lunch
Opening this as a point of discussion

gep13 · 2016-06-09T12:54:07Z

Things that still need to be done:

Complete all publishing steps
Upload NUnit Results to AppVeyor
Decide what we are doing with ReleaseNotes
NuGet Packaging
Chocolatey Packaging
Publishing VSTS Extension to Marketplace (@pascalberger needs to describe how)

robdmoore · 2016-06-09T12:54:54Z

Is it wrong of me to prefer what was there before since it's so much more succinct/understandable and has less rigmarole? #controversial

JakeGinnivan · 2016-06-09T12:59:24Z

@robdmoore we are moving the build logic into build scripts. At the moment the release process is manual and AppVeyor releases can't be automated more.

asbjornu · 2016-06-09T13:31:17Z

@gep13 @JakeGinnivan ❤️ This is awesome! What are your current thoughts regarding release notes?

pascalberger · 2016-06-09T13:35:27Z

Can we also integrate publishing the VSTS Extension to the Marketplace?

gep13 · 2016-06-09T14:03:55Z

@pascalberger said...
Can we also integrate publishing the VSTS Extension to the Marketplace?

Yes, assuming you can tell me what is required, then we can definitely get this added in 😄

gep13 · 2016-06-09T14:08:27Z

@asbjornu said...
What are your current thoughts regarding release notes?

The current thought process is the following...

A GitHub Release will be created, either using GitReleaseNotes or GitReleaseManager
This will be marked as draft, until the release is ready to go
Once everyone is happy, the Release will be switched from Draft to Published, which will have the following effect
- A tag will be added to the Repository
- A new build will be triggered on AppVeyor, which will then start doing the publishing cycle
GitReleaseManager will then be used to export the notes from the GitHub release, which will then be added into the NuGet and Chocolatey packages.

asbjornu · 2016-06-09T14:10:10Z

@gep13 Sounds good! 👍 from me 😄

eatdrinksleepcode · 2016-06-09T17:31:05Z

From my very recent experience finding and copying commands out of the build server log in order to reproduce the build locally, I wholeheartedly support the introduction of a build script :)

I am curious: is Cake already decided on, or is it just a current contender? Cake is great; I am implementing it at work right now and overall find it very useful. But I have selfish reasons for proposing something else if people are open to it...

One thing that I notice is that this build script removes the existing dogfooding aspect of the build. The current Windows build actually dogfoods itself by building, then running GitVersion.exe, then building again. This makes the build time longer, but it also serves as an extra layer of testing of the packaging itself (specifically the CommandLine package) within the build. By invoking GitVersion at the start of the build script using a pre-existing package, that dogfooding behavior is lost.

gep13 · 2016-06-09T18:06:56Z

@eatdrinksleepcode said...
But I have selfish reasons for proposing something else if people are open to it...

I spoke with Jake while we are both here at NDC, and we have pretty much decided to go with Cake. I would be curious as to what other suggestions you would have though.

@eatdrinksleepcode said...
By invoking GitVersion at the start of the build script using a pre-existing package, that dogfooding behavior is lost.

Yes, we are aware of this. Cake allows us easily to switch from using the latest version on NuGet to a version built locally. This first pass at a Cake Build Script is literally just that. Something that was cooked up over lunch while we are both here. We will likely fine tune this over the next couple weeks until it does everything the previous script did, and more.

JakeGinnivan · 2016-06-14T15:43:48Z

So I am playing with a crazy cake/appveyor setup for Shouldly to get a bit more of a deployment pipeline. You can see it at shouldly/shouldly@80554d6

The flow is:

CI Build doesn't build tags and uploads all artifacts to AppVeyor. Also includes
- A .artifacts which is just a keyvalue pair to lookup the filename for specific packages. i.e nuget => Shouldly.2.8.0-beta1.nupkg to save regex on the other side
- releasenotes.md - annoyingly there is no way to push this into GitHub releases through AppVeyor atm, have opened some support requests to enable it :)
Deploy to github release as a draft through the appveyor UI
Copy releasenotes.md into the github release, fix tag and do any manual fixes to release notes
Publish GitHub release
A Shouldly Deploy build will build the tag, calling deploy.ps1/deploy.cake.
- Build script will find the github release with that tag, download all the artifacts to a local folder. Then it can do the release of all those artifacts as normal

And we don't have to rebuild any artifacts, yay :) What do you think of that process, something which would work here?

asbjornu · 2016-06-15T08:21:17Z

@JakeGinnivan If that's the only way to do it, I'd say 👍. But isn't it possible to avoid the manual release step, for example by invoking that in CI somehow only when the commit being built is tagged (with a valid version number, if that's possible to verify)?

asbjornu · 2016-06-28T12:24:31Z

build.cake

+{
+    if(IsRunningOnUnix())
+    {
+        XBuild("./Source/Gep13.Cake.Sample.WebApplication.sln", new XBuildSettings()


Why not build ./src/GitVersion.sln?

- Lets see how well this works - Thrown together over lunch - Opening this as a point of discussion

…project folder)

…no docs warning in release build

… AppVeyor

JakeGinnivan · 2016-07-09T07:57:58Z

Am uploading the test results but they don't seem to be picked up. Not too fussed, can look into that later. Do you have any ideas @gep13?

gep13 force-pushed the CakeBuild branch from d7f05d7 to 190fd8d Compare June 9, 2016 19:01

asbjornu reviewed Jun 28, 2016
View reviewed changes

JakeGinnivan force-pushed the CakeBuild branch from 6d63ea2 to 7bedd29 Compare July 2, 2016 02:04

asbjornu mentioned this pull request Jul 7, 2016

Can't find the .git directory in C:\TeamCity\buildAgent\work\d8b51bbe303335d5 #926

Closed

gep13 and others added 14 commits July 9, 2016 11:05

Initial Cake Build Script

aaf37c5

- Lets see how well this works - Thrown together over lunch - Opening this as a point of discussion

Dogfood in build and generate release notes

c0ea9d9

Fixed few issues

c39d96f

Use default target

522e310

Some targets changes

942c86b

Fixed target ordering issue

92d538c

Stopped tests being duplicated (GitVersion.Core existed in each test …

3bd0c2e

…project folder)

Stop tests being run by appveyor as well as the build script, remove …

596daec

…no docs warning in release build

Make sure artifacts get built

f4d620f

Make sure tfx is installed before build and only produce artifacts on…

6d26820

… AppVeyor

Pass more variables

59075bc

Upload gem

e27ed30

Setup deploy script

f1c37e0

Fix mono cake build issue

65a4b34

JakeGinnivan added 4 commits July 9, 2016 11:05

Use dynamic repos on travis

86aab0f

Trying to get travis builds to pass

ed74139

Exclude no-mono tests again

e59a8d6

Missing brace

b6fa048

JakeGinnivan force-pushed the CakeBuild branch from df18bd4 to b6fa048 Compare July 9, 2016 03:14

JakeGinnivan added 3 commits July 9, 2016 11:15

Don't need travis workaround anymore

8385b54

Don't need dynamic repos any more on Travis

dd98bde

Upload test results to appveyor

a67533c

JakeGinnivan force-pushed the CakeBuild branch from f6fd5bb to a67533c Compare July 9, 2016 07:37

Added additional logging

249be69

JakeGinnivan merged commit 2e3d5e5 into master Jul 9, 2016

JakeGinnivan deleted the CakeBuild branch July 9, 2016 07:59

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Initial Cake Build Script #898

Initial Cake Build Script #898

Uh oh!

gep13 commented Jun 9, 2016

Uh oh!

gep13 commented Jun 9, 2016 •

edited by JakeGinnivan

Loading

Uh oh!

robdmoore commented Jun 9, 2016

Uh oh!

JakeGinnivan commented Jun 9, 2016

Uh oh!

asbjornu commented Jun 9, 2016

Uh oh!

pascalberger commented Jun 9, 2016

Uh oh!

gep13 commented Jun 9, 2016

Uh oh!

gep13 commented Jun 9, 2016

Uh oh!

asbjornu commented Jun 9, 2016

Uh oh!

eatdrinksleepcode commented Jun 9, 2016

Uh oh!

gep13 commented Jun 9, 2016

Uh oh!

JakeGinnivan commented Jun 14, 2016

Uh oh!

asbjornu commented Jun 15, 2016

Uh oh!

asbjornu Jun 28, 2016

Uh oh!

JakeGinnivan commented Jul 9, 2016

Uh oh!

Uh oh!

Initial Cake Build Script #898

Initial Cake Build Script #898

Uh oh!

Conversation

gep13 commented Jun 9, 2016

Uh oh!

gep13 commented Jun 9, 2016 • edited by JakeGinnivan Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

robdmoore commented Jun 9, 2016

Uh oh!

JakeGinnivan commented Jun 9, 2016

Uh oh!

asbjornu commented Jun 9, 2016

Uh oh!

pascalberger commented Jun 9, 2016

Uh oh!

gep13 commented Jun 9, 2016

Uh oh!

gep13 commented Jun 9, 2016

Uh oh!

asbjornu commented Jun 9, 2016

Uh oh!

eatdrinksleepcode commented Jun 9, 2016

Uh oh!

gep13 commented Jun 9, 2016

Uh oh!

JakeGinnivan commented Jun 14, 2016

Uh oh!

asbjornu commented Jun 15, 2016

Uh oh!

asbjornu Jun 28, 2016

Choose a reason for hiding this comment

Uh oh!

JakeGinnivan commented Jul 9, 2016

Uh oh!

Uh oh!

gep13 commented Jun 9, 2016 •

edited by JakeGinnivan

Loading