apr-bugs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject [Bug 58473] New: iconv unnecessary includes unistd.h on Windows
Date Fri, 02 Oct 2015 16:59:42 GMT
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


Mime
View raw message