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

Windows Support #1

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

Windows Support #1

wants to merge 5 commits into from

Conversation

Prince213
Copy link

No description provided.

@aligrudi
Copy link
Owner

aligrudi commented Aug 25, 2021 via email

@Prince213
Copy link
Author

Prince213 commented Aug 25, 2021

Yes, though there are still some problems to be fixed. I'm working on that.
What do you think of making neatroff work on Windows?

@aligrudi
Copy link
Owner

aligrudi commented Aug 25, 2021 via email

@Prince213
Copy link
Author

Natively. Since it can already compile for Windows, I would say the code won't need to change too much. The problems left are probably path handling, encoding handling, and relocatability support.
As for Cygwin, it's probably fine to treat it as a Unix variant.

@Prince213
Copy link
Author

The changes in this very PR is enough to compile it on Windows.

@aligrudi
Copy link
Owner

aligrudi commented Aug 25, 2021 via email

@Prince213
Copy link
Author

Other programs can be compiled without change. However, work are required to make them work properly.

@Prince213 Prince213 changed the title otf.c: Fix compilation on Windows Windows Support Aug 26, 2021
@aligrudi
Copy link
Owner

aligrudi commented Aug 31, 2021 via email

@Prince213
Copy link
Author

Instead of maintaining separate branches or repos, I would suggest creating a platform abstraction layer. You can extract POSIX-related code to a separate file, implement OS-independent functions with POSIX-specific code, and use those functions instead. This can make the code more portable and focus on logic itself.

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