Bug #278
closed[PATCH] Guard unistd.h includes
Description
unistd.h is not available when building for windows natively
Files
Updated by martin over 3 years ago
- Status changed from New to Feedback
Hmm, I'm unable to apply this patch to GIT master. Did you create those patches against a current master?
Updated by wrobelda over 3 years ago
Yeah, my bad, this was created after the patch I included here: https://www.aquamaniac.de/rdm/issues/282
Basically what it does is that it literally wraps all occurrences of #include <unistd.h> with HAVE_UNISTD_H guard
Updated by martin over 3 years ago
Yes, I understand what the patch does, its just that I can't apply it.
I don't understand why that is because the patch looks quite straightforward ...
Updated by wrobelda over 3 years ago
Some of the files it touches I had also previously modified in relation to string.h includes, hence the rejections. I have ~20 of these patches, which I kept adding to my local branch, but not all of them are actually inclusion-worthy, like the <strings.h> patch I discuss in that other issue.
Updated by martin about 1 month ago
- Status changed from Feedback to Closed
No follow-up for a long time, closing. Please reopen if still relevant.