X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C78150.1A8EAADF@onstor-exch02.onstor.net>; Tue, 17 Apr 2007 17:25:44 -0700
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----_=_NextPart_001_01C78150.1A8EAADF"
References: <WEBMAILzvP8miPpQBZN00000481@mail.onstor.com><BB375AF679D4A34E9CA8DFA650E2B04E02F3CEDF@onstor-exch02.onstor.net> <20070417171421.468e2ca2@ripper.onstor.net>
Content-class: urn:content-classes:message
Subject: RE: PERFORCE change 23606 for review
Date: Tue, 17 Apr 2007 17:25:24 -0700
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E02F3CEE4@onstor-exch02.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: PERFORCE change 23606 for review
Thread-Index: AceBToNWnSXCh3XxTIKAdUK7ibrBvgAAYsoA
From: "Ken Renshaw" <ken.renshaw@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>
Cc: "Larry Scheer" <larry.scheer@onstor.com>

This is a multi-part message in MIME format.

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

Yep, saw that. Sorry for the 'duh'.

________________________________

From: Andy Sharp
Sent: Tue 4/17/2007 5:14 PM
To: Ken Renshaw
Cc: Larry Scheer
Subject: Re: PERFORCE change 23606 for review



I included that in my subsequent checkin 23607

a

On Tue, 17 Apr 2007 17:08:02 -0700 "Ken Renshaw"
<ken.renshaw@onstor.com> wrote:

> Thanks, it was only a one-line change I was trying to do, not sure
> why p4 munged it ( or I bungled it ).
> Anyway, the last line of your changelist reverted the single line I
> was trying to add, this changelist:
> <diff of Andy's checkin to backout the merge>
> 278c262
> < (cd /var/run && { rm -rf -- !(pm.core*); install -c -m 664 -g
> utmp /dev/null utmp; }) ---
> > (cd /var/run && { rm -rf -- *; install -c -m 664 -g utmp /dev/null
> > utmp; })
>=20
>=20
>
> [build@k2 usr]$ p4 describe 23394
> Change 23394 by danqingj@danqingj-r133work on 2007/04/03 13:05:07
>         For CQ16618, do not delete pm.core* files upon reboot.
>         Reviewed by: edk
> Affected files ...
> ... //depot/R1_3_3_work/openbsd/src/etc/rc#3 edit
> Differences ...
> =3D=3D=3D=3D //depot/R1_3_3_work/openbsd/src/etc/rc#3 (text) =
=3D=3D=3D=3D
> 278c278
> < (cd /var/run && { rm -rf -- *; install -c -m 664 -g utmp /dev/null
> utmp; }) ---
> > (cd /var/run && { rm -rf -- !(pm.core*); install -c -m 664 -g
> > utmp /dev/null utmp; })
>
>=20
> I'm going to go ahead and put the one-liner back in since we have to
> chip with it...again, sorry to have pulled the wrong change/file in.
> -Ken
>=20
>=20
>
> ________________________________
>
> From: Andy Sharp
> Sent: Tue 4/17/2007 2:53 PM
> To: Andy Sharp; Brian DeForest; Charissa Willard; Chris Vandever; Ian
> Brown; Jobi Ariyamannil; Jonathan Goldick; Ken Renshaw; Larry Scheer;
> Maxim Kozlovsky; Nagendra Subramanya; Sandrine Boulanger; Tim Gardner
> Subject: PERFORCE change 23606 for review
>
>
>
> Change 23606 by andys@linux-compile-andys on 2007/04/17 13:48:52
>
>         Reverting broken merge/integrate from 1.3.3.
>         Reviewed by larrys
>
> Affected files ...
>
> ... //depot/FB-DELOREAN/openbsd/src/etc/rc#4 edit
>
>
>



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

<HTML dir=3Dltr><HEAD><TITLE>Re: PERFORCE change 23606 for review</TITLE>=0A=
<META http-equiv=3DContent-Type content=3D"text/html; charset=3Dunicode">=0A=
<META content=3D"MSHTML 6.00.2800.1106" name=3DGENERATOR></HEAD>=0A=
<BODY>=0A=
<DIV id=3DidOWAReplyText89664 dir=3Dltr>=0A=
<DIV dir=3Dltr><FONT face=3DArial color=3D#000000 size=3D2>Yep, saw =
that. Sorry for the 'duh'.</FONT></DIV></DIV>=0A=
<DIV dir=3Dltr><BR>=0A=
<HR tabIndex=3D-1>=0A=
<FONT face=3DTahoma size=3D2><B>From:</B> Andy Sharp<BR><B>Sent:</B> Tue =
4/17/2007 5:14 PM<BR><B>To:</B> Ken Renshaw<BR><B>Cc:</B> Larry =
Scheer<BR><B>Subject:</B> Re: PERFORCE change 23606 for =
review<BR></FONT><BR></DIV>=0A=
<DIV>=0A=
<P><FONT size=3D2>I included that in my subsequent checkin =
23607<BR><BR>a<BR><BR>On Tue, 17 Apr 2007 17:08:02 -0700 "Ken =
Renshaw"<BR>&lt;ken.renshaw@onstor.com&gt; wrote:<BR><BR>&gt; Thanks, it =
was only a one-line change I was trying to do, not sure<BR>&gt; why p4 =
munged it ( or I bungled it ).<BR>&gt; Anyway, the last line of your =
changelist reverted the single line I<BR>&gt; was trying to add, this =
changelist:<BR>&gt; &lt;diff of Andy's checkin to backout the =
merge&gt;<BR>&gt; 278c262<BR>&gt; &lt; (cd /var/run &amp;&amp; { rm -rf =
-- !(pm.core*); install -c -m 664 -g<BR>&gt; utmp /dev/null utmp; }) =
---<BR>&gt; &gt; (cd /var/run &amp;&amp; { rm -rf -- *; install -c -m =
664 -g utmp /dev/null<BR>&gt; &gt; utmp; =
})<BR>&gt;&nbsp;<BR>&gt;&nbsp;<BR>&gt;<BR>&gt; [build@k2 usr]$ p4 =
describe 23394<BR>&gt; Change 23394 by danqingj@danqingj-r133work on =
2007/04/03 =
13:05:07<BR>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; For =
CQ16618, do not delete pm.core* files upon =
reboot.<BR>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Reviewed =
by: edk<BR>&gt; Affected files ...<BR>&gt; ... =
//depot/R1_3_3_work/openbsd/src/etc/rc#3 edit<BR>&gt; Differences =
...<BR>&gt; =3D=3D=3D=3D //depot/R1_3_3_work/openbsd/src/etc/rc#3 (text) =
=3D=3D=3D=3D<BR>&gt; 278c278<BR>&gt; &lt; (cd /var/run &amp;&amp; { rm =
-rf -- *; install -c -m 664 -g utmp /dev/null<BR>&gt; utmp; }) =
---<BR>&gt; &gt; (cd /var/run &amp;&amp; { rm -rf -- !(pm.core*); =
install -c -m 664 -g<BR>&gt; &gt; utmp /dev/null utmp; =
})<BR>&gt;<BR>&gt;&nbsp;<BR>&gt; I'm going to go ahead and put the =
one-liner back in since we have to<BR>&gt; chip with it...again, sorry =
to have pulled the wrong change/file in.<BR>&gt; =
-Ken<BR>&gt;&nbsp;<BR>&gt;&nbsp;<BR>&gt;<BR>&gt; =
________________________________<BR>&gt;<BR>&gt; From: Andy =
Sharp<BR>&gt; Sent: Tue 4/17/2007 2:53 PM<BR>&gt; To: Andy Sharp; Brian =
DeForest; Charissa Willard; Chris Vandever; Ian<BR>&gt; Brown; Jobi =
Ariyamannil; Jonathan Goldick; Ken Renshaw; Larry Scheer;<BR>&gt; Maxim =
Kozlovsky; Nagendra Subramanya; Sandrine Boulanger; Tim Gardner<BR>&gt; =
Subject: PERFORCE change 23606 for =
review<BR>&gt;<BR>&gt;<BR>&gt;<BR>&gt; Change 23606 by =
andys@linux-compile-andys on 2007/04/17 =
13:48:52<BR>&gt;<BR>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; =
Reverting broken merge/integrate from =
1.3.3.<BR>&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Reviewed =
by larrys<BR>&gt;<BR>&gt; Affected files ...<BR>&gt;<BR>&gt; ... =
//depot/FB-DELOREAN/openbsd/src/etc/rc#4 =
edit<BR>&gt;<BR>&gt;<BR>&gt;<BR></FONT></P></DIV></BODY></HTML>
------_=_NextPart_001_01C78150.1A8EAADF--
