From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Dan Nicolaescu Newsgroups: gmane.emacs.bugs Subject: bug#1191: How to contribute back changes for Emacs on Solaris Date: Mon, 20 Oct 2008 15:29:18 -0700 (PDT) Message-ID: <200810202229.m9KMTIjS025889@mothra.ics.uci.edu> References: <48F80A14.7010502@emvision.com> <200810170622.m9H6Mipv013865@mothra.ics.uci.edu> <48F8BD05.5010102@emvision.com> <200810172352.m9HNqwSb016244@mothra.ics.uci.edu> <48FAA706.2050806@emvision.com> Reply-To: Dan Nicolaescu , 1191@emacsbugs.donarmstrong.com NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: ger.gmane.org 1224545820 29317 80.91.229.12 (20 Oct 2008 23:37:00 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 20 Oct 2008 23:37:00 +0000 (UTC) Cc: 1191@emacsbugs.donarmstrong.com To: Ali Bahrami Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Tue Oct 21 01:37:58 2008 connect(): Connection refused Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.50) id 1Ks3ap-0003fW-Uk for geb-bug-gnu-emacs@m.gmane.org; Tue, 21 Oct 2008 00:51:20 +0200 Original-Received: from localhost ([127.0.0.1]:49248 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Ks3Zk-0000Pr-5c for geb-bug-gnu-emacs@m.gmane.org; Mon, 20 Oct 2008 18:50:12 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Ks3Zf-0000PK-EN for bug-gnu-emacs@gnu.org; Mon, 20 Oct 2008 18:50:07 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Ks3Ze-0000P7-MW for bug-gnu-emacs@gnu.org; Mon, 20 Oct 2008 18:50:07 -0400 Original-Received: from [199.232.76.173] (port=49392 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Ks3Ze-0000P4-Fj for bug-gnu-emacs@gnu.org; Mon, 20 Oct 2008 18:50:06 -0400 Original-Received: from rzlab.ucr.edu ([138.23.92.77]:37045) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1Ks3Zd-0003We-UC for bug-gnu-emacs@gnu.org; Mon, 20 Oct 2008 18:50:06 -0400 Original-Received: from rzlab.ucr.edu (rzlab.ucr.edu [127.0.0.1]) by rzlab.ucr.edu (8.13.8/8.13.8/Debian-3) with ESMTP id m9KMo18u028191; Mon, 20 Oct 2008 15:50:01 -0700 Original-Received: (from debbugs@localhost) by rzlab.ucr.edu (8.13.8/8.13.8/Submit) id m9KMZ4UR024605; Mon, 20 Oct 2008 15:35:04 -0700 X-Loop: don@donarmstrong.com Resent-From: Dan Nicolaescu Resent-To: bug-submit-list@donarmstrong.com Resent-CC: Emacs Bugs Resent-Date: Mon, 20 Oct 2008 22:35:04 +0000 Resent-Message-ID: Resent-Sender: don@donarmstrong.com X-Emacs-PR-Message: report 1191 X-Emacs-PR-Package: emacs X-Emacs-PR-Keywords: Original-Received: via spool by 1191-submit@emacsbugs.donarmstrong.com id=B1191.122454177223243 (code B ref 1191); Mon, 20 Oct 2008 22:35:04 +0000 Original-Received: (at 1191) by emacsbugs.donarmstrong.com; 20 Oct 2008 22:29:32 +0000 Original-Received: from barrelv2.ics.uci.edu (barrelv2.ics.uci.edu [128.195.1.114]) by rzlab.ucr.edu (8.13.8/8.13.8/Debian-3) with ESMTP id m9KMTTPO023236 for <1191@emacsbugs.donarmstrong.com>; Mon, 20 Oct 2008 15:29:30 -0700 Original-Received: from mothra.ics.uci.edu (mothra.ics.uci.edu [128.195.6.93]) by barrelv2.ics.uci.edu (8.13.7+Sun/8.13.7) with ESMTP id m9KMTIxQ002417; Mon, 20 Oct 2008 15:29:18 -0700 (PDT) Original-Received: (from dann@localhost) by mothra.ics.uci.edu (8.13.8+Sun/8.13.6/Submit) id m9KMTIjS025889; Mon, 20 Oct 2008 15:29:18 -0700 (PDT) In-Reply-To: <48FAA706.2050806@emvision.com> (Ali Bahrami's message of "Sat, 18 Oct 2008 21:18:30 -0600") Original-Lines: 143 X-ICS-MailScanner-Information: Please contact the ISP for more information X-ICS-MailScanner-ID: m9KMTIxQ002417 X-ICS-MailScanner: Found to be clean X-ICS-MailScanner-SpamCheck: not spam, SpamAssassin (score=0.521, required 5, autolearn=disabled, ALL_TRUSTED -1.44, URIBL_BLACK 1.96) X-ICS-MailScanner-From: dann@mothra.ics.uci.edu X-detected-operating-system: by monty-python.gnu.org: GNU/Linux 2.6 (newer, 3) Resent-Date: Mon, 20 Oct 2008 18:50:07 -0400 X-BeenThere: bug-gnu-emacs@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:21715 Archived-At: Ali Bahrami writes: > Dan Nicolaescu wrote: > > > > > I don't feel comfortable doing these changes without any testing at > > all. In that case, the best would be to add a new file > > emacs/src/s/sol2-10.h that includes sol2-6.h and also contains your > > changes. > > Then change configure.in to use that new file for Solaris 10+. > > I've done that, and built the result on Solaris Nevada (amd64), > Solaris 10 (sparc), and Solaris 9 (x86). It seems to work properly, > and to use dldump() for S10 and Nevada (aka OpenSolaris), while > using the generic dump for S9. > > I rolled the amd64 configure change into this as well --- I needed it > to test, and it makes sense to do the complete deal in one go. > > Here is the contents of src/s/sol2-10.h, in between the '====' lines: > > ========================================================================== > /* Handle Solaris 2.10 */ > > #include "sol2-6.h" > > /* > * Use the Solaris dldump() function to dump emacs, instead of Please mention here that the dldump function is used in unexsol.c, otherwise people would have to guess what is going on. > * the generic unexelf code. The resulting binary has a complete > * symbol table, and is better for debugging and other observability > * tools (debuggers, pstack, etc). > * Note: Emacs used to use dldump() before, but this was changed > * in revision 1.3 of sol2-6.h, on September 13, 2002, with the comment: > * > * dldump does not handle all the extensions used by GNU ld > * > * It is not clear what extensions this refers to. Testing with > * OpenSolaris shows that it works on newer Solaris versions. > * If a problem surfaces, please post the details to the OpenSolaris > * mailing list given above. This is not useful, you can omit it. > * It is likely that dldump() works with older Solaris too, > * but this has not been tested, and so, this change is for S10 ^^^ Solaris 10 > * and newer only at this time. > */ > > #undef UNEXEC > #define UNEXEC unexsol.o You also had a change to use system malloc. Don't you want that in here too? > ========================================================================== > > And here are the context diffs for the changes to configure.in > > ========================================================================== > *** configure.in.orig Sat Oct 18 17:40:27 2008 > --- configure.in Sat Oct 18 18:27:43 2008 > *************** > *** 974,980 **** > ;; > > *-auspex-sunos* | *-sun-sunos* | *-sun-bsd* | *-sun-solaris* \ > ! | i[3456]86-*-solaris2* | i[3456]86-*-sunos5* | powerpc*-*-solaris2* \ > | rs6000-*-solaris2*) > case "${canonical}" in > m68*-sunos1* ) machine=sun1 ;; > --- 974,980 ---- > ;; > > *-auspex-sunos* | *-sun-sunos* | *-sun-bsd* | *-sun-solaris* \ > ! | i[3456]86-*-solaris2* | x86_64-*-solaris2* | i[3456]86-*-sunos5* | power > pc*-*-solaris2* \ > | rs6000-*-solaris2*) > case "${canonical}" in > m68*-sunos1* ) machine=sun1 ;; > *************** > *** 988,993 **** > --- 988,1001 ---- > sparc* ) machine=sparc ;; > * ) unported=yes ;; > esac > + ## FIXME: make this into a proper fix that checks the compiler type, > + ## rather than relying on path. Or is /usr/ccs/lib/cpp a bad default now? > + if [ "x$CC" = x/opt/SUNWspro/bin/cc ]; then > + ## -Xs prevents spurious whitespace. > + SOLARIS_NON_GNU_CPP="/opt/SUNWspro/bin/cc -E -Xs" > + else > + SOLARIS_NON_GNU_CPP=/usr/ccs/lib/cpp > + fi > case "${canonical}" in > ## The Sun386 didn't get past 4.0. > i[3456]86-*-sunos4 ) opsys=sunos4-0 ;; > *************** > *** 1028,1044 **** > NON_GNU_CPP=/usr/ccs/lib/cpp > RANLIB="ar -ts" > ;; > ! *-sunos5* | *-solaris* ) > opsys=sol2-6 > ! ## FIXME: make this into a proper fix that checks the compiler type, > ! ## rather than relying on path. Or is /usr/ccs/lib/cpp a bad default now? > ! if [ "x$CC" = x/opt/SUNWspro/bin/cc ]; then > ! ## -Xs prevents spurious whitespace. > ! NON_GNU_CPP="/opt/SUNWspro/bin/cc -E -Xs" > ! else > ! NON_GNU_CPP=/usr/ccs/lib/cpp > ! fi > ;; > * ) opsys=bsd4-2 ;; > esac > ## Watch out for a compiler that we know will not work. > --- 1036,1049 ---- > NON_GNU_CPP=/usr/ccs/lib/cpp > RANLIB="ar -ts" > ;; > ! *-sunos5.[7-9] | *-solaris2.[7-9] ) > opsys=sol2-6 > ! NON_GNU_CPP=${SOLARIS_NON_GNU_CPP} > ;; > + *-sunos5* | *-solaris* ) > + opsys=sol2-10 > + NON_GNU_CPP=${SOLARIS_NON_GNU_CPP} > + ;; > * ) opsys=bsd4-2 ;; > esac > ## Watch out for a compiler that we know will not work. > ========================================================================== Can you generate this vs configure.in from CVS HEAD, that file has changed a bit.