Re: [PATCH macros v2] Fix XORG_WITH_XMLTO to work with xmlto >= 0.0.27

2016-01-13 Thread Gaetan Nadon
o: https://fedorahosted.org/xmlto/changeset/77 > > This patch fixes this by additionally testing xmlto with a non-empty XML > file. > > More details can be found at: > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=613674 > > Thanks to Peter Pearse, Helmut Grohne and Gaetan N

Re: [PATCH RFC macros] Fix XORG_WITH_XMLTO to work with xmlto >= 0.0.27

2015-12-10 Thread Gaetan Nadon
On 15-12-10 03:08 PM, Andreas Boll wrote: > @Gaetan: > Do you still have a concern on the patch? > >> > The one thing that bugs me about the patch is that the version of >> > docbook may eventually be >> > no longer available and this macro must remain backward compatible for >> > eternity. > An

Re: [PATCH util-modular 1/3] xorg.modules: Add libevdev requirement to synaptics

2015-03-16 Thread Gaetan Nadon
On 15-03-15 06:47 PM, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- xorg.modules | 1 + 1 file changed, 1 insertion(+) diff --git a/xorg.modules b/xorg.modules index 7216192..6d80c6b 100644 --- a/xorg.modules +++ b/xorg.modules @@ -1974,6 +1974,7 @@

[PATCH app-xinit 0/3] Clean-up some dead code left by previous patches

2015-02-16 Thread Gaetan Nadon
Gaetan Nadon (3): Remove SCO support for SHELL_CMD and startx man page. Remove support for ancient A/UX 3.0 support Remove left over $(launchagents_DATA) in CLEANFILES Makefile.am | 2 +- configure.ac| 5 - man/Makefile.am | 1 - man/startx.man | 15

[PATCH app-xinit 1/3] Remove SCO support for SHELL_CMD and startx man page.

2015-02-16 Thread Gaetan Nadon
SCO support was removed from startx.cpp and xinitrc.cpp earlier. Remove unixware / sco support http://cgit.freedesktop.org/xorg/app/xinit/commit/ ?id=fdf03cd2fdfd9cd5635334c5e4dc2bb23e92e37a Signed-off-by: Gaetan Nadon mems...@videotron.ca --- configure.ac| 5 - man

[PATCH app-xinit 2/3] Remove support for ancient A/UX 3.0 support

2015-02-16 Thread Gaetan Nadon
This was Apple Computer’s implementation of the Unix operating system for some of their Macintosh computers. From 1988 to 1995. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- startx.cpp | 5 - 1 file changed, 5 deletions(-) diff --git a/startx.cpp b/startx.cpp index ce4713f..8520399

[PATCH app-xinit 3/3] Remove left over $(launchagents_DATA) in CLEANFILES

2015-02-16 Thread Gaetan Nadon
This was left over when reorganizing layout of launchd sources in commit 567f59d3f8189b92bc46e2af1260f9340f462bdb Signed-off-by: Gaetan Nadon mems...@videotron.ca --- Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile.am b/Makefile.am index 3867bea..c1eb5a9

[PATCH app-xinit 1/3] Remove SCO support for SHELL_CMD and startx man page.

2015-02-16 Thread Gaetan Nadon
SCO support was removed from startx.cpp and xinitrc.cpp earlier. Remove unixware / sco support http://cgit.freedesktop.org/xorg/app/xinit/commit/ ?id=fdf03cd2fdfd9cd5635334c5e4dc2bb23e92e37a Signed-off-by: Gaetan Nadon mems...@videotron.ca --- configure.ac| 5 - man

Re: Migrating away from using cpp for startx and xinitrc in xinit

2015-02-10 Thread Gaetan Nadon
On 15-02-10 10:44 AM, Alan Coopersmith wrote: Right - build time uses such as startx xinitrc should be replacable with a bit of work, such as Gaetan did in the past to use sed instead of cpp on a bunch of man pages. I had worked on this three years ago. I tried removing the use of cpp as a

Re: [PATCH xorg-fonts] configure.ac: remove C compiler dependency

2014-12-16 Thread Gaetan Nadon
On 14-12-15 07:54 PM, Alan Coopersmith wrote: On 12/12/14 09:43 AM, Richard Tollerton wrote: Font compilation does not require a C compiler, but configure will fail anyway if one isn't found, because of the inclusion of XORG_DEFAULT_OPTIONS. Fix this by including only those macros normally in

Re: [PATCH util/modular] Add gpg signing to release.sh

2014-06-08 Thread Gaetan Nadon
not appear for me. Perhaps because the function is called from $(). I do get the gpg message: You need a passphrase to unlock the secret key for user: Gaetan Nadon mems...@videotron.ca 1024-bit DSA key, ID FB9EC9FC, created 2008-12-16 Not a big deal, Reviewed-by: Gaetan Nadonmems

Re: [PATCH 1.5/4 util-modular] release.sh: force /bin/bash

2014-06-03 Thread Gaetan Nadon
and upload a git module tarball # Reviewed-by: Gaetan Nadon mems...@videotron.ca ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH util-modular 2/4] release.sh: move the bit to extract the section into a function

2014-05-30 Thread Gaetan Nadon
On 14-05-29 06:24 PM, Peter Hutterer wrote: I was wondering about that, but at least bash --posix was happy with it and I'm struggling how to test pure Bourne shell compatibility. I used http://www.hoomanb.com/cs/QuickRef/bash_ref.pdf when working on build.sh. I got feedback from people running

Re: [PATCH util-modular 2/4] release.sh: move the bit to extract the section into a function

2014-05-29 Thread Gaetan Nadon
On 14-05-29 12:51 AM, Peter Hutterer wrote: No functional changes intended Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- release.sh | 128 +++-- 1 file changed, 74 insertions(+), 54 deletions(-) diff --git a/release.sh

Re: [PATCH 1/2] man/Makefile.am: Fix Xorg.wrap.man Xwrapper.config.man missing from make dist

2014-04-18 Thread Gaetan Nadon
On 14-04-18 05:31 AM, Hans de Goede wrote: Fix suggested by: Gaetan Nadon mems...@videotron.ca Cc: Gaetan Nadon mems...@videotron.ca Signed-off-by: Hans de Goede hdego...@redhat.com --- hw/xfree86/man/Makefile.am | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/xfree86/man

Re: server make dist not picking up optional manpages

2014-04-16 Thread Gaetan Nadon
On 14-04-16 10:57 AM, Hans de Goede wrote: Hi All, In hw/xfree86/man/Makefile.am we now have: include $(top_srcdir)/manpages.am appman_PRE = Xorg.man fileman_PRE = xorg.conf.man xorg.conf.d.man if SUID_WRAPPER appman_PRE += Xorg.wrap.man fileman_PRE += Xwrapper.config.man endif And

Re: [PATCH 3/3] configure.ac: Remove check for WAYLAND_SCANNER_RULES

2014-04-08 Thread Gaetan Nadon
([XWAYLAND_SYS_LIBS]) - WAYLAND_SCANNER_RULES(['$(top_srcdir)/hw/xwayland']) fi Reviewed-by: Gaetan Nadon mems...@videotron.ca ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org

Re: Xwayland DDX: build breaks in configuration

2014-04-06 Thread Gaetan Nadon
On 14-04-06 06:02 AM, Yaakov (Cygwin/X) wrote: Any idea how to fix this? I would prefer that the RC I'm still trying to construct will build for most people... Adding a copy of wayland-scanner.m4 to m4/ should work. This would work if this macro does not itself introduces more dependencies.

Re: [PATCH xfs] Fix CFLAGS and LDFLAGS for Cygwin/MinGW

2014-04-06 Thread Gaetan Nadon
On 14-04-06 03:45 PM, Yaakov (Cygwin/X) wrote: +case $host_os in + cygwin*|mingw*) +CFLAGS=$CFLAGS -DFD_SETSIZE=256 +LDFLAGS=$LDFLAGS -Wl,--export-all ;; +esac We are not supposed to clobber user variables like CFLAGS. I didn't look at the particular cygwin environment, maybe it is

[PATCH RESEND2 xserver] test: create a link to the generated hw/xfree86/sdksyms.c at build time

2014-04-04 Thread Gaetan Nadon
. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- test/.gitignore |1 + test/Makefile.am |8 +++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/test/.gitignore b/test/.gitignore index acbda7a..da86d6e 100644 --- a/test/.gitignore +++ b/test/.gitignore @@ -4,6 +4,7

[PATCH util-macros 1/2] Add XORG_AUTHORS macro to generate a list of authors from git

2014-04-03 Thread Gaetan Nadon
-by: Gaetan Nadon mems...@videotron.ca --- xorg-macros.m4.in |1 + xorgversion.m4| 16 2 files changed, 17 insertions(+) diff --git a/xorg-macros.m4.in b/xorg-macros.m4.in index f160a40..20b999e 100644 --- a/xorg-macros.m4.in +++ b/xorg-macros.m4.in @@ -1807,6 +1807,7

[PATCH util-macros 2/2] Use the new XORG_AUTHORS macro for the util-macros package

2014-04-03 Thread Gaetan Nadon
Signed-off-by: Gaetan Nadon mems...@videotron.ca --- .gitignore |1 + Makefile.am |9 +++-- configure.ac |1 + 3 files changed, 9 insertions(+), 2 deletions(-) diff --git a/.gitignore b/.gitignore index 61198c9..058b4b5 100644 --- a/.gitignore +++ b/.gitignore @@ -5,6 +5,7

Xwayland DDX: build breaks in configuration

2014-04-03 Thread Gaetan Nadon
http://tinderbox.x.org/builds/2014-04-03-0020/logs/xserver/#configure checking whether to build Xwayland DDX... no /home/robclark/xorg/xserver/configure: line 31420: syntax error near unexpected token `'$(top_srcdir)/hw/xwayland'' /home/robclark/xorg/xserver/configure: line 31420: `

Re: [PATCH xinit v3 1/3] Bump required util-macros version to 1.19

2014-04-01 Thread Gaetan Nadon
On 14-03-31 09:51 AM, Hans de Goede wrote: Signed-off-by: Hans de Goede hdego...@redhat.com --- configure.ac | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) Reviewed-by: Gaetan Nadon mems...@videotron.ca ___ xorg-devel@lists.x.org: X.Org

Re: [PATCH xinit v3 2/3] Drop Replace $RAWCPPFLAGS with $TRADITIONALCPPFLAGS when processing cpp files

2014-04-01 Thread Gaetan Nadon
This commit makes these work again. Signed-off-by: Hans de Goede hdego...@redhat.com Reviewed-by: Gaetan Nadon mems...@videotron.ca ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman

Re: Not able to cross-compile xserver

2014-04-01 Thread Gaetan Nadon
On 14-04-01 04:45 AM, Prashant Bhumkar wrote: configure: WARNING: unrecognized options: --with-driver, --with-mesa-source, --enable-malloc0returnsnull Regarding --enable-malloc0returnsnull, see: http://cgit.freedesktop.org/xorg/util/macros/commit/?id=72fdc868b56fe2b7bdc9a69872651baeca728fb6

[ANNOUNCE] util-macros 1.19.0

2014-03-28 Thread Gaetan Nadon
Alan Coopersmith (1): XORG_COMPILER_FLAGS: Add -Wlogical-op to default warning set Arnaud Fontaine (1): Add XORG_WITH_M4 macro Gaetan Nadon (3): Bump minimum Autoconf required version to 2.62 Provide the automake INSTALL file at level 1.11 Version bump: 1.19.0 Hans

[ANNOUNCE] util-macros 1.19.0

2014-03-28 Thread Gaetan Nadon
Alan Coopersmith (1): XORG_COMPILER_FLAGS: Add -Wlogical-op to default warning set Arnaud Fontaine (1): Add XORG_WITH_M4 macro Gaetan Nadon (3): Bump minimum Autoconf required version to 2.62 Provide the automake INSTALL file at level 1.11 Version bump: 1.19.0 Hans

Re: [PATCH util/macros] XORG_PROG_RAWCPP: Add TRADITIONALCPPFLAGS

2014-03-27 Thread Gaetan Nadon
@ version 1.19. Any module using TRADITIONALCPPFLAGS will need to require version 1.19. Reviewed-by: Gaetan Nadon mems...@videotron.ca ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman

Re: [PATCH xinit v2 1/3] Remove unixware / sco support

2014-03-27 Thread Gaetan Nadon
On 14-03-27 07:55 AM, Hans de Goede wrote: We don't support SCO / Unixware anymore, so lets remove the SCO / Unixware specific bits from startx and xinitrc Signed-off-by: Hans de Goede hdego...@redhat.com Reviewed-by: Gaetan Nadon mems...@videotron.ca SCO support was removed from the server

Re: [PATCH util/macros] XORG_PROG_RAWCPP: Add TRADITIONALCPPFLAGS

2014-03-27 Thread Gaetan Nadon
On 14-03-27 08:48 AM, Hans de Goede wrote: Once the 1.19 release is done I'll write a patch for xinit to bump the required util-macros version (and out that in my patchset before the patch using the new TRADITIONALCPPFLAGS. Release 1.19 is out.

Re: [PATCH xinit 1/3] Drop $RAWCPPFLAGS when generating startx

2014-03-26 Thread Gaetan Nadon
On 14-03-26 07:37 AM, Hans de Goede wrote: Hi, On 03/25/2014 06:22 PM, Gaetan Nadon wrote: On 14-03-25 07:56 AM, Hans de Goede wrote: startx.cpp contains things like #if defined(__SCO__), and $RAWCPPFLAGS contains -undef causing these to not get set. Signed-off-by: Hans de Goede hdego

[PATCH xserver] suid: adding Xorg.sh.in to EXTRA_DIST is redundant

2014-03-25 Thread Gaetan Nadon
All files specified in AC_CONFIG_FILES get distributed automatically. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- hw/xfree86/Makefile.am |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/xfree86/Makefile.am b/hw/xfree86/Makefile.am index 18fa99b..ceb4c6a 100644

[PATCH xorg-docs] Fix typo in Release Notes.

2014-03-25 Thread Gaetan Nadon
Signed-off-by: Gaetan Nadon mems...@videotron.ca --- general/ReleaseNotes.xml |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/general/ReleaseNotes.xml b/general/ReleaseNotes.xml index 8fd43ba..72fb644 100644 --- a/general/ReleaseNotes.xml +++ b/general/ReleaseNotes.xml

Re: Problems while compiling xorg on Ubuntu

2014-03-23 Thread Gaetan Nadon
Fedora patches it too, see the end of: http://pkgs.fedoraproject.org/cgit/xorg-x11-drv-sis.git/tree/sis-0.10.7-git.patch |+miPointerSetPosition(inputInfo.pointer, Absolute, dx, dy, NULL, NULL);| This patch gets rid of the compile error but does not work. A real event list and

Re: [PATCH xserver] xfree86: glamor_egl subdir must be distributed - breaks distcheck

2014-03-22 Thread Gaetan Nadon
On 14-03-21 04:53 PM, Eric Anholt wrote: Gaetan Nadon mems...@videotron.ca writes: Signed-off-by: Gaetan Nadon mems...@videotron.ca I've reviewed these two patches of yours and they'll be in my next pull request. Thanks! Great, thanks. ___ xorg

Re: Problems while compiling xorg on Ubuntu

2014-03-22 Thread Gaetan Nadon
On 14-03-21 04:20 PM, pplive wrote: I try to compile xorg according to the following instruction: http://www.x.org/wiki/Building_the_X_Window_System/ When I tried to execute ./util/modular/build.sh --clone $HOME/build, I have received the following error information: The sis driver has been

Re: Problems while compiling xorg on Ubuntu

2014-03-22 Thread Gaetan Nadon
On 14-03-22 06:06 PM, Felix Miata wrote: On 2014-03-22 16:58 (GMT-0400) Gaetan Nadon composed: The sis driver has been unmaintained for a long time. The build.sh contains pretty much all of the video drivers one may need. There are about 50 of them. Most would work only with a specific video

Re: [PATCH 3/2] Build fbcmap_mi.c once, rather than for each DDX

2014-03-22 Thread Gaetan Nadon
On 14-03-22 03:27 PM, Jon TURNEY wrote: On 19/03/2014 21:29, Gaetan Nadon wrote: On 14-03-03 10:08 AM, Jon TURNEY wrote: Build fbcmap_mi.c once, rather than for each DDX, and make it part of libfb or libwfb convenience library. Since 84e8de1271bb11b5b4b9747ae4647f47333a8ab7 we don't have

Re: [PATCH] .gitignore: Add new autotools file 'test-driver'

2014-03-19 Thread Gaetan Nadon
On 14-03-19 01:12 AM, Kristian Høgsberg wrote: Recent automake introduces a new shell script helper for the automake test framework. Add it to .gitignore. Signed-off-by: Kristian Høgsberg k...@bitplanet.net --- Gaetan, not sure what the convention is for .gitignore. You added the Do not

Re: [PATCH] .gitignore: Add new autotools file 'test-driver'

2014-03-19 Thread Gaetan Nadon
On 14-03-19 07:44 AM, Gaetan Nadon wrote: Can you specify which level of automake in the commit text? Will all modules need this? Or just those including some specific autoconf macros? It was introduced in automake 1.12 with the changes in the test suite framework. It would be best to write

[PATCH xserver] ephyr: spurious $() in the Makefile breaks the make 'dist' target

2014-03-19 Thread Gaetan Nadon
Introduced in commit 9fe052d90cca90fdf750d3a45b151be2ac7f0ebd Signed-off-by: Gaetan Nadon mems...@videotron.ca --- hw/kdrive/ephyr/Makefile.am | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/hw/kdrive/ephyr/Makefile.am b/hw/kdrive/ephyr/Makefile.am index

[PATCH xserver] xfree86: glamor_egl subdir must be distributed - breaks distcheck

2014-03-19 Thread Gaetan Nadon
Signed-off-by: Gaetan Nadon mems...@videotron.ca --- hw/xfree86/Makefile.am |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/xfree86/Makefile.am b/hw/xfree86/Makefile.am index 73e1b4c..c4aa177 100644 --- a/hw/xfree86/Makefile.am +++ b/hw/xfree86/Makefile.am @@ -43,7 +43,7

Re: [PATCH 1/2] .gitignore: Add new autotools file 'test-driver'

2014-03-19 Thread Gaetan Nadon
On 14-03-19 01:27 PM, Kristian Høgsberg wrote: Automake 1.12 introduces a new parallel test framework that uses a shell script helper and generates *.log and *.trs files. Add to .gitignore. Signed-off-by: Kristian Høgsberg k...@bitplanet.net Cc: Gaetan Nadon mems...@videotron.ca

Re: [PATCH] .gitignore: Add new autotools file 'test-driver'

2014-03-19 Thread Gaetan Nadon
On 14-03-19 12:42 PM, Guillem Jover wrote: Actually, why are those in the root dir, and not confined under something like AC_CONFIG_AUX_DIR([build-aux])? A handful of modules do that and is perfectly ok. A while ago I had considered doing just to all xorg modules. The vast majority of 250 xorg

Re: [PATCH xserver] ephyr: spurious $() in the Makefile breaks the make 'dist' target

2014-03-19 Thread Gaetan Nadon
On 14-03-19 12:51 PM, Guillem Jover wrote: One reason to use something (in spirit) like this is to avoid unnecessary line changes whenever a new entry is appended to the list, which might also generate less conflicts. I tend to use $(nil) as the last item in those cases. The other option is to

[PATCH xserver] ephyr: typo where () should be $() in the Makefile - breaks make dist

2014-03-19 Thread Gaetan Nadon
make[3]: Entering directory `/home/nadon/xorg/src/xserver/hw/kdrive/ephyr' make[3]: *** No rule to make target `()', needed by `distdir'. Stop. make[3]: Leaving directory `/home/nadon/xorg/src/xserver/hw/kdrive/ephyr' make[2]: *** [distdir] Error 1 Signed-off-by: Gaetan Nadon mems

Re: [PATCH 3/2] Build fbcmap_mi.c once, rather than for each DDX

2014-03-19 Thread Gaetan Nadon
On 14-03-03 10:08 AM, Jon TURNEY wrote: Build fbcmap_mi.c once, rather than for each DDX, and make it part of libfb or libwfb convenience library. Since 84e8de1271bb11b5b4b9747ae4647f47333a8ab7 we don't have fbcmap.c This is a sort of revert of 7ecc2d526c4ea5db2589644a2fec0daf71df36da

Re: [PATCH RESEND2 xserver] Default font path: remove the check for ${sysconfdir}/X11/fontpath.d

2014-03-19 Thread Gaetan Nadon
On 14-02-08 09:28 AM, Gaetan Nadon wrote: I'd like to get some review from people familiar with the distro packaging process to understand if this analysis is accurate; I haven't any way to know what the effect of this change would be otherwise. I would like to as well. The dead code

[PATCH RESEND xserver] test: create a link to the generated hw/xfree86/sdksyms.c at build time

2014-03-19 Thread Gaetan Nadon
. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- test/.gitignore |1 + test/Makefile.am |8 +++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/test/.gitignore b/test/.gitignore index acbda7a..da86d6e 100644 --- a/test/.gitignore +++ b/test/.gitignore @@ -4,6 +4,7

Re: [PATCH:mkfontscale 1/2] Only include config.h if it exists.

2014-03-18 Thread Gaetan Nadon
On 14-03-18 06:18 PM, Thomas Klausner wrote: Signed-off-by: Thomas Klausner w...@netbsd.org --- hash.c| 2 ++ mkfontscale.c | 2 ++ 2 files changed, 4 insertions(+) Reviewed-by: Gaetan Nadon mems...@videotron.ca ___ xorg-devel@lists.x.org

Re: [PATCH util/modular] jhbuildrc: Restore mesa-demos, mesa-glut, mesa-glu

2014-03-18 Thread Gaetan Nadon
On 14-03-18 12:56 PM, Jon TURNEY wrote: On 16/03/2014 18:15, Gaetan Nadon wrote: On 14-03-15 02:17 PM, Jon TURNEY wrote: On 14/03/2014 20:03, Jon TURNEY wrote: As of 9167c5e7177a758fce55afe759fa48c47a4f7f4e we had mesa-demos, mesa-glut and mesa-glu modules

Re: [PATCH util/modular] jhbuildrc: Restore mesa-demos, mesa-glut, mesa-glu

2014-03-16 Thread Gaetan Nadon
On 14-03-15 02:17 PM, Jon TURNEY wrote: On 14/03/2014 20:03, Jon TURNEY wrote: As of 9167c5e7177a758fce55afe759fa48c47a4f7f4e we had mesa-demos, mesa-glut and mesa-glu modules. 9167c5e7177a758fce55afe759fa48c47a4f7f4e add missing modules and meta modules (confusingly) removes them.

Re: [PATCH libxtrans] configure: under glibc define _GNU_SOURCE rather then _BSD_SOURCE

2014-03-08 Thread Gaetan Nadon
On 14-03-08 09:03 AM, Hans de Goede wrote: So I've just been looking into using AC_USE_SYSTEM_EXTENSIONS but that seem pretty useless, it only adds the relevant defines to confdefs.h, not into cflags, and in libxtrans we want to add them to the .pc file, so we need them in a variable. I'm

Re: Compiling xf86-video-xgi driver for ARM

2014-02-27 Thread Gaetan Nadon
On 14-02-27 04:47 AM, Artem Makarov wrote: Too bad. XGIFB is very slow, I wanted to get at least some acceleration with xorg driver. Indeed. This is an open source project, if anyone is willing to contribute... http://www.x.org/wiki/Development/Documentation/SubmittingPatches/

Re: Compiling xf86-video-xgi driver for ARM

2014-02-26 Thread Gaetan Nadon
On 14-02-26 07:48 AM, Artemy Makarov wrote: I have a ARM PC, similar to hp t5325, based on Marvell CPU and XGI Volari z11 video chip. I've compiled xf86-video-xgi driver for it, but it doesn't work. FYI, this driver was removed from the regular build in Dec 2011. xorg.modules: Remove

Re: [PATCH xfs 1/3] Warning fixes.

2014-02-25 Thread Gaetan Nadon
On 14-01-29 04:14 PM, Keith Packard wrote: This patch also stops re-using the 'port' variable in GetInetdListenInfo when fetching the ReopenInfo. That's not strictly necessary here, but will be useful when the Reopen API in libxtrans changes to take a constant parameter. Signed-off-by: Keith

Re: [PATCH xserver] test: create a link to the generated hw/xfree86/sdksyms.c at build time

2014-02-24 Thread Gaetan Nadon
On 14-02-23 11:46 PM, Keith Packard wrote: Gaetan Nadon mems...@videotron.ca writes: Automake 1.14 gives us warning about source code specified in _SOURCES that comes from directories other than the current one. It suggests to enable the subdir-objects feature which only supports code in sub

Re: [PATCH 0/2] Build Xi and DPMS stubs as convenience libraries

2014-02-24 Thread Gaetan Nadon
| 4 +++- Xi/Makefile.am | 5 +++-- hw/vfb/Makefile.am | 6 +++--- hw/xnest/Makefile.am | 6 +++--- hw/xwin/Makefile.am | 8 test/Makefile.am | 6 +++--- 6 files changed, 19 insertions(+), 16 deletions(-) Reviewed-by: Gaetan Nadon mems...@videotron.ca And passes

Re: [PATCH xserver] test: create a link to the generated hw/xfree86/sdksyms.c at build time

2014-02-24 Thread Gaetan Nadon
On 14-02-24 05:27 PM, Keith Packard wrote: Gaetan Nadon mems...@videotron.ca writes: Note that we do not need the subdir-objects feature, but automake 2.0 will turn it on any (and it cannot be turned off). We happen to have code that will trip this feature. So we need to fix it before 2.0

Re: Incorrect conditional testing in servermd.h related to linux/arm

2014-02-21 Thread Gaetan Nadon
On 14-02-20 10:33 AM, Arnaud Fontaine wrote: while compiling mesa on Linux/ARM (and for Linux/ARM), I had a problem with the servermd.h file coming from xorg-server. I found incorrect conditional tests on linux variable while it should be on __linux__ variable. This mistake occurs at several

Re: [PATCH xserver 0/5] Prepare makefiles for automake 2.0 mandatory subdir-objects

2014-02-19 Thread Gaetan Nadon
On 14-02-18 12:24 PM, Matthieu Herrb wrote: On Tue, Feb 18, 2014 at 06:09:20PM +0100, Mark Kettenis wrote: Date: Mon, 17 Feb 2014 22:26:17 -0500 From: Gaetan Nadon mems...@videotron.ca On 14-02-17 05:42 PM, Mark Kettenis wrote: Ugh, please no. Can't we just stick to automake 1.x? We can't

Re: [PATCH xserver 0/5] Prepare makefiles for automake 2.0 mandatory subdir-objects

2014-02-19 Thread Gaetan Nadon
On 14-02-19 12:59 PM, Mark Kettenis wrote: Date: Wed, 19 Feb 2014 12:45:32 -0500 From: Gaetan Nadon mems...@videotron.ca This is great feedback. So links are not the ideal solution, in fact, no solution at all for many. To my knowledge, this leaves us with the ideal solution I call code re

[PATCH xserver] test: create a link to the generated hw/xfree86/sdksyms.c at build time

2014-02-19 Thread Gaetan Nadon
. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- test/.gitignore |1 + test/Makefile.am |8 +++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/test/.gitignore b/test/.gitignore index acbda7a..da86d6e 100644 --- a/test/.gitignore +++ b/test/.gitignore @@ -4,6 +4,7

[PATCH xserver] test: remove source file from hashtabletest LDADD

2014-02-19 Thread Gaetan Nadon
LDADD is for libraries and not for source code. Introduced in commit: ccb3e78124fb05defd0c9b438746b79d84dfc3ae Signed-off-by: Gaetan Nadon mems...@videotron.ca --- test/Makefile.am |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/Makefile.am b/test/Makefile.am index

Re: [PATCH xserver 1/5] hw: use links in git to share source files due to upcoming automake changes

2014-02-18 Thread Gaetan Nadon
On 14-02-18 07:35 AM, Jon TURNEY wrote: I'd say if we are building the same object in different places, this is a sign that something else is wrong. I wrote a patch some time ago to build dpmsstubs.c as a convenience library [1], and this would seem to be the better approach to this problem

[PATCH xserver 0/5] Prepare makefiles for automake 2.0 mandatory subdir-objects

2014-02-17 Thread Gaetan Nadon
works by accident. This is an opinion, just to set expectations for anyone who would be hoping to change automake to preserve status quo. Gaetan Nadon (5): hw: use links in git to share source files due to upcoming automake changes test: remove source file from hashtabletest LDADD fb and xi

[PATCH xserver 4/5] xfree86: use links in git to share files due to upcoming automake changes

2014-02-17 Thread Gaetan Nadon
The Makefile will simply lists kmod_noop.c as opposed to $(srcdir)/../shared/kmod_noop.c The will not trigger the subdir-objects feature, even in automake 2.0. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- hw/xfree86/int10/Makefile.am |4 ++-- hw/xfree86/int10/linux.c

[PATCH xserver 1/5] hw: use links in git to share source files due to upcoming automake changes

2014-02-17 Thread Gaetan Nadon
$(top_srcdir) which appears to be a copy/paste error. This should in theory have failed but it didn't. Automake appends a workaround: `test -f '$(top_builddir)/fb/fbcmap_mi.c' || echo '$(srcdir)/'`$(top_builddir)/fb/fbcmap_mi.c Signed-off-by: Gaetan Nadon mems...@videotron.ca --- hw/dmx/Makefile.am

[PATCH xserver 3/5] fb and xi: source files in EXTRA_DIST are redundant

2014-02-17 Thread Gaetan Nadon
This statement has never been needed. The source files were distributed as they are part of many DDXes libraries. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- Xi/Makefile.am |1 - fb/Makefile.am |1 - 2 files changed, 2 deletions(-) diff --git a/Xi/Makefile.am b/Xi/Makefile.am

[PATCH xserver 5/5] test: create a link to the generated hw/xfree86/sdksyms.c at build time

2014-02-17 Thread Gaetan Nadon
nor under a sub directory of test. In 1.14 we get a warning, in 2.0 it will break as it will overwrite the object code in xfree86. The solution in this case is to create a link to hw/xfree86/sdksyms.c at build time. It's just like any other built source file. Signed-off-by: Gaetan Nadon mems

[PATCH xserver 2/5] test: remove source file from hashtabletest LDADD

2014-02-17 Thread Gaetan Nadon
LDADD is for libraries and not for source code. Introduced in commit: ccb3e78124fb05defd0c9b438746b79d84dfc3ae Signed-off-by: Gaetan Nadon mems...@videotron.ca --- test/Makefile.am |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/Makefile.am b/test/Makefile.am index

Re: [PATCH xserver 0/5] Prepare makefiles for automake 2.0 mandatory subdir-objects

2014-02-17 Thread Gaetan Nadon
On 14-02-17 05:42 PM, Mark Kettenis wrote: Ugh, please no. Can't we just stick to automake 1.x? We can't stop people from using whatever comes with their distro. With automake 2.0, it will break and will submit patches to fix it. From a practical point of view, some of us import Xorg into

Compiling xserver with --disable-xorg fails in test subdir

2014-02-14 Thread Gaetan Nadon
make[1]: Entering directory `/home/nadon/xorg/src/xserver/test' ../dix/dix.O: In function `dix_main': /home/nadon/xorg/src/xserver/dix/main.c:200: undefined reference to `InitOutput' /home/nadon/xorg/src/xserver/dix/main.c:264: undefined reference to `InitInput'

Re: [PATCH] Enable subdir-objects

2014-02-13 Thread Gaetan Nadon
On 14-02-13 03:51 AM, Thierry Reding wrote: On Wed, Feb 12, 2014 at 08:33:21PM -0500, Gaetan Nadon wrote: On 14-02-12 05:50 PM, Gaetan Nadon wrote: If the automake defined variables $(top_srcdir) does not work, then something is seriously broken and the root cause should be found, not worked

[PATCH xserver] config: fails to create tarball as xorg-server.conf file removed

2014-02-13 Thread Gaetan Nadon
Just need to update EXTRA_DIST Signed-off-by: Gaetan Nadon mems...@videotron.ca --- config/Makefile.am |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/Makefile.am b/config/Makefile.am index e0f0a8d..c6350be 100644 --- a/config/Makefile.am +++ b/config/Makefile.am

[PATCH xserver] config: fails to create tarball as xorg-server.conf file removed

2014-02-13 Thread Gaetan Nadon
Just need to update EXTRA_DIST Reviewed-by: Peter Hutterer peter.hutte...@who-t.net Signed-off-by: Gaetan Nadon mems...@videotron.ca --- config/Makefile.am |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/Makefile.am b/config/Makefile.am index e0f0a8d..c6350be 100644

Re: [PULL] button mapping fix and unconstify patches

2014-02-12 Thread Gaetan Nadon
. Too many code changes for me to give a rb. I did manage to start X however. Tested-by: Gaetan Nadon mems...@videotron.ca ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman

Re: [PULL] button mapping fix and unconstify patches

2014-02-12 Thread Gaetan Nadon
On 14-02-12 12:57 PM, Keith Packard wrote: This function is supposed to be automatically replaced with os/strlcpy.c when not present in your C library. Something appears to be amiss here; I don't have strlcpy *or* strlcat in my glibc, and so configure correctly adds os/strlcpy.c and

Re: [PATCH] Enable subdir-objects

2014-02-12 Thread Gaetan Nadon
On 14-02-12 11:15 AM, Thierry Reding wrote: automake complains about the subdir-objects being missing. Enabling it, Is automake issuing a real warning or just making a suggestion? however, causes various build issues to pop up because $(srcdir), $(top_srcdir), $(builddir) and $(top_builddir)

Re: [PATCH] Enable subdir-objects

2014-02-12 Thread Gaetan Nadon
On 14-02-12 05:50 PM, Gaetan Nadon wrote: +AM_INIT_AUTOMAKE([foreign dist-bzip2 subdir-objects]) This is not a new feature in 1.14, but it has never been used so far. All 250 X modules have the same line. If you want to propose this, as it changes all of xserver, you would have to test all 98

Re: [PULL] button mapping fix and unconstify patches

2014-02-12 Thread Gaetan Nadon
On 14-02-12 06:24 PM, Keith Packard wrote: So, we could simply add #ifndef _FORTIFY_SOURCE #define _FORTIFY_SOURCE 2 #endif to some X server header file, or you could do something fancier with the configure macros. I'm easy, but I definitely want it somehow :-)

Re: [PULL] button mapping fix and unconstify patches

2014-02-12 Thread Gaetan Nadon
On 14-02-12 07:37 PM, Peter Hutterer wrote: just an issue with incomplete builds, happens to me when I rebase or switch branches. make clean in os/ usually does the job, otherwise autoreconf/configure definitely does. yep, I rebased, applied a patch, ran ./configure (but not autogen) Thanks

Re: [PATCH] Enable subdir-objects

2014-02-12 Thread Gaetan Nadon
On 14-02-12 05:50 PM, Gaetan Nadon wrote: If the automake defined variables $(top_srcdir) does not work, then something is seriously broken and the root cause should be found, not worked around :-) A lot has been written about this already. I was able to reproduce the problem with 1.11. Various

Re: [PULL] button mapping fix and unconstify patches

2014-02-11 Thread Gaetan Nadon
On 14-02-11 12:08 AM, Keith Packard wrote: It's not just the GCC version, unfortunately. I have a pile of versions, and cannot generate any of the (useful) warnings that you have managed to elicit from the compiler. I tried Ubuntu 13-10 with gcc 4.8.2. The 32-bit version 35 -Wunused-result

Re: [PULL] button mapping fix and unconstify patches

2014-02-10 Thread Gaetan Nadon
On 14-02-09 07:03 PM, Keith Packard wrote: Gaetan Nadon mems...@videotron.ca writes: I have a total of 242 warnings. 202-Wshadow 35 -Wunused-result 2 -Wpointer-arith 2 -Wformat 1 -Wunused-function Are you running the latest headers, libraries, mesa and X server bits? Yes, just

Re: [PULL] button mapping fix and unconstify patches

2014-02-10 Thread Gaetan Nadon
On 14-02-10 06:36 PM, Alan Coopersmith wrote: I believe the ones about shadowing system functions like index are silenced in newer gcc versions - for instance I see them with gcc 4.5 but not 4.7. Which gcc did you use in this build? gcc (Ubuntu/Linaro 4.6.3-1ubuntu5) 4.6.3. From Ubuntu 12.04

Re: [PULL] button mapping fix and unconstify patches

2014-02-10 Thread Gaetan Nadon
On 14-02-10 06:57 PM, Keith Packard wrote: Gaetan Nadon mems...@videotron.ca writes: Ok, except for -Wshadow, these are all legitimate warnings (and a few actual bugs!) that we should fix. I would love for someone to explain why my build doesn't generate the useful warnings and why Gaedon's

Re: [PULL] button mapping fix and unconstify patches

2014-02-09 Thread Gaetan Nadon
On 14-02-09 05:55 AM, Keith Packard wrote: Gaetan Nadon mems...@videotron.ca writes: |||Perhaps some of its companions could be re-instated as well, should the xserver code base now be considered fully modernized? Patches to make the X server compile without warning would really be nice

Re: [PULL] button mapping fix and unconstify patches

2014-02-09 Thread Gaetan Nadon
On 14-02-09 02:12 PM, Keith Packard wrote: If you can reproduce this on your machine (and, I hope that is trivial), then you should feel free to play with compiler flags and send email to the list about what you find; if the pain threshold isn't too high, we can fix the warnings and add the

Re: [PULL] button mapping fix and unconstify patches

2014-02-09 Thread Gaetan Nadon
On 14-02-09 02:42 PM, Alan Coopersmith wrote: On 02/ 9/14 11:12 AM, Keith Packard wrote: Gaetan Nadon mems...@videotron.ca writes: Ok, so the recent patch wave did not also address all commented out warning flags. Just asking in case some of them needed to be re-instated. With Alan's

Re: [PATCH RESEND2 xserver] Default font path: remove the check for ${sysconfdir}/X11/fontpath.d

2014-02-08 Thread Gaetan Nadon
On 14-02-07 10:05 PM, Keith Packard wrote: Gaetan Nadon mems...@videotron.ca writes: The location ${sysconfdir}/X11/fontpath.d is unknown at configuration time (only at make time) as evidenced by the configuration output: checking for ${prefix}/etc/X11/fontpath.d... no Unlike font-util

Re: [PATCH] glx: Ignore unused function warnings

2014-02-08 Thread Gaetan Nadon
On 14-02-07 07:10 PM, Keith Packard wrote: indirect_dispatch_swap.c, which is autogenerated from a mesa script, contains the function 'bswap_CARD64', which is never used in the file. While it would be nice to fix this upstream, that's hard, so this change causes these warnings to be ignored

[PATCH util-macros 2/2] Add XORG_WITH_M4 macro

2014-02-08 Thread Gaetan Nadon
From: Arnaud Fontaine ar...@debian.org Originally from XCB, this macro checks for the presence of m4 or gm4 which supports -I dir. The AC_PATH_PROGS_FEATURE_CHECK autoconf macro requires autoconf 2.62. Signed-off-by: Gaetan Nadon mems...@videotron.ca --- xorg-macros.m4.in | 23

Re: [RFC] X.Org minimum requirements for Autotools policy review

2014-02-06 Thread Gaetan Nadon
On 13-10-24 02:17 PM, Gaetan Nadon wrote: Recommendations 1. We should prereq libtool 2.2 as a minimum (available since 2008). This has been implemented in the server and I haven't heard of any complaints. 1. We should prereq autoconf 2.62 as a minimum (available since 2008). Keeps us

[PATCH util-macros] Bump minimum Autoconf required version to 2.62

2014-02-06 Thread Gaetan Nadon
://www.x.org/wiki/Building_the_X_Window_System/?updated#index2h3 Discussion on xorg minimum autotools requirements: http://lists.x.org/archives/xorg-devel/2013-October/038325.html Signed-off-by: Gaetan Nadon mems...@videotron.ca --- configure.ac |2 +- 1 file changed, 1 insertion(+), 1 deletion

Re: [PULL] button mapping fix and unconstify patches

2014-02-05 Thread Gaetan Nadon
On 14-02-05 03:25 AM, Keith Packard wrote: With this hidesight, I think now what I should have done is to cast at the assignment of the string constant and leave the types alone. Then, compiling with -Wcast-qual would have quickly identified all places in the code which needed fixes like those

Re: [PATCH xkbcomp] man: replace default include directory with the one from configure

2014-02-04 Thread Gaetan Nadon
On 14-02-02 04:31 PM, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- man/Makefile.am | 2 ++ man/xkbcomp.man | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) Reviewed-by: Gaetan Nadon mems...@videotron.ca

Re: [PATCH xfs 1/3] Warning fixes.

2014-02-01 Thread Gaetan Nadon
-by: Gaetan Nadon mems...@videotron.ca ___ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel

Re: [PATCH] Replace 'pointer' with the equivalent 'void *'.

2014-01-29 Thread Gaetan Nadon
On 14-01-27 04:59 PM, Keith Packard wrote: Cool. Any comments on the libXfont patch, or should I just push it? I had not tried it (confusion on my part), but it fails for me as it does for Knut. ___ xorg-devel@lists.x.org: X.Org development Archives:

  1   2   3   4   5   6   7   8   9   10   >