<div dir="ltr">Please disregard this patch. I sent a better one.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Mar 9, 2019 at 9:33 AM James Larrowe <<a href="mailto:larrowe.semaj11@gmail.com">larrowe.semaj11@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Sorry about that, accidentally responded to the digest:</div><div><br></div><div>So add another check for cygwin or disable getpagesize use on windows?</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 8, 2019 at 8:19 PM Jon Turney <<a href="mailto:jon.turney@dronecode.org.uk" target="_blank">jon.turney@dronecode.org.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 05/03/2019 17:43, Alan Coopersmith wrote:<br>
> What's wrong with it is mainly that the set of people who know about X <br>
> libraries<br>
> on Windows is small, and the set of people who deal with libXaw is <br>
> small, and<br>
> the intersection of those two sets is tiny.   (It also doesn't help that <br>
> we've<br>
> sucked for years at handling patches by email for the unmaintained <br>
> modules like<br>
> libXaw - we're trying to get better with gitlab merge requests now, but <br>
> we're<br>
> still not great - unmaintained modules are still not well maintained.)<br>
> <br>
> I was hoping one of our Windows folks would comment on whether or not <br>
> that's the<br>
> right way to handle this, or if there's some Windows header to include <br>
> instead.<br>
> But if no one pipes up soon, I'll likely just go ahead and push it and <br>
> see if<br>
> anyone complains afterwards that it broke something.<br>
> <br>
>      -Alan Coopersmith-               <a href="mailto:alan.coopersmith@oracle.com" target="_blank">alan.coopersmith@oracle.com</a><br>
>       Oracle Solaris Engineering - <a href="https://blogs.oracle.com/alanc" rel="noreferrer" target="_blank">https://blogs.oracle.com/alanc</a><br>
> <br>
> On 03/ 4/19 06:54 AM, James Larrowe wrote:<br>
>> Ping. Anything wrong with this? It's only defined in the plugin header <br>
>> files for gcc.<br>
<br>
I guess this means that you're using a MinGW-type Windows-targetted gcc <br>
and this function is present in libgcc.a (for who knows what historical <br>
reason...), which is turning on HAVE_GETPAGESIZE?<br>
<br>
In that environment, I think there is a unistd.h, which could be <br>
included to provide a prototype.<br>
<br>
Alternatively, since getpagesize() is a POSIX function, and not in the <br>
Windows API, it's possibly more correct for this code not to be used at <br>
all on Windows?<br>
<br>
James: Going forward, if you'd like to Cc: me on your patches, I will do <br>
my best to give them review.<br>
<br>
>> On Thu, Feb 28, 2019 at 12:56 PM James R Larrowe <br>
>> <<a href="mailto:larrowe.semaj11@gmail.com" target="_blank">larrowe.semaj11@gmail.com</a> <mailto:<a href="mailto:larrowe.semaj11@gmail.com" target="_blank">larrowe.semaj11@gmail.com</a>>> wrote:<br>
>><br>
>>     Fix build with --enable-selective-werror.<br>
>><br>
>>     Signed-off-by: James R Larrowe <<a href="mailto:larrowe.semaj11@gmail.com" target="_blank">larrowe.semaj11@gmail.com</a><br>
>>     <mailto:<a href="mailto:larrowe.semaj11@gmail.com" target="_blank">larrowe.semaj11@gmail.com</a>>><br>
>>     ---<br>
>>       src/OS.c | 3 +++<br>
>>       1 file changed, 3 insertions(+)<br>
>><br>
>>     diff --git a/src/OS.c b/src/OS.c<br>
>>     index 4c69005..cd0c517 100644<br>
>>     --- a/src/OS.c<br>
>>     +++ b/src/OS.c<br>
>>     @@ -37,6 +37,9 @@ _XawGetPageSize(void)<br>
>>       #endif<br>
>><br>
>>       #ifdef HAVE_GETPAGESIZE<br>
>>     +# if defined(_WIN32)<br>
<br>
Also note that, unfortunately, since _WIN32 is overloaded as meaning <br>
both "the target is Win32" and "the Win32 API is available", this <br>
construction can cause difficulties when compiling for the Cygwin target <br>
if "windows.h" has got included somewhere.<br>
<br>
>>     +extern int getpagesize(void);<br>
>>     +# endif<br>
>>           if (pagesize == -1)<br>
>>              pagesize = getpagesize();<br>
>>       #endif<br>
>>     --     2.11.0<br>
>><br>
</blockquote></div>
</blockquote></div>