search

Unidata / UDUNITS-2

API and utility for arithmetic manipulation of units of physical quantities
http://www.unidata.ucar.edu/software/udunits
Other
62 stars 36 forks source link

Add AppVeyor CI #47

Closed ocefpaf closed 7 years ago

ocefpaf commented 7 years ago

This PR adds Continuous integration tests for Windows. Right now it only test the build itself and if the commands udunits2 -h and udunits2 -H meter -W miles works. We can add more tests later.

In the PR we use conda to acquire the dependency: expat, and the build tools: cmake and the various Visual Studio versions: vc 9, 10, and 14.

The devs need to activate AppVeyor for this repository to activate the tests. Here are the logs from my account: https://ci.appveyor.com/project/ocefpaf/udunits-2

PS: this PR is similar to https://github.com/Unidata/netcdf-c/pull/347.

DPeterK commented 7 years ago

Nice one @ocefpaf! :grinning: :+1:

semmerson commented 7 years ago

@ocefpaf Apparently, an upgrade at AppVeyer has broken this pull-request. You can see this by trying to rebuild the last successful build (1.0.7).

Can you fix it?

ocefpaf commented 7 years ago

Sure. I am out office until the end of the month but I'll take a look as soon as I get back.

On May 18, 2017 11:37 AM, "Steven Emmerson" notifications@github.com wrote:

@ocefpaf https://github.com/ocefpaf Apparently, an upgrade at AppVeyer has broken this pull-request. You can see this by trying to rebuild the last successful build (1.0.7).

Can you fix it?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Unidata/UDUNITS-2/pull/47#issuecomment-302504811, or mute the thread https://github.com/notifications/unsubscribe-auth/AA6BL3M-Euv3U0vRfIGdLMQvBP6QaDTRks5r7I_wgaJpZM4LrUCe .

ocefpaf commented 7 years ago

@semmerson can you send me a link to the build that is broken? Was that in a PR?

semmerson commented 7 years ago

@ocefpaf All the recent builds have failed. Here's a link.