Skip to content

Two small fixes #209

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 13, 2021
Merged

Two small fixes #209

merged 2 commits into from
Jul 13, 2021

Conversation

bgamari
Copy link
Contributor

@bgamari bgamari commented Jul 12, 2021

  • Fix const-correctness in non-posix_spawnp codepath
  • Consolidate Windows check in testsuite driver.

bgamari added 2 commits July 12, 2021 17:04
@bgamari
cbits/posix: Fix const-correctness in non-posix_spawnp path
6c69780 
Previously this would throw a warning when posix_spawnp isn't available.
@bgamari
test: Use isWindows instead of System.Info.os
fc05e9f
@bgamari bgamari mentioned this pull request Jul 12, 2021
Merged
@snoyberg snoyberg merged commit 516dc23 into haskell:master Jul 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@snoyberg snoyberg snoyberg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bgamari @snoyberg