This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


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

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

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

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]