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

Add .appveyor.yml #219

Merged
merged 3 commits into from
Sep 30, 2017
Merged

Add .appveyor.yml #219

merged 3 commits into from
Sep 30, 2017

Conversation

mbland
Copy link
Owner

@mbland mbland commented Sep 30, 2017

This follows the example from https://www.appveyor.com/docs/lang/ruby/ except that it doesn't need install or before_test steps.

Inspired by bats-core/bats-core#8.

This follows the example from https://www.appveyor.com/docs/lang/ruby/
except that it doesn't need `install` or `before_test` steps.

Inspired by bats-core/bats-core#8.
@mbland mbland added this to the v1.7.0 milestone Sep 30, 2017
@mbland mbland self-assigned this Sep 30, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.145% when pulling c97855a on appveyor into d4d6fc2 on master.

The following test cases failed in the initial build:

  https://ci.appveyor.com/project/mbland/go-script-bash/build/v1.6.0.1

  log/log-command: fatal status for subcommand of command in another
    language
  vars: run perl script; _GO_* variables are exported

The failures were due to `/tmp` getting expanded to
`C:/Users/appveyor/AppData/Local/Temp/1` and `/c` getting expanded to
`C:`. This is probably because running `bash ./go` directly is setting
`BASH` and other variables using the native path format, rather than the
MSYS2 Unix-like translation.

Launching the `./go` script via `bash -c` should fix this, by ensuring
`./go` launches inside a MSYS2 `bash` process that sets the paths to
the Unix-like format.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.145% when pulling 34452a8 on appveyor into d4d6fc2 on master.

The previous commit failed with the same errors as before:

  https://ci.appveyor.com/project/mbland/go-script-bash/build/v1.6.0.3

Upon closer inspection, both of the failing tests executed `perl`.
Indeed, the Appveyor images have Perl installed separately from Git and
available in the PATH:

  https://www.appveyor.com/docs/build-environment/#perl

This Perl is most likely doing the Unix-to-native path translation. This
change attempts to fix the tests by ensuring the MSYS2 Perl that comes
with Git for Windows will get executed instead.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.145% when pulling 17ebdc8 on appveyor into d4d6fc2 on master.

@mbland mbland merged commit 219b2ca into master Sep 30, 2017
@mbland mbland deleted the appveyor branch September 30, 2017 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants