From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: Building Emacs from a new MinGW environment Date: Tue, 17 Sep 2013 11:30:28 +0300 Message-ID: <83d2o7q28r.fsf@gnu.org> References: <83sixwfdjk.fsf@gnu.org> <834n9nhhp6.fsf@gnu.org> <83ppsbfoe7.fsf@gnu.org> <83mwnffkom.fsf@gnu.org> <83k3ijfhk0.fsf@gnu.org> <83hadnf7gs.fsf@gnu.org> <83hadmqvql.fsf@gnu.org> <83txhkpszv.fsf@gnu.org> <83pps8pnc9.fsf@gnu.org> <83ob7splia.fsf@gnu.org> <83k3ifq5o5.fsf@gnu.org> Reply-To: Eli Zaretskii NNTP-Posting-Host: plane.gmane.org X-Trace: ger.gmane.org 1379406646 16962 80.91.229.3 (17 Sep 2013 08:30:46 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 17 Sep 2013 08:30:46 +0000 (UTC) Cc: emacs-devel@gnu.org To: Dani Moncayo Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Sep 17 10:30:49 2013 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1VLqgL-0002BK-4U for ged-emacs-devel@m.gmane.org; Tue, 17 Sep 2013 10:30:49 +0200 Original-Received: from localhost ([::1]:38760 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VLqgK-0006IB-M8 for ged-emacs-devel@m.gmane.org; Tue, 17 Sep 2013 04:30:48 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:52909) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VLqgB-0006Dn-NP for emacs-devel@gnu.org; Tue, 17 Sep 2013 04:30:44 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VLqg6-0006H6-8V for emacs-devel@gnu.org; Tue, 17 Sep 2013 04:30:39 -0400 Original-Received: from mtaout22.012.net.il ([80.179.55.172]:45523) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VLqg6-0006Gv-0d for emacs-devel@gnu.org; Tue, 17 Sep 2013 04:30:34 -0400 Original-Received: from conversion-daemon.a-mtaout22.012.net.il by a-mtaout22.012.net.il (HyperSendmail v2007.08) id <0MT900700G24EL00@a-mtaout22.012.net.il> for emacs-devel@gnu.org; Tue, 17 Sep 2013 11:30:30 +0300 (IDT) Original-Received: from HOME-C4E4A596F7 ([87.69.4.28]) by a-mtaout22.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0MT9007Q7GYUCD60@a-mtaout22.012.net.il>; Tue, 17 Sep 2013 11:30:30 +0300 (IDT) In-reply-to: X-012-Sender: halo1@inter.net.il X-detected-operating-system: by eggs.gnu.org: Solaris 10 X-Received-From: 80.179.55.172 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:163383 Archived-At: > Date: Tue, 17 Sep 2013 10:17:38 +0200 > From: Dani Moncayo > Cc: Emacs development discussions > > > Change this: > > > > case "${srcdir}" in > > [[\\/]]* | ?:[[\\/]]*) ;; > > > > into this: > > > > case "${srcdir}" in > > [[\\/]]* | ?:[[\\/]]*) > > test "$MSYSTEM" = "MINGW32" && srcdir=`(cd "$srcdir"; pwd -W)` > > ;; > > > > This fix, if it works, is better, since it will fix also the stuff > > that gets written into .gdbinit file in the build directory (it didn't > > work for you because an MSYS file name was written there, and the > > MinGW build of GDB doesn't grok that). > > This does't work in my case, because at that point, "${srcdir}" > contains the relative path "../emacs.git", not something like > "/c/whatever" or "c:/whatever". OK, then we could put this line test "$MSYSTEM" = "MINGW32" && srcdir=`(cd "$srcdir"; pwd -W)` immediately below the 'esac' in the fragment I've shown. Does that do the job for you? > I think that the right fix is to replace *all* the treatment of > patterns like the above ("/c/whatever" or "c:/whatever") with logic > based on the "pwd -W" feature of MSYS. > > IOW, an MSYS path doesn't have to match none of the above patterns, so > the only reliable way of getting a Windows-native path is with "pwd > -W". > > Therefore, things line "msys_to_w32", "msys_lisppath_to_w32" should be > replaced with the above criteria. I'm not sure I understand what you are suggesting, specifically. Can you show a patch that works for you? In any case, the problem with src/.gdbinit in the build tree still needs to be solved; no amount of changes in the Makefile's can do that, because that file is created by config.status. So we still need something in configure.ac as well. The advantage of my suggestion is that it solves this problem in a single place, once and for all, while what you seem to suggest would need multiple changes in many places. Why is that better?