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

Github Actions - Windows #352

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

karbassi
Copy link
Member

@karbassi karbassi commented Aug 3, 2021

Before submitting a pull request, please make sure the following is done:

  • Fork the repository and create your branch from master.
  • If you've added code that should be tested, add tests!
  • Ensure the test suite passes.
  • Format your code with ShellCheck.
  • Include a human-readable description of what the pull request is trying to accomplish.
  • Steps for the reviewer(s) on how they can manually QA the changes.
  • Have a fixes #XX reference to the issue that this pull request fixes.

@karbassi karbassi force-pushed the github-actions/windows-test branch from bae9a1d to 921b148 Compare August 4, 2021 17:06
@karbassi karbassi force-pushed the github-actions/windows-test branch from 921b148 to d72054d Compare August 4, 2021 17:14
@karbassi
Copy link
Member Author

karbassi commented Aug 4, 2021

I don't have a windows device or vm. Can someone update the tests to make sure they run?

Also, do we want to support Windows?

@inkarkat @ginatrapani

@inkarkat
Copy link
Member

inkarkat commented Aug 4, 2021

So it's just some completion tests that fail on Windows? It should be possible to fix that.

Years ago, I've also used todo.sh on Windows / Cygwin, and I still remember that the tests succeeded back then. Extending the test coverage to Windows would be great; I think some folks have installed Cygwin just for that.

I also don't have easy access to a Windows + Cygwin system any longer; maybe I'll still find some system.

@inkarkat
Copy link
Member

inkarkat commented Aug 6, 2021

I was able to run the tests on a Microsoft Windows Server 2018 system (so actually quite close to the Windows 2019 used by the CI), with Cygwin 64-bit. All tests succeeded!

Looking more closely, I think the CI is just using the Bash that's provided with Git, and the corresponding mingw32 tools (unfortunately, Bash doesn't identify itself, but I see mingw32-make).

For me, in the past, todo.sh on Windows always meant Cygwin. I'm not sure whether there's a Windows + Cygwin system available for CI; bootstrapping Cygwin each time doesn't sound attractive. On the other hand, mingw32 might be sufficient to run todo.sh (maybe with some adaptations to the tests); I think I can obtain another Windows VM with Git Bash; will check on Monday.

@karbassi
Copy link
Member Author

karbassi commented Aug 6, 2021

windows-latest is Microsoft Windows Server 2019 with the following installed. https://github.com/actions/virtual-environments/blob/main/images/win/Windows2019-Readme.md

Bash 4.4.23(1)-release

Looking at the test for make test, it looks like it is running on PowerShell. https://github.com/todotxt/todo.txt-cli/pull/352/checks?check_run_id=3244083563#step:5:3

> make test
 shell: C:\Program Files\PowerShell\7\pwsh.EXE -command ". '{0}'"

@inkarkat inkarkat removed their assignment Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants