Skip to content

fix: add ".exe" in the C++ compilation command on Windows#873

Merged
neko-para merged 2 commits intomasterfrom
fix-872
Jun 15, 2021
Merged

fix: add ".exe" in the C++ compilation command on Windows#873
neko-para merged 2 commits intomasterfrom
fix-872

Conversation

@ouuan
Copy link
Copy Markdown
Member

@ouuan ouuan commented Jun 14, 2021

Related Issues / Pull Requests

This fixes #872.

How Has This Been Tested?

Not tested yet. I bought a new laptop and my Windows dev environment isn't prepared yet.

Checklist

  • If the key of a setting is changed, the old attribute is updated or it is resolved in SettingsUpdater.
  • If there are changes of the text displayed in the UI, they are wrapped in tr() or QCoreApplication::translate().
  • If needed, I have opened a pull request or an issue to update the documentation.
  • If these changes are notable, they are documented in CHANGELOG.md.

@ouuan ouuan requested review from coder3101 and neko-para June 14, 2021 13:28
@neko-para neko-para merged commit fac4eec into master Jun 15, 2021
@neko-para neko-para deleted the fix-872 branch June 15, 2021 02:56
ouuan added a commit that referenced this pull request Jun 17, 2021
I made a silly mistake in #873, and it magically got approved 🤔
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.

Detached Execution is not working in Windows when using clang

3 participants