From owner-freebsd-gnome@FreeBSD.ORG Mon Oct 6 11:40:29 2003 Return-Path: Delivered-To: freebsd-gnome@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id F419C16A4B3 for ; Mon, 6 Oct 2003 11:40:28 -0700 (PDT) Received: from mail.liwing.de (mail.liwing.de [213.70.188.162]) by mx1.FreeBSD.org (Postfix) with ESMTP id 0682D43FE5 for ; Mon, 6 Oct 2003 11:40:27 -0700 (PDT) (envelope-from rehsack@liwing.de) Received: (qmail 88411 invoked from network); 6 Oct 2003 18:40:26 -0000 Received: from stingray.liwing.de (HELO liwing.de) ([213.70.188.164]) (envelope-sender ) by mail.liwing.de (qmail-ldap-1.03) with SMTP for ; 6 Oct 2003 18:40:26 -0000 Message-ID: <3F81B719.4000604@liwing.de> Date: Mon, 06 Oct 2003 18:40:25 +0000 From: Jens Rehsack User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.4) Gecko/20031004 X-Accept-Language: de-de, de, en-us, en MIME-Version: 1.0 To: Joe Marcus Clarke References: <200310061745.h96Hjwr4037843@helo.liwing.de> <1065463144.748.58.camel@gyros> In-Reply-To: <1065463144.748.58.camel@gyros> Content-Type: multipart/mixed; boundary="------------040600030504050603010109" cc: FreeBSD GNOME Users Subject: Re: [PATCH] x11-toolkits/scintilla misses gthread library X-BeenThere: freebsd-gnome@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: GNOME for FreeBSD -- porting and maintaining List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 06 Oct 2003 18:40:29 -0000 This is a multi-part message in MIME format. --------------040600030504050603010109 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Joe Marcus Clarke wrote: > I don't think your patch is correct. Simply linking in gthread will not > work as it will leave unresolved symbols to pthread*. You might want to > add a call to pkg-config --libs gthread instead of linking in the > libraries explicitly. I sent a new patch to . GNATS seems a little bit slow today, so here's the patch for review again :-) Jens --------------040600030504050603010109 Content-Type: text/plain; name="patch-add-libgthread" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="patch-add-libgthread" Index: x11-toolkits/scintilla/files/patch-aa =================================================================== diff -u x11-toolkits/scintilla/files/patch-aa.orig x11-toolkits/scintilla/files/patch-aa --- x11-toolkits/scintilla/files/patch-aa.orig Mon Oct 6 17:21:45 2003 +++ x11-toolkits/scintilla/files/patch-aa Mon Oct 6 18:03:06 2003 @@ -1,5 +1,5 @@ ---- makefile.orig Sun Sep 21 11:19:30 2003 -+++ makefile Sun Sep 28 11:48:04 2003 +--- makefile.orig Sun Sep 21 02:19:30 2003 ++++ makefile Mon Oct 6 18:02:43 2003 @@ -8,8 +8,8 @@ # To force GTK+ 1 build, define GTK1 on the make command line. @@ -21,7 +21,7 @@ vpath %.h ../src ../include vpath %.cxx ../src -@@ -33,9 +34,9 @@ +@@ -33,29 +34,33 @@ endif ifdef DEBUG @@ -33,16 +33,24 @@ endif # If explicit setting of GTK1 or GTK2 then use that else look for -@@ -44,7 +45,7 @@ + # pkg-config which is an OK indication that GTK2 is available + ifdef GTK2 CONFIGFLAGS=pkg-config --cflags gtk+-2.0 ++MORELIBS=pkg-config --libs gthread-2.0 else ifdef GTK1 -CONFIGFLAGS=gtk-config --cflags +CONFIGFLAGS=pkg-config --cflags gtk+ ++MORELIBS=pkg-config --libs gthread else ifneq (,$(findstring /,$(shell whereis -b pkg-config))) CONFIGFLAGS=pkg-config --cflags gtk+-2.0 -@@ -55,7 +56,7 @@ ++MORELIBS=pkg-config --libs gthread-2.0 + else + CONFIGFLAGS=gtk-config --cflags ++MORELIBS=pkg-config --libs gthread + endif + endif endif .cxx.o: @@ -51,7 +59,7 @@ #++Autogenerated -- run src/LexGen.py to regenerate #**LEXOBJS=\\\n\(\*.o \) -@@ -69,7 +70,7 @@ +@@ -69,7 +74,7 @@ # The LEXOBJS have to be treated specially as the functions in them are not called from external code @@ -60,16 +68,16 @@ clean: rm -f *.o $(COMPLIB) -@@ -81,8 +82,10 @@ +@@ -81,8 +86,10 @@ ScintillaBase.o ContractionState.o Editor.o ExternalLexer.o PropSet.o PlatGTK.o \ KeyMap.o LineMarker.o ScintillaGTK.o CellBuffer.o ViewStyle.o \ RESearch.o Style.o Indicator.o AutoComplete.o UniConversion.o XPM.o - $(AR) rc $@ $^ - $(RANLIB) $@ -+ $(AR) `$(CONFIGFLAGS:cflags=libs)` -o $@ $^ ++ $(AR) `$(CONFIGFLAGS:cflags=libs)` `${MORELIBS}` -o $@ $^ + + $(LEXRLIB): $(LEXOBJS) -+ $(AR) `$(CONFIGFLAGS:cflags=libs)` -o $@ $^ ++ $(AR) `$(CONFIGFLAGS:cflags=libs)` `${MORELIBS}` -o $@ $^ # Automatically generate header dependencies with "make deps" include deps.mak --------------040600030504050603010109--