From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Giorgos Keramidas Newsgroups: gmane.emacs.devel Subject: Re: [PATCH] Unbreak `make install' in leim/ when leim's ${INSTALLDIR} doesn't exist Date: Thu, 12 Oct 2006 16:12:32 +0300 Message-ID: <20061012131232.GA8507@gothmog.pc> References: <549c434ebfb50d1a4ca2.1160608974@gothmog.pc> <20061012123556.GA89997@gothmog.pc> NNTP-Posting-Host: main.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: sea.gmane.org 1160658820 8660 80.91.229.2 (12 Oct 2006 13:13:40 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Thu, 12 Oct 2006 13:13:40 +0000 (UTC) Cc: emacs-devel@gnu.org, Kenichi Handa Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Thu Oct 12 15:13:37 2006 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by ciao.gmane.org with esmtp (Exim 4.43) id 1GY0MP-0004Fy-V1 for ged-emacs-devel@m.gmane.org; Thu, 12 Oct 2006 15:12:30 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1GY0MP-0005wF-Cc for ged-emacs-devel@m.gmane.org; Thu, 12 Oct 2006 09:12:29 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1GY0MC-0005tX-TF for emacs-devel@gnu.org; Thu, 12 Oct 2006 09:12:16 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1GY0MB-0005qV-9I for emacs-devel@gnu.org; Thu, 12 Oct 2006 09:12:16 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1GY0MB-0005qO-4O for emacs-devel@gnu.org; Thu, 12 Oct 2006 09:12:15 -0400 Original-Received: from [62.1.205.36] (helo=igloo.linux.gr) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA:32) (Exim 4.52) id 1GY0UO-0002de-SF for emacs-devel@gnu.org; Thu, 12 Oct 2006 09:20:45 -0400 Original-Received: from gothmog.pc (host5.bedc.ondsl.gr [62.103.39.229]) (authenticated bits=128) by igloo.linux.gr (8.13.8/8.13.8/Debian-2) with ESMTP id k9CDBteW007645 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Thu, 12 Oct 2006 16:11:57 +0300 Original-Received: from gothmog.pc (gothmog [127.0.0.1]) by gothmog.pc (8.13.8/8.13.8) with ESMTP id k9CDCWoa008547; Thu, 12 Oct 2006 16:12:33 +0300 (EEST) (envelope-from keramida@ceid.upatras.gr) Original-Received: (from giorgos@localhost) by gothmog.pc (8.13.8/8.13.8/Submit) id k9CDCWRc008546; Thu, 12 Oct 2006 16:12:32 +0300 (EEST) (envelope-from keramida@ceid.upatras.gr) Original-To: Andreas Schwab Content-Disposition: inline In-Reply-To: X-Hellug-MailScanner: Found to be clean X-Hellug-MailScanner-SpamCheck: not spam, SpamAssassin (score=-2.594, required 5, AWL -0.20, BAYES_00 -2.60, DNS_FROM_RFC_ABUSE 0.20, UNPARSEABLE_RELAY 0.00) X-Hellug-MailScanner-From: keramida@ceid.upatras.gr X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:60660 Archived-At: On 2006-10-12 14:47, Andreas Schwab wrote: >Giorgos Keramidas writes: >> It seems to include this part: >> >> + if [ ! -d ${INSTALLDIR} ] ; then \ >> + ${srcdir}/${dot}${dot}/mkinstalldirs ${INSTALLDIR}; \ >> + else true; fi >> >> We don't really *need* to have an else branch in all statements. We can >> write this as: >> >> + if [ ! -d ${INSTALLDIR} ] ; then \ >> + ${srcdir}/${dot}${dot}/mkinstalldirs ${INSTALLDIR}; \ >> + fi > > Except that some old shells will cause the whole command to fail if the > condition fails. I see. I didn't know about this. Thanks for the clarification :)