You are viewing a plain text version of this content. The canonical link for it is here.

Posted to bugs@apr.apache.org by bu...@apache.org on 2015/10/02 18:59:42 UTC

[Bug 58473] New: iconv unnecessary includes unistd.h on Windows

https://bz.apache.org/bugzilla/show_bug.cgi?id=58473

            Bug ID: 58473
           Summary: iconv unnecessary includes unistd.h on Windows
           Product: APR
           Version: HEAD
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: APR-iconv
          Assignee: bugs@apr.apache.org
          Reporter: tschoening@am-soft.de

iconv.c includes unistd.h, which is not available on Windows, but not needed at
all as well, because iconv uses APR and its apr_getopt itself. The comment
around unistd.h states that it was only needed for getopt. My current
workaround, because I don't want to patch the source, is to provide an empty
unistd.h file in the search paths for includes.

So I propose to simply remove that line entirely. Thanks!

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org