Skip to content

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