Re: [ANNOUNCEMENT] Updated/Test: base-files-4.3-1

[Christian Franke <Christian.Franke@t-online.de>]

[-- Attachment #1: Type: text/plain, Size: 558 bytes --]

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


[-- Attachment #2: base-files-4.3-1-x-perm.patch --]
[-- Type: text/x-patch, Size: 496 bytes --]

--- 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."


[-- Attachment #3: Type: text/plain, Size: 218 bytes --]

--
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