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

Try to add jenkins build of 'net40' without VS installed #1724

Merged
merged 1 commit into from
Nov 8, 2016

Conversation

dsyme
Copy link
Contributor

@dsyme dsyme commented Nov 8, 2016

#1723 fixes the build of 'net40' without VS installed.

This PR attempts to add a CI build of 'net40' without VS installed by using the Jenkins affinity spec latest-or-auto instead of latest-dev15. No tests are run, just build.

Because of the way jenkins triggers work, we wil have to accept this PR before it can be tested.

It might be that we have other dependencies (e.g. Windows SDK for sure) but this will help flush out what they are.

@dsyme dsyme changed the title Try to add junkins build of 'net40' without VS installed Try to add jenkins build of 'net40' without VS installed Nov 8, 2016
@@ -68,7 +71,8 @@ def static getBuildJobName(def configuration, def os) {
// TODO: set to false after tests are fully enabled
def skipIfNoTestFiles = true

Utilities.setMachineAffinity(newJob, os, os == 'Windows_NT' ? 'latest-dev15' : 'latest-or-auto')
def affinity = configuration == 'Release_net40_no_vs' ? 'latest-or-auto' : (os == 'Windows_NT' ? 'latest-dev15' : 'latest-or-auto')
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 have no idea where to find out what gets installed with latest-or-auto or latest-dev15 etc., or what the latest allowable values are I've searched around for Utilities.setMachineAffinity but can't find the source for those utilites

@dsyme dsyme merged commit 4160e3f into master Nov 8, 2016
dsyme added a commit that referenced this pull request Nov 8, 2016
@dsyme dsyme deleted the dsyme-patch-3 branch November 8, 2016 23:01
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