Skip to content

Commit

Permalink
[2.0>master] [1.4>2.0] [MERGE #2933 @dilijev] Fix #962: Copyright che…
Browse files Browse the repository at this point in the history
…ck versus target branch; Fix #2928: use Ubuntu 16.04 for style checks

Merge pull request #2933 from dilijev:jenkins-fixes

Replaces #2927

Fix #962: Copyright Check should check versus target branch instead of master.

See tests of the functionality in the following PRs:

- #2896 (release/1.4-ci)
- #2924 (release/1.4 test output comparison)
- #2925 (release/2.0-ci)
- #2926 (release/2.0 test output comparison)

Fix #2928: Update Style Checks to use Ubuntu16.04.

- Tested in #2929

Fixes #962
Fixes #2928
  • Loading branch information
dilijev committed May 10, 2017
2 parents 0a347b5 + 7a170f3 commit cafe937
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion jenkins/check_copyright.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ rm -f $ERRFILE
rm -f $ERRFILETEMP

echo "Check Copyright > Begin Checking..."
git diff --name-only `git merge-base origin/master HEAD` HEAD |
git diff --name-only `git merge-base origin/$ghprbTargetBranch HEAD` HEAD |
grep -v -E '\.git.*' |
grep -v -E '\.xml$' |
grep -v -E '\.props$' |
Expand Down
3 changes: 1 addition & 2 deletions netci.groovy
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ def msbuildTypeMap = [
def machineTypeToOSTagMap = [
'Windows 7': 'Windows 7', // Windows Server 2008 R2, equivalent to Windows 7
'Windows_NT': 'Windows', // Windows Server 2012 R2, equivalent to Windows 8.1 (aka Blue)
'Ubuntu14.04': 'Ubuntu14.04',
'Ubuntu16.04': 'Ubuntu',
'OSX': 'OSX'
]
Expand Down Expand Up @@ -215,7 +214,7 @@ def CreateStyleCheckTasks = { taskString, taskName, checkName ->
Utilities.addGithubPushTrigger(newJob)
}

Utilities.setMachineAffinity(newJob, 'Ubuntu14.04', 'latest-or-auto')
Utilities.setMachineAffinity(newJob, 'Ubuntu16.04', 'latest-or-auto')
}
}

Expand Down

0 comments on commit cafe937

Please sign in to comment.