GNU bug report logs - #18302
MSYS2 build issues

Previous Next

Package: emacs;

Reported by: Karol Ostrovsky <karol.ostrovsky <at> gmail.com>

Date: Wed, 20 Aug 2014 15:46:02 UTC

Severity: normal

Done: Noam Postavsky <npostavs <at> users.sourceforge.net>

Bug is archived. No further changes may be made.

Full log

View this message in rfc822 format

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ken Brown <kbrown <at> cornell.edu>
Cc: karol.ostrovsky <at> gmail.com, chriszheng99 <at> gmail.com, 18302 <at> debbugs.gnu.org
Subject: bug#18302: MSYS2 build issues
Date: Thu, 21 Aug 2014 22:22:36 +0300

> Date: Thu, 21 Aug 2014 14:38:13 -0400
> From: Ken Brown <kbrown <at> cornell.edu>
> CC: chriszheng99 <at> gmail.com, 18302 <at> debbugs.gnu.org
> 
> > I'd urge the Cygwin Emacs maintainers to revert that special case, but
> > that's their call.  For native Windows builds, I certainly object to
> > introducing this deviation.
> 
> The Cygwin situation is not comparable.  The headers are installed in 
> the standard places.  But Cygwin provides two versions of xpm.h, one in 
> /usr/include/X11 and one in /usr/include/noX.  The Cygwin w32 build 
> needs to add -I/usr/include/noX to CPPFLAGS (and -L/usr/lib/noX to 
> LDFLAGS) in order to pick up the correct version.

No, the solution is to use

  #if defined __CYGWIN__ && !defined HAVE_X_WINDOWS
  #include <noX/xpm.h>
  #else
  #include <xpm.h>
  #endif

The way we work around the problem now will break if someone installs
the standard header files in a place other than /usr/include.

And if you disagree, then at least please put the above explanation in
configure.ac, so that we won't need to have this discussion a year
from now.
This bug report was last modified 7 years and 178 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.