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

Travis: Run tests on Windows. #109

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
1 participant
@dmitshur
Contributor

dmitshur commented Oct 13, 2018

There has been work done to the library over time to support Windows.
There's a chance for Windows support to deteriorate over time, since
it's not tested often. Fix that by running tests on Windows in CI.
Travis CI has recently added support for Windows, making this possible.

References:

Travis: Run tests on Windows.
There has been work done to the library over time to support Windows.
There's a chance for Windows support to deteriorate over time, since
it's not tested often. Fix that by running tests on Windows in CI.
Travis CI has recently added support for Windows, making this possible.

Reference: https://blog.travis-ci.com/2018-10-11-windows-early-release.
@dmitshur

This comment has been minimized.

Show comment
Hide comment
@dmitshur

dmitshur Oct 13, 2018

Contributor

Looks like it'll take some work to get tests to pass on Windows. I see the following issues so far:

  • gofmt reports every file as not formatted; this is likely because the line endings of .go files being checked out are converted to from \n to \r\n. This shouldn't be done.
  • On Go 1.11.1, ./vcs/... packages are failing because of exec: "gcc": executable file not found in %PATH%.
    • Not sure why this is happening in Go 1.11.1 build but not Go 1.9.7 build.
  • On Go 1.9.7, gcc is not an issue and packages are building, but an SSH test in ./vcs fails because a key file fail to be deleted: Error removing SSH key file C:\Users\travis\AppData\Local\Temp\go-vcs-gitcmd-key385111935: remove C:\Users\travis\AppData\Local\Temp\go-vcs-gitcmd-key385111935: The process cannot access the file because it is being used by another process..
  • On master, gimme fails to install Go.
Contributor

dmitshur commented Oct 13, 2018

Looks like it'll take some work to get tests to pass on Windows. I see the following issues so far:

  • gofmt reports every file as not formatted; this is likely because the line endings of .go files being checked out are converted to from \n to \r\n. This shouldn't be done.
  • On Go 1.11.1, ./vcs/... packages are failing because of exec: "gcc": executable file not found in %PATH%.
    • Not sure why this is happening in Go 1.11.1 build but not Go 1.9.7 build.
  • On Go 1.9.7, gcc is not an issue and packages are building, but an SSH test in ./vcs fails because a key file fail to be deleted: Error removing SSH key file C:\Users\travis\AppData\Local\Temp\go-vcs-gitcmd-key385111935: remove C:\Users\travis\AppData\Local\Temp\go-vcs-gitcmd-key385111935: The process cannot access the file because it is being used by another process..
  • On master, gimme fails to install Go.
@dmitshur

This comment has been minimized.

Show comment
Hide comment
@dmitshur

dmitshur Oct 13, 2018

Contributor

Looks like @keegancsmith is seeing very similar issues in sourcegraph/go-langserver#329, and has filed an issue about them at https://travis-ci.community/t/go-cant-find-gcc-with-go1-11-1-on-windows/293. This PR can wait until that's resolved.

Contributor

dmitshur commented Oct 13, 2018

Looks like @keegancsmith is seeing very similar issues in sourcegraph/go-langserver#329, and has filed an issue about them at https://travis-ci.community/t/go-cant-find-gcc-with-go1-11-1-on-windows/293. This PR can wait until that's resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment