Bug#778176: Revised gcc-5 patch for x11-xserver-utils

2015-07-19 Thread Matthias Klose
On 07/18/2015 10:34 PM, Julien Cristau wrote:
 On Sat, Jul 18, 2015 at 09:39:03 -0700, Daniel Schepler wrote:
 
 On Sat, 7 Mar 2015 08:57:48 +0100, Julien Cristau wrote:
 I don't think that's an acceptable way to fix this.

 How about this?  (This is a patch to be directly quilt imported, not a 
 debdiff.)
 
 How portable is -P?  What change in gcc makes this necessary?  What does
 the output without -P look like with cpp 5 vs 4.9?

hmm, maybe check it out yourself? or read
https://gcc.gnu.org/gcc-5/porting_to.html Preprocessor issues (linked from the
GCC 5 wiki page)?

anyway, here is another approach:
https://github.com/tsuna/boost.m4/commit/32553aaf4d5090da19aa0ec33b936982c685009f

clang seems to support -P as well. I don't know about other compilers.


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#778176: Revised gcc-5 patch for x11-xserver-utils

2015-07-19 Thread Julien Cristau
On Sun, Jul 19, 2015 at 12:51:05 +0200, Matthias Klose wrote:

 clang seems to support -P as well. I don't know about other compilers.
 
sun^Woracle studio seems to have a -P, too.
http://docs.oracle.com/cd/E37069_01/html/E37074/bjapp.html#OSSCGbjarx

Cheers,
Julien


signature.asc
Description: Digital signature


Bug#778176: Revised gcc-5 patch for x11-xserver-utils

2015-07-18 Thread Julien Cristau
On Sat, Jul 18, 2015 at 09:39:03 -0700, Daniel Schepler wrote:

 On Sat, 7 Mar 2015 08:57:48 +0100, Julien Cristau wrote:
  I don't think that's an acceptable way to fix this.
 
 How about this?  (This is a patch to be directly quilt imported, not a 
 debdiff.)

How portable is -P?  What change in gcc makes this necessary?  What does
the output without -P look like with cpp 5 vs 4.9?

Cheers,
Julien


signature.asc
Description: Digital signature


Bug#778176: Revised gcc-5 patch for x11-xserver-utils

2015-07-18 Thread Daniel Schepler
On Sat, 7 Mar 2015 08:57:48 +0100, Julien Cristau wrote:
 I don't think that's an acceptable way to fix this.

How about this?  (This is a patch to be directly quilt imported, not a 
debdiff.)
-- 
Daniel Schepler
Index: x11-xserver-utils-7.7+4/sessreg/man/Makefile.am
===
--- x11-xserver-utils-7.7+4.orig/sessreg/man/Makefile.am
+++ x11-xserver-utils-7.7+4/sessreg/man/Makefile.am
@@ -10,7 +10,7 @@ SUFFIXES = .$(APP_MAN_SUFFIX) .man
 AM_CPPFLAGS = -I$(top_builddir) -I$(top_srcdir)
 filenames.sed: filenames.sed.c
 	$(AM_V_GEN)$(CPP) $(DEFS) $(DEFAULT_INCLUDES) $(INCLUDES) \
-	$(AM_CPPFLAGS) $(CPPFLAGS) $(srcdir)/filenames.sed.c | \
+	$(AM_CPPFLAGS) $(CPPFLAGS) -P $(srcdir)/filenames.sed.c | \
 	$(SED) -n -e '/s|__/ p' -e '/^\/__/ p'  $@
 
 # String replacements in MAN_SUBSTS now come from xorg-macros.m4 via configure