unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Germán Arias" <german@xelalug.org>
To: emacs-devel@gnu.org
Subject: Re: [PATCH] Changes to work with GNUstep
Date: Fri, 26 Aug 2011 17:39:28 -0600	[thread overview]
Message-ID: <1314401968.5663.3.camel@german-desktop> (raw)
In-Reply-To: <1314143667.5655.5.camel@german-desktop>

On mar, 2011-08-23 at 17:54 -0600, Germán Arias wrote:
> Below is a patch to configure and compile with latest gnustep packages.
> Let me know if this is OK or I need change something. 
> 
> This patch define _NATIVE_OBJC_EXCEPTIONS, that is needed to build
> gnustep's apps without using gnustep-make. And add the flags to found
> the gnustep headers. Necessary if gnustep is installed with gnustep
> layout instead FHS.
> 

Can someone check this patch, please? Apply it or refuse it? I resend
it, because configure.in has changed in trunk. Thanks (I don't have
write permission in repository).

=== modified file 'configure.in'
*** configure.in	2011-08-26 07:12:16 +0000
--- configure.in	2011-08-26 22:06:04 +0000
*************** test "X${with_gnustep_conf}" != X && tes
*** 219,224 ****
--- 219,232 ----
  test "X$GNUSTEP_CONFIG_FILE" = "X" && \
       GNUSTEP_CONFIG_FILE=/etc/GNUstep/GNUstep.conf
  
+ GS_LOCAL_HEADERS="$(. $GNUSTEP_CONFIG_FILE; echo
$GNUSTEP_LOCAL_HEADERS)"
+ 
+ if grep "BASE_NATIVE_OBJC_EXCEPTIONS     1"
${GS_LOCAL_HEADERS}/GNUstepBase/GSConfig.h; then
+   AC_DEFINE(_NATIVE_OBJC_EXCEPTIONS, 1, [Synchronize native exceptions
with gnustep-base.])
+ else
+   AC_DEFINE(_NATIVE_OBJC_EXCEPTIONS, 0, [Synchronize native exceptions
with gnustep-base.])
+ fi
+ 
  AC_ARG_ENABLE(ns-self-contained,
  [AS_HELP_STRING([--disable-ns-self-contained],
                  [disable self contained build under NeXTstep])],
*************** tmp_CFLAGS="$CFLAGS"
*** 1499,1504 ****
--- 1507,1515 ----
  CPPFLAGS="$CPPFLAGS -x objective-c"
  CFLAGS="$CFLAGS -x objective-c"
  TEMACS_LDFLAGS2="\${LDFLAGS}"
+ if _NATIVE_OBJC_EXCEPTIONS=1; then
+    CFLAGS="$CFLAGS -fobjc-exceptions"
+ fi
  dnl I don't think it's especially important, but src/Makefile.in
  dnl (now the only user of ns_appdir) used to go to the trouble of
adding a
  dnl trailing "/" to it, so now we do it here.
*************** if test "${HAVE_NS}" = "yes"; then
*** 2571,2578 ****
      AC_DEFINE(NS_IMPL_GNUSTEP, 1, [Define to 1 if you are using NS
windowing under GNUstep.])
      # See also .m.o rule in Makefile.in */
      # FIXME: are all these flags really needed?  Document here why.
*/
!     C_SWITCH_X_SYSTEM="-D_REENTRANT -fPIC -fno-strict-aliasing"
      GNU_OBJC_CFLAGS="-fgnu-runtime -Wno-import
-fconstant-string-class=NSConstantString -DGNUSTEP_BASE_LIBRARY=1
-DGNU_GUI_LIBRARY=1 -DGNU_RUNTIME=1 -DGSWARN -DGSDIAGNOSE"
    fi
    if test "${NS_HAVE_NSINTEGER}" = "yes"; then
      AC_DEFINE(NS_HAVE_NSINTEGER, 1, [Define to 1 if `NSInteger' is
defined.])
--- 2582,2592 ----
      AC_DEFINE(NS_IMPL_GNUSTEP, 1, [Define to 1 if you are using NS
windowing under GNUstep.])
      # See also .m.o rule in Makefile.in */
      # FIXME: are all these flags really needed?  Document here why.
*/
!     C_SWITCH_X_SYSTEM="-D_REENTRANT -fPIC -fno-strict-aliasing -I
${GNUSTEP_SYSTEM_HEADERS} ${GNUSTEP_LOCAL_HEADERS}"
      GNU_OBJC_CFLAGS="-fgnu-runtime -Wno-import
-fconstant-string-class=NSConstantString -DGNUSTEP_BASE_LIBRARY=1
-DGNU_GUI_LIBRARY=1 -DGNU_RUNTIME=1 -DGSWARN -DGSDIAGNOSE"
+     if _NATIVE_OBJC_EXCEPTIONS=1; then
+        GNU_OBJC_CFLAGS="$GNU_OBJC_CFLAGS -fobjc-exceptions"
+     fi
    fi
    if test "${NS_HAVE_NSINTEGER}" = "yes"; then
      AC_DEFINE(NS_HAVE_NSINTEGER, 1, [Define to 1 if `NSInteger' is
defined.])





  reply	other threads:[~2011-08-26 23:39 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-23 23:54 [PATCH] Changes to work with GNUstep Germán Arias
2011-08-26 23:39 ` Germán Arias [this message]
2011-08-27  0:11   ` David De La Harpe Golden
2011-09-15  3:32   ` Glenn Morris

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1314401968.5663.3.camel@german-desktop \
    --to=german@xelalug.org \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).