X-Recipient: archive-cygwin AT delorie DOT com
DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:subject:to:references:from:message-id:date
	:mime-version:in-reply-to:content-type; q=dns; s=default; b=oii/
	S9EuADMqva8NmP6pYYaj5MrkqgKerA9b4dmKldLbiySGerFVm2wymlj9b4WYzz/n
	vw9XNts4AjxEWrdBlBE140haKsQs4/Cmnxb+aAbSIf8t1D4DUcgXvav8qBgX3tDA
	Zl6wTfCAa1c4XBa48zh+pvCIuj55Dzu3It+iwaM=
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:subject:to:references:from:message-id:date
	:mime-version:in-reply-to:content-type; s=default; bh=SAfcW8xFij
	wUZU7KpcW4T7YTb8c=; b=OCGfW0eOCyD9zB3AdpDACEzJ2UDLTpjx0/uGRf7fMa
	N76B9qeE8ChkUwLPApAD84ObiuS10VpNdsxWEQMN4m/tCmtcwNnseY2dXmYC2xxL
	mHQ70z7tVjqujzN582M12MH+Sx1JC4cZlfDcKSGYfNejxJQpb2+rqc/T7zOgRlr7
	4=
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id: <cygwin.cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=0.1 required=5.0 tests=BAYES_50,KAM_LAZY_DOMAIN_SECURITY,RCVD_IN_DNSWL_NONE,RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=sk:base-fi, sk:basefi, postinstall, msg00617.html
X-HELO: mailout09.t-online.de
Subject: Re: [ANNOUNCEMENT] Updated/Test: base-files-4.3-1
To: cygwin AT cygwin DOT com
References: <announce DOT 878tv914ya DOT fsf AT Rainer DOT invalid>
From: Christian Franke <Christian DOT Franke AT t-online DOT de>
Message-ID: <57CC0128.3080900@t-online.de>
Date: Sun, 4 Sep 2016 13:10:32 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:43.0) Gecko/20100101 Firefox/43.0 SeaMonkey/2.40
MIME-Version: 1.0
In-Reply-To: <announce.878tv914ya.fsf@Rainer.invalid>
Content-Type: multipart/mixed; boundary="------------030802020006030802090704"
X-IsSubscribed: yes

--------------030802020006030802090704
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

Achim Gratz wrote:
> 4.3-1 (feature release)
>      * Cygwin.bat: Add to base-files.  This version is independent of
>        the actual install path, see:
>        cygwin.com/ml/cygwin/2016-08/msg00617.html

Thanks for accepting the patch.

Unfortunately I missed a detail: The postinstall script does not 
preserve the x-permission of Cygwin.bat. This is required if run from 
explorer or with 'cygstart /Cygwin.bat'.

Possible fix attached. Alternatively remove the 'touch' command and the 
x-permission from etc/defaults/etc/profile.

Thanks,
Christian


--------------030802020006030802090704
Content-Type: text/x-patch;
 name="base-files-4.3-1-x-perm.patch"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
 filename="base-files-4.3-1-x-perm.patch"

--- base-files-profile.sh.orig	2016-09-03 13:37:59.000000000 +0200
+++ base-files-profile.sh	2016-09-04 12:42:40.805222900 +0200
@@ -18,7 +18,7 @@
   if [ ! -e ${fDest} -a ! -L ${fDest} ]; then
     echo "Using the default version of ${fDest} (${fSrc})"
     /usr/bin/mkdir -p $(dirname ${fDest})
-    /usr/bin/touch ${fDest}
+    test "${fDest}" = "/Cygwin.bat" || /usr/bin/touch ${fDest}
     /usr/bin/cp ${fSrc} ${fDest}
   else
     echo "${fDest} is already in existence, not overwriting."


--------------030802020006030802090704
Content-Type: text/plain; charset=us-ascii

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
--------------030802020006030802090704--