xextproto and libXext cleanup
Peter Hutterer
peter.hutterer at who-t.net
Wed Jul 22 03:58:35 PDT 2009
On Wed, Jul 22, 2009 at 09:40:27AM +0100, Magnus Kessler wrote:
> On Monday 20 Jul 2009 23:11:14 Jon TURNEY wrote:
> > On 16/07/2009 04:56, Peter Hutterer wrote:
> > > Those of you watching xorg-announce or xorg-commit may have noticed that
> > > I pushed the changes to xextproto and libXext.
> > >
> > > This commit series removed all client-side headers from xextproto and
> > > moved them into libXext instead. Clients should not be affected by this
> > > other than that the BuildRequires have changed.
> > >
> > > Most video drivers were affected (dpms.h -> dpmsconst.h). The patches
> > > for intel and radeonhd are in Keith's + Eric and Matthias inboxes. I've
> > > pushed updates for the rest of the video drivers myself.
> > >
> > > The rule for header files are now:
> > > Xfoo.h - library header file in libXext
> > > foo.h - extension constants, xextproto
> > > fooproto.h - protocol structures, xextproto
> > >
> > > for those headers where foo.h was already in use as client-side header
> > > (e.g. sync.h, dpms.h) the naming is
> > > foo.h - library header in libXext
> > > fooconst.h - extension constants, xextproto
> > > fooproto.h - protocol structures, xextproto
> > >
> > > If there are any build errors, please let me know. I'll be monitoring
> > > tinderbox for the next few hours.
> >
> > This pattern doesn't seem to have been followed for MITMisc, as there is
> > now
> >
> > MITMisc.h - library header in libXext
> > mitmisc.h - extension constants, xextproto
> > mitmiscproto.h - protocol structures, xextproto
> >
> > This fails on case-insensitive filesystems
> > (case-insensitive filesystems are also fail, but that's a separate issue
> > :-) )
> >
> > Attached are patches to rename the header in xextproto from mitmisc.h to
> > mitmiscconst.h, which I think is the way to fix this if I've understood
> > your mail correctly.
> >
> > Apologies for not noticing this sooner.
>
> The same goes for Xtest.h vs xtest.h. Patches attached.
>
> Cheers,
>
> Magnus
Thank you, both pushed.
Cheers,
Peter
> From b4f1eb1054fa413317f64a6ac1bca061a1e171b1 Mon Sep 17 00:00:00 2001
> From: Magnus Kessler <Magnus.Kessler at gmx.net>
> Date: Wed, 22 Jul 2009 09:29:16 +0100
> Subject: [PATCH] Rename xtest.h to xtestconst.h to avoid a collision with XTest.h on case-insensitive filesystems
>
> ---
> include/X11/extensions/XTest.h | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/include/X11/extensions/XTest.h b/include/X11/extensions/XTest.h
> index f27407a..aad9420 100644
> --- a/include/X11/extensions/XTest.h
> +++ b/include/X11/extensions/XTest.h
> @@ -30,7 +30,7 @@ in this Software without prior written authorization from The Open Group.
> #define _XTEST_H_
>
> #include <X11/Xfuncproto.h>
> -#include <X11/extensions/xtest.h>
> +#include <X11/extensions/xtestconst.h>
> #include <X11/extensions/XInput.h>
>
> _XFUNCPROTOBEGIN
> --
> 1.6.3.3
>
> From 3f3038dae60e7ae64c9ddba7d458a865219f1445 Mon Sep 17 00:00:00 2001
> From: Magnus Kessler <Magnus.Kessler at gmx.net>
> Date: Wed, 22 Jul 2009 09:37:38 +0100
> Subject: [PATCH] Rename xtest.h to xtestconst.h to avoid a collision with XTest.h on case-insensitive filesystems
>
> ---
> Makefile.am | 2 +-
> xtest.h | 41 -----------------------------------------
> xtestconst.h | 41 +++++++++++++++++++++++++++++++++++++++++
> xtestproto.h | 2 +-
> 4 files changed, 43 insertions(+), 43 deletions(-)
> delete mode 100644 xtest.h
> create mode 100644 xtestconst.h
>
> diff --git a/Makefile.am b/Makefile.am
> index 6ba1b08..8cdd0ac 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -28,7 +28,7 @@ xext_HEADERS = \
> EVIproto.h \
> xtestext1proto.h \
> xtestext1const.h \
> - xtest.h \
> + xtestconst.h \
> xtestproto.h
>
> pkgconfigdir = $(libdir)/pkgconfig
> diff --git a/xtest.h b/xtest.h
> deleted file mode 100644
> index e74b904..0000000
> --- a/xtest.h
> +++ /dev/null
> @@ -1,41 +0,0 @@
> -/*
> -
> -Copyright 1992, 1998 The Open Group
> -
> -Permission to use, copy, modify, distribute, and sell this software and its
> -documentation for any purpose is hereby granted without fee, provided that
> -the above copyright notice appear in all copies and that both that
> -copyright notice and this permission notice appear in supporting
> -documentation.
> -
> -The above copyright notice and this permission notice shall be included in
> -all copies or substantial portions of the Software.
> -
> -THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> -IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> -FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
> -OPEN GROUP BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN
> -AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> -CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
> -
> -Except as contained in this notice, the name of The Open Group shall not be
> -used in advertising or otherwise to promote the sale, use or other dealings
> -in this Software without prior written authorization from The Open Group.
> -
> -*/
> -
> -#ifndef _XTEST_CONST_H_
> -#define _XTEST_CONST_H_
> -
> -#define XTestNumberEvents 0
> -
> -#define XTestNumberErrors 0
> -
> -#define XTestCurrentCursor ((Cursor)1)
> -
> -#define XTestMajorVersion 2
> -#define XTestMinorVersion 2
> -
> -#define XTestExtensionName "XTEST"
> -
> -#endif
> diff --git a/xtestconst.h b/xtestconst.h
> new file mode 100644
> index 0000000..e74b904
> --- /dev/null
> +++ b/xtestconst.h
> @@ -0,0 +1,41 @@
> +/*
> +
> +Copyright 1992, 1998 The Open Group
> +
> +Permission to use, copy, modify, distribute, and sell this software and its
> +documentation for any purpose is hereby granted without fee, provided that
> +the above copyright notice appear in all copies and that both that
> +copyright notice and this permission notice appear in supporting
> +documentation.
> +
> +The above copyright notice and this permission notice shall be included in
> +all copies or substantial portions of the Software.
> +
> +THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
> +IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
> +FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
> +OPEN GROUP BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN
> +AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
> +CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
> +
> +Except as contained in this notice, the name of The Open Group shall not be
> +used in advertising or otherwise to promote the sale, use or other dealings
> +in this Software without prior written authorization from The Open Group.
> +
> +*/
> +
> +#ifndef _XTEST_CONST_H_
> +#define _XTEST_CONST_H_
> +
> +#define XTestNumberEvents 0
> +
> +#define XTestNumberErrors 0
> +
> +#define XTestCurrentCursor ((Cursor)1)
> +
> +#define XTestMajorVersion 2
> +#define XTestMinorVersion 2
> +
> +#define XTestExtensionName "XTEST"
> +
> +#endif
> diff --git a/xtestproto.h b/xtestproto.h
> index 2eb832e..4f8db26 100644
> --- a/xtestproto.h
> +++ b/xtestproto.h
> @@ -27,7 +27,7 @@ in this Software without prior written authorization from The Open Group.
> #ifndef _XTESTPROTO_H_
> #define _XTESTPROTO_H_
>
> -#include <X11/extensions/xtest.h>
> +#include <X11/extensions/xtestconst.h>
>
> #define Window CARD32
> #define Time CARD32
> --
> 1.6.3.3
>
More information about the xorg-devel
mailing list