Add AppVeyor.yml for Windows contious integration #1292

BurtHarris · 2016-09-26T00:50:37Z

This adds a simple configuration file to the root directory which enables use of appveyor.com for Windows-based integration testing. See a sample build from my fork at https://ci.appveyor.com/project/BurtHarris/antlr4

BurtHarris · 2016-09-26T00:53:18Z

@parrt, I think you asked for integration with AppVeyor.com.

It was easier than I thought. While I don't understand all the log spew, it seems like a step in the right direction.

You'll need to configure an account on AppVeyor.com pointing to the official repository, but that also seems easy.

KvanTTT · 2016-09-26T08:52:31Z

Can you try to use verbosity: minimal in build section option to decrease log size?
Add build status badge to README.

BurtHarris · 2016-09-26T15:30:40Z

@KvanTTT, I'm not sure where you are suggesting I would add the verbosity: minimal, but it sounds like a very good idea. It appears adding a -q to the mvn command line can reduce the spew. The new output seems like an improvement, but still leaves unclear why riving the majority of the lines from testing are needed..

I think we want the badge to reflect the builds on the official fork rather than mine, so I don't think I can propose an edit for this until that's set-up.

KvanTTT · 2016-09-26T15:36:59Z

build:
  verbosity: quiet|minimal|normal|detailed

See also an official appveyor.yml reference. Maybe it does not affect result log because here is not a MSBuild system.

BurtHarris · 2016-09-26T15:38:41Z

@KvanTTT, adding verbosity:minimal to appveyor.yml doesn't have any effect on the log spew.

That seems to be because it's maven & the antlr4 runtime-testsuite are controlling all the output. The integration approach used by runtime-testsuite with the target-speific languages, c# in particular, is generating much of the output by building and running the tests one-at-a-time rather than in a large batch (which would run much faster.) @ericvergnaud seem to know the most about these tests, perhaps he can review confirm: https://ci.appveyor.com/project/BurtHarris/antlr4/build/4.5.4-SNAPSHOT+AppVeyor.21

sharwell · 2016-09-26T19:11:01Z

appveyor.yml

@@ -0,0 +1,9 @@
+version: '4.5.4-SNAPSHOT+AppVeyor.{build}'


💡 This will quickly become incorrect. You can improve on this by moving the version handling to the build script like this. To determine the specific build number for the call to Update-AppveyorBuild, you can extract it from the pom.xml file.

sharwell · 2016-09-26T19:15:50Z

appveyor.yml

+version: '4.5.4-SNAPSHOT+AppVeyor.{build}'
+os: Windows Server 2012
+build_script:
+  - mvn compile -q --batch-mode 


❓ Does AppVeyor come with Maven installed? If not, you can install it using Chocolatey like this followed by setting M2_HOME like this.

@sharwell, yes, Maven is available in AppVeyor.

sharwell · 2016-09-26T19:16:28Z

appveyor.yml

+  - mvn install -q -Dantlr-python2-python="C:\Python27\python.exe" -Dantlr-python3-python="C:\Python35\python.exe" -Dantlr-javascript-nodejs="C:\Program Files (x86)\nodejs\node.exe" --batch-mode
+build:
+  verbosity: minimal
+


💡 I would prefer to see the file end with a trailing newline character

sharwell · 2016-09-26T19:18:27Z

appveyor.yml

+build_script:
+  - mvn compile -q --batch-mode 
+test_script:
+  - mvn install -q -Dantlr-python2-python="C:\Python27\python.exe" -Dantlr-python3-python="C:\Python35\python.exe" -Dantlr-javascript-nodejs="C:\Program Files (x86)\nodejs\node.exe" --batch-mode


💭 I have no experience with the dependencies listed here (w.r.t. AppVeyor). Do you know if they come pre-installed on AppVeyor?

All AppVeyor preinstalled tools listed here: Build Worker installed software.

from last line of appvayor.yml

BurtHarris · 2016-09-26T22:41:03Z

Thank you @sharwell & @KvanTTT I've pushed a change to eliminate the trailing whitespace on the last line. I found Maven was already installed on the default appveyor worker image part way through working on this. How convinent. 👍

parrt · 2016-09-27T18:27:33Z

hiya. so does this correctly build antlr on a windows machine? that would be nice.

BurtHarris · 2016-09-27T20:12:27Z

Yes, it seems to. The tests say they are passing, but I'd still like an expert to review.
It works well on appveyor because their worker machines have all the prerequisites pre-installed.

parrt · 2016-11-21T19:37:32Z

@BurtHarris How would i know if this stuff works? Do i need to become member of AppVeyor etc...? How can I test?

KvanTTT · 2016-11-21T21:19:42Z

@parrt, yes, you should sign up to AppVeyor, link your account with GitHub account, select ANTLR repository and setup build config manually via Web UI or add appveyor.yml file to the repository. After it you will be able to build ANTLR project and run unit-tests as you can do it with Travis-CI.

parrt · 2016-11-21T22:37:53Z

@KvanTTT ok, created account. Will investigate

KvanTTT · 2016-11-23T23:04:18Z

I also suggest to add AppVeyor build status on README page.

parrt · 2016-11-25T20:18:44Z

added badge.

Appveyor w/ semantic version

d904711

BurtHarris mentioned this pull request Sep 26, 2016

Runtime-tests prerequsites should be clarified #1269

Closed

appveyor.yml cpaitialization

7e2213d

Adding -q to mvn command line for appveyor

e71ebef

Addomg verbosity: minimal fpr appveyor

7ebfabc

sharwell reviewed Sep 26, 2016

View reviewed changes

eliminte trailing whitespace

80a7589

from last line of appvayor.yml

parrt added this to the 4.6 milestone Nov 21, 2016

parrt added comp:build cross-platform-issue labels Nov 21, 2016

parrt merged commit 6818011 into antlr:master Nov 21, 2016

BurtHarris deleted the addAppveyor branch March 6, 2017 21:23

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add AppVeyor.yml for Windows contious integration #1292

Add AppVeyor.yml for Windows contious integration #1292

BurtHarris commented Sep 26, 2016 •

edited

Loading

BurtHarris commented Sep 26, 2016

KvanTTT commented Sep 26, 2016

BurtHarris commented Sep 26, 2016 •

edited

Loading

KvanTTT commented Sep 26, 2016 •

edited

Loading

BurtHarris commented Sep 26, 2016 •

edited

Loading

sharwell Sep 26, 2016

sharwell Sep 26, 2016

KvanTTT Sep 26, 2016 •

edited

Loading

sharwell Sep 26, 2016

sharwell Sep 26, 2016

KvanTTT Sep 26, 2016

BurtHarris commented Sep 26, 2016

parrt commented Sep 27, 2016

BurtHarris commented Sep 27, 2016 •

edited

Loading

parrt commented Nov 21, 2016

KvanTTT commented Nov 21, 2016

parrt commented Nov 21, 2016

KvanTTT commented Nov 23, 2016

parrt commented Nov 25, 2016

Add AppVeyor.yml for Windows contious integration #1292

Add AppVeyor.yml for Windows contious integration #1292

Conversation

BurtHarris commented Sep 26, 2016 • edited Loading

BurtHarris commented Sep 26, 2016

KvanTTT commented Sep 26, 2016

BurtHarris commented Sep 26, 2016 • edited Loading

KvanTTT commented Sep 26, 2016 • edited Loading

BurtHarris commented Sep 26, 2016 • edited Loading

sharwell Sep 26, 2016

Choose a reason for hiding this comment

sharwell Sep 26, 2016

Choose a reason for hiding this comment

KvanTTT Sep 26, 2016 • edited Loading

Choose a reason for hiding this comment

sharwell Sep 26, 2016

Choose a reason for hiding this comment

sharwell Sep 26, 2016

Choose a reason for hiding this comment

KvanTTT Sep 26, 2016

Choose a reason for hiding this comment

BurtHarris commented Sep 26, 2016

parrt commented Sep 27, 2016

BurtHarris commented Sep 27, 2016 • edited Loading

parrt commented Nov 21, 2016

KvanTTT commented Nov 21, 2016

parrt commented Nov 21, 2016

KvanTTT commented Nov 23, 2016

parrt commented Nov 25, 2016

BurtHarris commented Sep 26, 2016 •

edited

Loading

BurtHarris commented Sep 26, 2016 •

edited

Loading

KvanTTT commented Sep 26, 2016 •

edited

Loading

BurtHarris commented Sep 26, 2016 •

edited

Loading

KvanTTT Sep 26, 2016 •

edited

Loading

BurtHarris commented Sep 27, 2016 •

edited

Loading