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

Add Streamly.System.Process.Posix and Streamly.System.Process.Windows modules #79

Open
harendra-kumar opened this issue Dec 14, 2023 · 1 comment

Comments

@harendra-kumar
Copy link
Member

To expose the Posix and Windows specific behavior.

@harendra-kumar
Copy link
Member Author

harendra-kumar commented Dec 14, 2023

Another alternative is to export the specific options from the common module and make them ineffective on the platform on which they do not apply. This will be more convenient to use but this lead to confusion as the code reader might think it is doing something.

Or we could prefix the posix and windows APIs with explicit posix, win32 prefixes.

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

No branches or pull requests

1 participant