[PATCH libXaw] Add definition of getpagesize(), missing on Windows.

Alan Coopersmith alan.coopersmith at oracle.com
Tue Mar 5 17:43:14 UTC 2019


What's wrong with it is mainly that the set of people who know about X libraries
on Windows is small, and the set of people who deal with libXaw is small, and
the intersection of those two sets is tiny.   (It also doesn't help that we've
sucked for years at handling patches by email for the unmaintained modules like
libXaw - we're trying to get better with gitlab merge requests now, but we're
still not great - unmaintained modules are still not well maintained.)

I was hoping one of our Windows folks would comment on whether or not that's the
right way to handle this, or if there's some Windows header to include instead.
But if no one pipes up soon, I'll likely just go ahead and push it and see if
anyone complains afterwards that it broke something.

	-Alan Coopersmith-               alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - https://blogs.oracle.com/alanc

On 03/ 4/19 06:54 AM, James Larrowe wrote:
> Ping. Anything wrong with this? It's only defined in the plugin header files for 
> gcc.
> 
> On Thu, Feb 28, 2019 at 12:56 PM James R Larrowe <larrowe.semaj11 at gmail.com 
> <mailto:larrowe.semaj11 at gmail.com>> wrote:
> 
>     Fix build with --enable-selective-werror.
> 
>     Signed-off-by: James R Larrowe <larrowe.semaj11 at gmail.com
>     <mailto:larrowe.semaj11 at gmail.com>>
>     ---
>       src/OS.c | 3 +++
>       1 file changed, 3 insertions(+)
> 
>     diff --git a/src/OS.c b/src/OS.c
>     index 4c69005..cd0c517 100644
>     --- a/src/OS.c
>     +++ b/src/OS.c
>     @@ -37,6 +37,9 @@ _XawGetPageSize(void)
>       #endif
> 
>       #ifdef HAVE_GETPAGESIZE
>     +# if defined(_WIN32)
>     +extern int getpagesize(void);
>     +# endif
>           if (pagesize == -1)
>              pagesize = getpagesize();
>       #endif
>     -- 
>     2.11.0
> 
> 
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: https://lists.x.org/mailman/listinfo/xorg-devel
> 





More information about the xorg-devel mailing list