Page MenuHomeWildfire Games

Windows Differential testing
ClosedPublic

Authored by Silier on Jun 2 2019, 9:25 PM.

Details

Summary

adding testing of diffs on windows

To make it work with bat command, need to add this to jenkins paths:

patch: C:\Program Files\Git\usr\bin

or where patch.exe is located

Test Plan

..

Diff Detail

Repository
rP 0 A.D. Public Repository
Branch
/ps/trunk
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 7750
Build 12622: Vulcan Build
Build 12621: arc lint + arc unit

Event Timeline

Silier created this revision.Jun 2 2019, 9:25 PM
Silier created this object with visibility "Custom Policy".
Owners added a subscriber: Restricted Owners Package.Jun 2 2019, 9:25 PM
Silier updated this revision to Diff 8289.Jun 3 2019, 11:57 AM
Silier edited the summary of this revision. (Show Details)

sh arc patch -> bat arc patch

Silier updated this revision to Diff 8290.Jun 3 2019, 1:20 PM

line 37

Itms added a comment.Jun 29 2019, 11:05 AM

I experimentally added this pipeline to Jenkins with the fix to arc patch, and made Phabricator run it on differentials. We'll see if there are unexpected issues. In the meantime, a few comments...

build/jenkins/win_diff.Jenkinsfile
35

This is not correct. The DIFF_ID sent by Phabricator is not the ID of the revision, but the one of the diff. For instance, the diff I'm commenting on is diff 8290. The two previous diffs for this revision (D1946) were 8282 and 8289.

The correct call is

arc patch --diff ${params.DIFF_ID} --force
39

same

46

I'd be in favor of building Atlas. Let's make sure it builds, without the same bug as the autobuilder.

59

Is there a reason you used unsuccessful instead of failure?

Silier added inline comments.Jun 29 2019, 12:37 PM
build/jenkins/win_diff.Jenkinsfile
35

Ok, I see my misstake

46

Do you mean to always build with --atlas and /t:AtlasUI ?

Itms added inline comments.Jun 29 2019, 6:48 PM
build/jenkins/win_diff.Jenkinsfile
46

Yes :)

Silier updated this revision to Diff 8654.Jun 30 2019, 3:04 PM
Silier retitled this revision from Windows Differential testing [WIP] to Windows Differential testing.

fix patch
add debug
add atlas

Silier updated this revision to Diff 9782.Sep 15 2019, 1:31 PM

add flags for gloox and collada
split testing so release tests follow right after build of release version so in case they do not pass, there will be no build of debug version

Silier changed the visibility from "Custom Policy" to "Public (No Login Required)".Sep 15 2019, 1:34 PM
Stan added a subscriber: Stan.Sep 15 2019, 3:27 PM

Does it support stacked patches ?

Stan added a comment.Sep 15 2019, 5:48 PM

@Angen stacked patches are patches that are dependant. Ex Patch A requires patch B to be applied. Then B is stacked on top of A.

no, but might be interesting improvement

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/186/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/733/display/redirect

Silier updated this revision to Diff 9822.Sep 17 2019, 9:06 PM

robocopy

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/734/display/redirect

Silier updated this revision to Diff 9831.Sep 17 2019, 10:32 PM

add timeout
fix robocopy

Silier planned changes to this revision.Sep 17 2019, 10:33 PM

try to fail after timeout

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/229/display/redirect

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/740/display/redirect

Silier added inline comments.Sep 18 2019, 7:28 AM
build/jenkins/win_diff.Jenkinsfile
14

only build

41

do not try to delete removed items by patch

115

do not try to delete removed items by patch

Silier requested review of this revision.Nov 12 2019, 6:55 PM
Silier planned changes to this revision.Dec 1 2019, 6:49 PM

what i said above

Silier updated this revision to Diff 10454.EditedDec 1 2019, 8:42 PM

moved to subdirectory
moving revert ahead of deletions so ugly spamming like in https://jenkins.wildfiregames.com/blue/organizations/jenkins/vs2015-differential/detail/vs2015-differential/604/pipeline will not happen

Vulcan added a comment.Dec 1 2019, 8:43 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/666/display/redirect

Vulcan added a comment.Dec 1 2019, 8:49 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1182/display/redirect

Silier updated this revision to Diff 10518.Dec 7 2019, 6:53 PM

do not delete dlls
allways pass
--build-shared-glooxwrapper

Vulcan added a comment.Dec 7 2019, 6:56 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/709/display/redirect

Vulcan added a comment.Dec 7 2019, 7:05 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1225/display/redirect

Itms accepted this revision.Dec 16 2019, 10:34 PM

I'm committing this with a license header and some whitespace fixes. Thanks for all the work!

This revision is now accepted and ready to land.Dec 16 2019, 10:34 PM