Add Windows support by Gaweringo · Pull Request #58 · Zeioth/compiler.nvim · GitHub
Skip to content

Add Windows support - #58

Open
Gaweringo wants to merge 17 commits into
Zeioth:mainfrom
Gaweringo:windows-support
Open

Add Windows support#58
Gaweringo wants to merge 17 commits into
Zeioth:mainfrom
Gaweringo:windows-support

Conversation

@Gaweringo

Copy link
Copy Markdown
Contributor

This PR would make all commands which use rm -f and mkdir -p windows compatible by adding the get_commands() function to utils which swaps the unix commands with the cmd commands erase /s /q and mkdir.
The way to ignore errors is also different and is changed from || true to > nul 2> nul & cd. for cmd. Ignoring errors is also needed for mkdir in cmd, since there is no -p equivalent.

I could not test the asm changes fully yet, since it uses linux/unix syscalls from what I understand. Building and linking works, but the "Run program" does not succeed and exits with a large negative exit code.

Languages which had changes:

  • asm
  • c
  • cpp
  • cs
  • dart
  • go
  • java
  • kotlin
  • python
  • rust
  • swift
  • fortran

Bau which had changes:

  • cmake
  • meson

From my testing, everything should still work on Linux with these changes.

Let me know if I need to change something, or if this PR might go in the wrong direction / uses an undesired approach.

@Zeioth

Zeioth commented Sep 24, 2024

Copy link
Copy Markdown
Owner

@Gaweringo

Gaweringo commented Sep 24, 2024

Copy link
Copy Markdown
Contributor Author

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