Mailing-List: contact cygwin-developers-help AT sourceware DOT cygnus DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-developers-subscribe AT sources DOT redhat DOT com>
List-Archive: <http://sources.redhat.com/ml/cygwin-developers/>
List-Post: <mailto:cygwin-developers AT sources DOT redhat DOT com>
List-Help: <mailto:cygwin-developers-help AT sources DOT redhat DOT com>, <http://sources.redhat.com/ml/#faqs>
Sender: cygwin-developers-owner AT sources DOT redhat DOT com
Delivered-To: mailing list cygwin-developers AT sources DOT redhat DOT com
Message-ID: <039201c1010b$856f0c80$806410ac@local>
From: "Robert Collins" <robert DOT collins AT itdomain DOT com DOT au>
To: <cygwin-developers AT cygwin DOT com>, <cygwin-patches AT cygwin DOT com>
References: <3B3324A7 DOT 49FFC98A AT yahoo DOT com> <054c01c0fbef$5f600e20$0200a8c0 AT lifelesswks> <06a001c0fc51$7a87e210$0200a8c0 AT lifelesswks> <20010629114004 DOT A6990 AT redhat DOT com> <VA DOT 00000842 DOT 01fd0b44 AT thesoftwaresource DOT com> <20010629172912 DOT A8991 AT redhat DOT com> <032001c100fe$d62310c0$806410ac AT local> <20010629205735 DOT K9607 AT redhat DOT com> <034701c10106$34f6b6e0$806410ac AT local> <036501c10108$b55383c0$806410ac AT local> <20010629221309 DOT A11334 AT redhat DOT com> <038301c1010a$bab7e840$806410ac AT local>
Subject: Re: hierarchy in setup (category stuff)
Date: Sat, 30 Jun 2001 12:22:32 +1000
MIME-Version: 1.0
Content-Type: multipart/mixed;
	boundary="----=_NextPart_000_038F_01C1015F.56488A40"
X-Priority: 3
X-MSMail-Priority: Normal
X-Mailer: Microsoft Outlook Express 5.50.4522.1200
X-MimeOLE: Produced By Microsoft MimeOLE V5.50.4522.1200
X-OriginalArrivalTime: 30 Jun 2001 02:10:09.0608 (UTC) FILETIME=[C9DC5C80:01C10109]

This is a multi-part message in MIME format.

------=_NextPart_000_038F_01C1015F.56488A40
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: 7bit

Applying this should fix my boo boo.

Rob
----- Original Message -----
From: "Robert Collins" <robert DOT collins AT itdomain DOT com DOT au>
To: <cygwin-developers AT cygwin DOT com>; <cygwin-patches AT cygwin DOT com>
Sent: Saturday, June 30, 2001 12:16 PM
Subject: Re: hierarchy in setup (category stuff)


>
> ----- Original Message -----
> From: "Christopher Faylor" <cgf AT redhat DOT com>
> To: <cygwin-developers AT cygwin DOT com>; <cygwin-patches AT cygwin DOT com>
> Sent: Saturday, June 30, 2001 12:13 PM
> Subject: Re: hierarchy in setup (category stuff)
>
>
> > On Sat, Jun 30, 2001 at 12:02:24PM +1000, Robert Collins wrote:
> > >Michael,
> > >    I think I trashed some of your src related patches, CVS didn't
report
> > >any errors merging, but I just have this nasty suspicion.
> >
> > You did.  All of my changes are gone, too.
> >
> > I'd like to revert CVS to pre-your-change and try again.  Is that ok?
>
> It's possibly easier just to grab the diffs that you committed before and
> apply those again. (ie cvs rdiff -r 2.33 -r 2.35 choose.cc )
>
> >
> > cgf
> >
>
>

------=_NextPart_000_038F_01C1015F.56488A40
Content-Type: application/octet-stream;
	name="oops.patch"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: attachment;
	filename="oops.patch"

Index: choose.cc=0A=
=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=0A=
RCS file: /cvs/src/src/winsup/cinstall/choose.cc,v=0A=
retrieving revision 2.36=0A=
diff -u -p -r2.36 choose.cc=0A=
--- choose.cc	2001/06/30 01:37:55	2.36=0A=
+++ choose.cc	2001/06/30 02:19:23=0A=
@@ -100,9 +100,11 @@ isinstalled (Package *pkg, int trust)=0A=
 static void=0A=
 set_action (Package *pkg, bool preinc)=0A=
 {=0A=
-  pkg->srcpicked =3D 0;=0A=
   if (!pkg->action || preinc)=0A=
-    ((int) pkg->action)++;=0A=
+    {=0A=
+      ((int) pkg->action)++;=0A=
+      pkg->srcpicked =3D 0;=0A=
+    }=0A=
 =0A=
   /* Exercise the action state machine. */=0A=
   for (;; ((int) pkg->action)++)=0A=
@@ -145,21 +147,28 @@ set_action (Package *pkg, bool preinc)=0A=
       case ACTION_UNINSTALL:=0A=
 	if (pkg->installed)=0A=
 	  return;=0A=
+	break;=0A=
       case ACTION_REDO:=0A=
-	if (pkg->installed)=0A=
+	if (pkg->installed && pkg->info[pkg->installed_ix].install_exists)=0A=
 	  {=0A=
 	    pkg->trust =3D pkg->installed_ix;=0A=
 	    return;=0A=
 	  }=0A=
+	break;=0A=
       case ACTION_SRC_ONLY:=0A=
-	if (pkg->installed && pkg->installed->source_exists)=0A=
-	  return;=0A=
+	if (pkg->info[pkg->trust].source_exists)=0A=
+	  {=0A=
+	    pkg->srcpicked =3D 1;=0A=
+	    return;=0A=
+	  }=0A=
 	break;=0A=
       case ACTION_SAME_LAST:=0A=
 	pkg->action =3D ACTION_SKIP;=0A=
 	/* Fall through intentionally */=0A=
       case ACTION_SKIP:=0A=
-	return;=0A=
+        if (!pkg->installed || =
!pkg->info[pkg->installed_ix].install_exists)=0A=
+	  return;=0A=
+	break;=0A=
       default:=0A=
 	log (0, "should never get here %d\n", pkg->action);=0A=
       }=0A=

------=_NextPart_000_038F_01C1015F.56488A40--