X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C7E5C5.BC83AF95@onstor-exch02.onstor.net>; Thu, 23 Aug 2007 12:39:43 -0800
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----_=_NextPart_001_01C7E5C5.BC83AF95"
References: <WEBMAILWATSYBGbJ4Yu00000a19@mail.onstor.com><BB375AF679D4A34E9CA8DFA650E2B04E02F3D6A8@onstor-exch02.onstor.net><20070823130528.4e418821@ripper.onstor.net><BB375AF679D4A34E9CA8DFA650E2B04E02F3D6B2@onstor-exch02.onstor.net> <20070823132626.1b9b3d92@ripper.onstor.net>
Content-class: urn:content-classes:message
Subject: RE: PERFORCE change 25147 for review
Date: Thu, 23 Aug 2007 12:36:05 -0800
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E02F3D6B3@onstor-exch02.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: PERFORCE change 25147 for review
Thread-Index: Acflw+FAAA1D1uO7TFaQgKZSzuycCwAAVlbY
From: "Ken Renshaw" <ken.renshaw@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

This is a multi-part message in MIME format.

------_=_NextPart_001_01C7E5C5.BC83AF95
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable

Gotcha, and I agree.

I fixed it locally in my workspace so I could continue with the =
submittal, but I'm glad to hear that Max is actually going to fix it. I =
was going to bug him shortly about it, but I agree with you that this is =
not how it's done, and I should push right back on any breakages to get =
the developer to fix it.

Thanks,

-Ken

-----Original Message-----
From: Andy Sharp
Sent: Thu 8/23/2007 1:26 PM
To: Ken Renshaw
Subject: Re: PERFORCE change 25147 for review
=20
On Thu, 23 Aug 2007 13:06:32 -0700 "Ken Renshaw"
<ken.renshaw@onstor.com> wrote:

> Nevermind. Too many breakages the last couple days ;)
>=20
> I was talking about the problem with authen-lib.c breaking the cg
> build for Jeff. I just checked in the fix for that, so we're back to
> having no compile issues in any of the eight variants. For the moment
> anyway B)

BSD doesn't currently build.  Max is working on it.  Don't let the
build of BSD slack anymore than the build of anything else.  The pace
of change is simply too great.  Max said he sent you some information
on how to fix it.  This is not how it's done, so please gently [not
like me] educate developers when they try to get away with this.  They
have to fix their own breakage, even if they have to seek out someone
like me or Larry or, ironically, Max to assist them.

Thanks,

a

> Thanks,
>=20
> -Ken
>=20
>=20
> -----Original Message-----
> From: Andy Sharp
> Sent: Thu 8/23/2007 1:05 PM
> To: Ken Renshaw
> Subject: Re: PERFORCE change 25147 for review
> =20
> Sorry, I don't actually know what you are talking about.  Was there
> some huge problem yesterday?  This was to fix a problem I created with
> a checkin a few minutes before this one.
>=20
> On Thu, 23 Aug 2007 12:04:22 -0700 "Ken Renshaw"
> <ken.renshaw@onstor.com> wrote:
>=20
> > Is that the magic fix for yesterday's issue Andy? I was hoping Tim's
> > fix addressed it but alas, no.=20
> >=20
> > Thanks much if so.
> >=20
> >=20
> > -----Original Message-----
> > From: Andy Sharp
> > Sent: Thu 8/23/2007 11:54 AM
> > To: Amit Bothra; Andy Sharp; Brian DeForest; Charissa Willard; Chris
> > Vandever; Deepak Veliath; Ian Brown; Jobi Ariyamannil; Jonathan
> > Goldick; Ken Renshaw; Larry Scheer; Maxim Kozlovsky; Sandrine
> > Boulanger; Tim Gardner Subject: PERFORCE change 25147 for review
> > Change 25147 by andys@ripper on 2007/08/23 11:49:47
> >=20
> > 	Fix build breakage: move PROD_PLATFORM variable setting
> > 	code outside it's original limited scope.
> > =09
> > 	Reviewed by jeffm
> >=20
> > Affected files ...
> >=20
> > ... //depot/dev/nfx-tree/Tools/defs.mk#6 edit
> >=20
> >=20
> > http://liszt:1818/@md=3Dd&cd=3D//depot/$c=3DG35@/25147?ac=3D10
> >=20
>=20


------_=_NextPart_001_01C7E5C5.BC83AF95
Content-Type: text/html;
	charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV=3D"Content-Type" CONTENT=3D"text/html; =
charset=3Diso-8859-1">
<META NAME=3D"Generator" CONTENT=3D"MS Exchange Server version =
6.5.7652.24">
<TITLE>RE: PERFORCE change 25147 for review</TITLE>
</HEAD>
<BODY>
<!-- Converted from text/plain format -->

<P><FONT SIZE=3D2>Gotcha, and I agree.<BR>
<BR>
I fixed it locally in my workspace so I could continue with the =
submittal, but I'm glad to hear that Max is actually going to fix it. I =
was going to bug him shortly about it, but I agree with you that this is =
not how it's done, and I should push right back on any breakages to get =
the developer to fix it.<BR>
<BR>
Thanks,<BR>
<BR>
-Ken<BR>
<BR>
-----Original Message-----<BR>
From: Andy Sharp<BR>
Sent: Thu 8/23/2007 1:26 PM<BR>
To: Ken Renshaw<BR>
Subject: Re: PERFORCE change 25147 for review<BR>
<BR>
On Thu, 23 Aug 2007 13:06:32 -0700 &quot;Ken Renshaw&quot;<BR>
&lt;ken.renshaw@onstor.com&gt; wrote:<BR>
<BR>
&gt; Nevermind. Too many breakages the last couple days ;)<BR>
&gt;<BR>
&gt; I was talking about the problem with authen-lib.c breaking the =
cg<BR>
&gt; build for Jeff. I just checked in the fix for that, so we're back =
to<BR>
&gt; having no compile issues in any of the eight variants. For the =
moment<BR>
&gt; anyway B)<BR>
<BR>
BSD doesn't currently build.&nbsp; Max is working on it.&nbsp; Don't let =
the<BR>
build of BSD slack anymore than the build of anything else.&nbsp; The =
pace<BR>
of change is simply too great.&nbsp; Max said he sent you some =
information<BR>
on how to fix it.&nbsp; This is not how it's done, so please gently =
[not<BR>
like me] educate developers when they try to get away with this.&nbsp; =
They<BR>
have to fix their own breakage, even if they have to seek out =
someone<BR>
like me or Larry or, ironically, Max to assist them.<BR>
<BR>
Thanks,<BR>
<BR>
a<BR>
<BR>
&gt; Thanks,<BR>
&gt;<BR>
&gt; -Ken<BR>
&gt;<BR>
&gt;<BR>
&gt; -----Original Message-----<BR>
&gt; From: Andy Sharp<BR>
&gt; Sent: Thu 8/23/2007 1:05 PM<BR>
&gt; To: Ken Renshaw<BR>
&gt; Subject: Re: PERFORCE change 25147 for review<BR>
&gt;&nbsp;<BR>
&gt; Sorry, I don't actually know what you are talking about.&nbsp; Was =
there<BR>
&gt; some huge problem yesterday?&nbsp; This was to fix a problem I =
created with<BR>
&gt; a checkin a few minutes before this one.<BR>
&gt;<BR>
&gt; On Thu, 23 Aug 2007 12:04:22 -0700 &quot;Ken Renshaw&quot;<BR>
&gt; &lt;ken.renshaw@onstor.com&gt; wrote:<BR>
&gt;<BR>
&gt; &gt; Is that the magic fix for yesterday's issue Andy? I was hoping =
Tim's<BR>
&gt; &gt; fix addressed it but alas, no.<BR>
&gt; &gt;<BR>
&gt; &gt; Thanks much if so.<BR>
&gt; &gt;<BR>
&gt; &gt;<BR>
&gt; &gt; -----Original Message-----<BR>
&gt; &gt; From: Andy Sharp<BR>
&gt; &gt; Sent: Thu 8/23/2007 11:54 AM<BR>
&gt; &gt; To: Amit Bothra; Andy Sharp; Brian DeForest; Charissa Willard; =
Chris<BR>
&gt; &gt; Vandever; Deepak Veliath; Ian Brown; Jobi Ariyamannil; =
Jonathan<BR>
&gt; &gt; Goldick; Ken Renshaw; Larry Scheer; Maxim Kozlovsky; =
Sandrine<BR>
&gt; &gt; Boulanger; Tim Gardner Subject: PERFORCE change 25147 for =
review<BR>
&gt; &gt; Change 25147 by andys@ripper on 2007/08/23 11:49:47<BR>
&gt; &gt;<BR>
&gt; &gt; &nbsp;&nbsp;&nbsp; Fix build breakage: move PROD_PLATFORM =
variable setting<BR>
&gt; &gt; &nbsp;&nbsp;&nbsp; code outside it's original limited =
scope.<BR>
&gt; &gt; &nbsp;&nbsp;&nbsp;<BR>
&gt; &gt; &nbsp;&nbsp;&nbsp; Reviewed by jeffm<BR>
&gt; &gt;<BR>
&gt; &gt; Affected files ...<BR>
&gt; &gt;<BR>
&gt; &gt; ... //depot/dev/nfx-tree/Tools/defs.mk#6 edit<BR>
&gt; &gt;<BR>
&gt; &gt;<BR>
&gt; &gt; <A =
HREF=3D"http://liszt:1818/@md=3Dd&cd=3D//depot/$c=3DG35@/25147?ac=3D10">h=
ttp://liszt:1818/@md=3Dd&cd=3D//depot/$c=3DG35@/25147?ac=3D10</A><BR>
&gt; &gt;<BR>
&gt;<BR>
<BR>
</FONT>
</P>

</BODY>
</HTML>
------_=_NextPart_001_01C7E5C5.BC83AF95--
