X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C7F57E.BDDA721A@onstor-exch02.onstor.net>; Wed, 12 Sep 2007 12:51:50 -0800
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Content-class: urn:content-classes:message
Subject: RE: PERFORCE change 25364 for review
Date: Wed, 12 Sep 2007 12:51:49 -0800
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E0579D5A7@onstor-exch02.onstor.net>
In-Reply-To: <20070912132159.73b8d32a@ripper.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: PERFORCE change 25364 for review
Thread-Index: Acf1epOV+UN44NGnQq6cqdf+TzLiYAABBTjw
From: "Ken Renshaw" <ken.renshaw@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

Agreed, I'll make the change as soon as I can tread water again.

Thanks Andy.

-Ken

-----Original Message-----
From: Andy Sharp=20
Sent: Wednesday, September 12, 2007 1:22 PM
To: Ken Renshaw
Subject: Re: PERFORCE change 25364 for review

I think it's safe to reject a checkin with either no or blank comment,
don't you?  I can't think of any reason why we would want to allow
that.  Unless some of the more essoteric branches allow that for some
reason.  Anyway it's got to be helpful too because it's almost certainly
a mistake, as I assume this one was. I'd be pretty thankful if I
accidentally did a submit with just a blank line for a comment and it
was rejected so I could get a second chance.

Cheers,

a

 On Wed, 12 Sep 2007 12:57:17 -0700 "Ken Renshaw"
<ken.renshaw@onstor.com> wrote:

> Yeah, I saw that too...he slipped it through since he noticed we
> filter on filetypes ( .c, .h., etc. etc. ).=20
>=20
> Would you like to see any and all checkins stopped at the gate then
> Andy?
>=20
>=20
> Thanks,
>=20
> -Ken
>=20
> -----Original Message-----
> From: Andy Sharp=20
> Sent: Wednesday, September 12, 2007 12:50 PM
> To: Maxim Kozlovsky
> Cc: Ken Renshaw
> Subject: Re: PERFORCE change 25364 for review
>=20
> Dude.
>=20
> [Ken, can we add a commit trigger that doesn't accept checkins with
> no (or only whitespace) comment?]
>=20
> a
>=20
> On 12 Sep 2007 10:59:47 -0700 Maxim Kozlovsky
> <maxim.kozlovsky@onstor.com> wrote:
>=20
> > Change 25364 by maximk@maximk-5 on 2007/09/12 10:54:47
> >=20
> >=20
> > Affected files ...
> >=20
> > ... //depot/dev/linux/kernel/linux-mips-2.6/bobcat-config#4 edit
> >=20
> >=20
> > http://liszt:1818/@md=3Dd&cd=3D//depot/$c=3DG35@/25364?ac=3D10
