DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:from:to:subject:date:message-id:mime-version
	:content-type; q=dns; s=default; b=sqZxHrpuSB6pqfxxzsn04STJPIlDR
	EOXxXhXWavWcGw6SKOY/Aj2bRlhZe/+eij4EeHAk02Jk/laD2pZ3BpG570Nr90yh
	QiNSUOJv/N/+eJ72AwytOTbega9h82PVYnLMk+RyNnReWt2hXXupECCSJZ+rOiIh
	yttuC4oD9qEg3Q=
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
From: "Jon Beniston" <jon@beniston.com>
To: <cygwin@cygwin.com>
Subject: setup-x86.exe exits before installation is complete
Date: Thu, 6 Mar 2014 12:18:19 -0000
Message-ID: <002301cf3936$2a25c7d0$7e715770$@beniston.com>
MIME-Version: 1.0
Content-Type: multipart/mixed;	boundary="----=_NextPart_000_0024_01CF3936.2A2615F0"

------=_NextPart_000_0024_01CF3936.2A2615F0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: 7bit

Hi,

It seems recent versions of setup-x86.exe exit before the installation is
complete. Without the --no-admin option, the main installation is run as a
child process, and the parent doesn't wait for the child to complete. This
can cause a bit of a problem for other installers that install Cygwin (as
when setup.exe exits, Cygwin isn't installed).

Is something like the attached patch appropriate? It just waits for the
child process before exiting.

Cheers,
Jon


------=_NextPart_000_0024_01CF3936.2A2615F0
Content-Type: text/plain;
	name="patch.txt"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: attachment;
	filename="patch.txt"

Index: main.cc
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
RCS file: /cvs/cygwin-apps/setup/main.cc,v
retrieving revision 2.75
diff -u -r2.75 main.cc
--- main.cc	18 Nov 2013 11:16:14 -0000	2.75
+++ main.cc	6 Mar 2014 12:02:16 -0000
@@ -295,6 +295,7 @@
 		sei.lpVerb =3D "runas";
 		sei.lpFile =3D exe_path;
 		sei.nShow =3D SW_NORMAL;
+		sei.fMask |=3D SEE_MASK_NOCLOSEPROCESS;
=20
 		// Avoid another isRunAsAdmin check in the child.
 		std::string command_line_cs (command_line);
@@ -322,6 +323,11 @@
 		  }
 		else
 		  exit_msg =3D IDS_ELEVATED;
+
+        // Wait until child process is finished
+		if (sei.hProcess !=3D NULL)
+		  WaitForSingleObject (sei.hProcess, INFINITE);
+=20=20=20=20=20=20=20=20=20=20=20=20
 		// Once we are set on course to privilege elevate, the parent
 		// process is unnecessary.
 		goto finish_up;


------=_NextPart_000_0024_01CF3936.2A2615F0
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
------=_NextPart_000_0024_01CF3936.2A2615F0--