X-Recipient: archive-cygwin AT delorie DOT com X-Spam-Check-By: sourceware.org Message-ID: <475C4E34.ADDC1217@dessent.net> Date: Sun, 09 Dec 2007 12:21:08 -0800 From: Brian Dessent X-Mailer: Mozilla 4.79 [en] (Windows NT 5.0; U) MIME-Version: 1.0 To: cygwin AT cygwin DOT com Subject: Re: cron_diagnose step in cron-config w/new mounts 1.5.25(0.156/4/2) References: <200712091554 DOT lB9Fs42H004614 AT tigris DOT pounder DOT sol DOT net> Content-Type: multipart/mixed; boundary="------------2D08AF037A4744F9AC4C07E0" X-IsSubscribed: yes Reply-To: cygwin AT cygwin DOT com Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner AT cygwin DOT com Mail-Followup-To: cygwin AT cygwin DOT com Delivered-To: mailing list cygwin AT cygwin DOT com --------------2D08AF037A4744F9AC4C07E0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Tom Rodman wrote: > It's possible the cron-config script does not understand the new > mount entries in the registry; or what am I missing? Yes, the script needs updating. However, it should *not* be looking at the registry directly. It should be using the mount command which is the only supported interface for the mount table, like in the attached patch. Brian --------------2D08AF037A4744F9AC4C07E0 Content-Type: text/plain; charset=us-ascii; name="cron-config-registry.patch" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="cron-config-registry.patch" --- cron-config.orig 2007-06-12 17:56:23.001000000 -0700 +++ cron-config 2007-12-09 12:19:11.953125000 -0800 @@ -343,8 +343,7 @@ mnt_point=$1 dos_dir=$2 - SYSTEM_MOUNTS='/proc/registry/HKey_Local_Machine/Software/Cygnus Solutions/Cygwin/mounts v2' - if ls "$SYSTEM_MOUNTS" | grep -Eq "^${mnt_point}$"; then + if mount | grep -Eq "on ${mnt_point} type.*system"; then true else echo; @@ -709,8 +708,7 @@ servtest=no fi elif [ -n "$nt" -a "${servtest}" = "0" ]; then - cronuserreg="/proc/registry/HKEY_LOCAL_MACHINE/SYSTEM/CurrentControlSet/Services/cron/ObjectName" - cronusername="$(cat "${cronuserreg}")" + cronusername=$(cygrunsrv -VQ cron | awk '/^Account/ { print $3 }') echo "Cron is already installed as a service under account ${cronusername}." if request "Do you want to remove or reinstall it?"; then if cygrunsrv -R cron; then --------------2D08AF037A4744F9AC4C07E0 Content-Type: text/plain; charset=us-ascii -- Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple Problem reports: http://cygwin.com/problems.html Documentation: http://cygwin.com/docs.html FAQ: http://cygwin.com/faq/ --------------2D08AF037A4744F9AC4C07E0--