From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Ali Bahrami Newsgroups: gmane.emacs.bugs Subject: bug#1191: How to contribute back changes for Emacs on Solaris Date: Fri, 17 Oct 2008 10:27:49 -0600 Message-ID: <48F8BD05.5010102@emvision.com> References: <48F80A14.7010502@emvision.com> <200810170622.m9H6Mipv013865@mothra.ics.uci.edu> Reply-To: Ali Bahrami , 1191@emacsbugs.donarmstrong.com NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; format=flowed; charset=ISO-8859-1 Content-Transfer-Encoding: 7BIT X-Trace: ger.gmane.org 1224262282 4810 80.91.229.12 (17 Oct 2008 16:51:22 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Fri, 17 Oct 2008 16:51:22 +0000 (UTC) Cc: 1191@emacsbugs.donarmstrong.com To: Dan Nicolaescu Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Fri Oct 17 18:52:21 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 1KqsXt-0004Fo-RF for geb-bug-gnu-emacs@m.gmane.org; Fri, 17 Oct 2008 18:51:26 +0200 Original-Received: from localhost ([127.0.0.1]:48514 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KqsWo-0008Mi-Pg for geb-bug-gnu-emacs@m.gmane.org; Fri, 17 Oct 2008 12:50:18 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1KqsWa-0008IN-GR for bug-gnu-emacs@gnu.org; Fri, 17 Oct 2008 12:50:04 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1KqsWZ-0008I2-0q for bug-gnu-emacs@gnu.org; Fri, 17 Oct 2008 12:50:03 -0400 Original-Received: from [199.232.76.173] (port=44666 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1KqsWY-0008Hx-Qx for bug-gnu-emacs@gnu.org; Fri, 17 Oct 2008 12:50:02 -0400 Original-Received: from rzlab.ucr.edu ([138.23.92.77]:56487) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1KqsWY-0008T8-4S for bug-gnu-emacs@gnu.org; Fri, 17 Oct 2008 12:50:02 -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 m9HGo0M9027379; Fri, 17 Oct 2008 09:50:00 -0700 Original-Received: (from debbugs@localhost) by rzlab.ucr.edu (8.13.8/8.13.8/Submit) id m9HGZ3rh023759; Fri, 17 Oct 2008 09:35:03 -0700 X-Loop: don@donarmstrong.com Resent-From: Ali Bahrami Original-Sender: Ali.Bahrami@Sun.COM Resent-To: bug-submit-list@donarmstrong.com Resent-CC: Emacs Bugs Resent-Date: Fri, 17 Oct 2008 16:35:03 +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.122426088022488 (code B ref 1191); Fri, 17 Oct 2008 16:35:03 +0000 Original-Received: (at 1191) by emacsbugs.donarmstrong.com; 17 Oct 2008 16:28:00 +0000 Original-Received: from brmea-mail-2.sun.com (brmea-mail-2.Sun.COM [192.18.98.43]) by rzlab.ucr.edu (8.13.8/8.13.8/Debian-3) with ESMTP id m9HGRt3p022482 for <1191@emacsbugs.donarmstrong.com>; Fri, 17 Oct 2008 09:27:56 -0700 Original-Received: from fe-amer-10.sun.com ([192.18.109.80]) by brmea-mail-2.sun.com (8.13.6+Sun/8.12.9) with ESMTP id m9HGRsHX018702 for <1191@emacsbugs.donarmstrong.com>; Fri, 17 Oct 2008 16:27:54 GMT Original-Received: from conversion-daemon.mail-amer.sun.com by mail-amer.sun.com (Sun Java System Messaging Server 6.2-8.04 (built Feb 28 2007)) id <0K8W00F014Y2YC00@mail-amer.sun.com> (original mail from ali_gnu@emvision.com) for 1191@emacsbugs.donarmstrong.com; Fri, 17 Oct 2008 10:27:54 -0600 (MDT) Original-Received: from [172.20.25.67] by mail-amer.sun.com (Sun Java System Messaging Server 6.2-8.04 (built Feb 28 2007)) with ESMTPSA id <0K8W00FS75QDJGF0@mail-amer.sun.com>; Fri, 17 Oct 2008 10:27:49 -0600 (MDT) In-reply-to: <200810170622.m9H6Mipv013865@mothra.ics.uci.edu> User-Agent: Thunderbird 2.0.0.16 (X11/20080922) X-detected-operating-system: by monty-python.gnu.org: GNU/Linux 2.6 (newer, 3) Resent-Date: Fri, 17 Oct 2008 12:50:03 -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:21621 Archived-At: Dan Nicolaescu wrote: > Ali Bahrami writes: > > > This message is a follow up to one I sent to emacs-devel > > last July: > > > > http://lists.gnu.org/archive/html/emacs-devel/2008-07/msg00070.html > > > > The question had to do with why emacs wasn't using dldump() > > to produce the dumped executable under Solaris. It had been using it > > up until 2002, at which point it was commented out, and it wasn't > > clear why. No one knew the reason, and the consensus was "try it and see". > > > > Since then, I've done some work to integrate emacs with > > OpenSolaris as a standard package. I turned on dldump(), > > and have been using the emacs binaries built with it daily > > since July without issue. > > > > So now, I have some things I'd like to send back up stream > > and get integrated into the emacs distribution: > > > > - Man pages for things installed to /usr/bin that didn't > > already have a manpage: > > b2m.1 > > ebrowse.1 > > grep-changelog.1 > > rcs-checkin.1 > > - Small changes to config.guess and configure to properly > > detect amd64 hardware running Solaris. > > - Use dldump() instead of the generic ELF dump code > > under Solaris. > > - Use the system malloc under Solaris (at least with > > amd64, emacs crashes without this). > > > > The manpages can be viewed at: > > > > http://src.opensolaris.org/source/xref/sfw/usr/src/cmd/emacs/augment/man/man1/b2m.1 > > http://src.opensolaris.org/source/xref/sfw/usr/src/cmd/emacs/augment/man/man1/ebrowse.1 > > http://src.opensolaris.org/source/xref/sfw/usr/src/cmd/emacs/augment/man/man1/grep-changelog.1 > > http://src.opensolaris.org/source/xref/sfw/usr/src/cmd/emacs/augment/man/man1/rcs-checkin.1 > > Hopefully someone else can address these changes, I'll address the rest. > > > and the patches at: > > > > http://src.opensolaris.org/source/xref/sfw/usr/src/cmd/emacs/augment/patch/config.guess.patch > > Please send this one to config-patches@gnu.org. > We don't modify this file directly in emacs, it gets updated from time > to time from: > http://savannah.gnu.org/cgi-bin/viewcvs/*checkout*/config/config/config.guess > > > http://src.opensolaris.org/source/xref/sfw/usr/src/cmd/emacs/augment/patch/configure.patch > > This one is fine, but it depends on the previous one being installed by > the config.guess maintainers. > BTW, do you know anything about emacs running on powerpc*-*-solaris2* > or rs6000-*-solaris2* ? configure.in mentions them. > > > http://src.opensolaris.org/source/xref/sfw/usr/src/cmd/emacs/augment/patch/dldump.patch > > Last time when you posted this you said you were going to test it on > multiple Solaris versions. Have you managed to do that? This is also > valid for the next patch. > > > http://src.opensolaris.org/source/xref/sfw/usr/src/cmd/emacs/augment/patch/sysmalloc.patch > > > > The patches are for emacs 22.1. I've examined 22.3, and the > > changes are applicable there as well. If 22.1 diffs are OK, > > that's easiest for me, but if not, then please let me know what > > I need to supply. These diffs are all very short, affecting a > > small number of lines in each file. > > AFAIK the 22.x branch is closed, there will not be any more releases > from that branch. But we could install your changes on HEAD. > > Thanks > > --dan Hi Dan, I will send the config.guess patch to config-patches@gnu.org today. In the meantime, the change to configure should be safe to make immediately. I'm only concerned about getting these changes into HEAD. My patches are for 22.1, but only because that's the version I used for this work. So that would be great. If you can use these old patches as a basis for making the necessary changes (which are small) to HEAD, then that would be a big time saver for me, and much appreciated. If you need specific patches against HEAD, then point me at it and I'll put some together. I only know a little about emacs on the power architecture: Sun funded a port during the 90's. As a point of irrelevant trivia, it was a little endian system, and not big endian as one might expect. For reasons that are lost in time, the port was completed, and a few pre-releases were made, but it never shipped as a standard product, wasn't updated with the main branch, and quickly faded away. My only quess is that powerpc*-*-solaris2* and rs6000-*-solaris2* (the rs6000 also being a Power architecture system) date from this port, and have probably never had more than a small handful of users --- probably none for a decade or so. There are people involved in a PowerPC port with OpenSolaris now (big endian, a completely different deal) starting with the modern OpenSolaris code base. I'm not sure if there's much overlap between the old stuff in emacs for power on Solaris, and what would be needed for a modern version. There's probably more affinity with the modern Linux ppc definitions. About testing different Solaris versions, I remember the question being asked. I thought I replied at the time, but can't be sure. In any event, the answer is no. My interest is with the modern Solaris, which is OpenSolaris, and Solaris 10. That's all I've tested, and I apologize for not having said that directly. There are several reasons: - Maximizing the value of the time I spend on this project - No evidence of an actual problem with dldump for older Solaris, including the fact that xemacs has apparently been using it all along - The fact that the generic ELF dump code does work, and can be used for those old releases. It isn't as nice as the dldump'd version, but the resulting executables would run as well as they ever did. Hence, there's an easy fallback if something did come up. None of which, I admit, is actual testing. I feel that the risk of switching the old versions to dldump() is minimal. However, if you feel more comfortable applying this change such that it only applies to Solaris 10 and newer, that would also be fine. Thanks... - Ali