X-Sylpheed-Account-Id:1
S:andy.sharp@onstor.com
SCF:#mh/Mailbox/sent
X-Sylpheed-Sign:0
X-Sylpheed-Encrypt:0
X-Sylpheed-Privacy-System:
RMID:#imap/andys@onstor.net@exch1.onstor.net/INBOX	72547	2779531E7C760D4491C96305019FEEB52AD1CA881B@exch1.onstor.net
X-Sylpheed-End-Special-Headers: 1
Date: Wed, 24 Jun 2009 15:50:45 -0700
From: Andrew Sharp <andy.sharp@onstor.com>
To: Maxim Kozlovsky <maxim.kozlovsky@onstor.com>
Bcc: Brian Stark <brian.stark@onstor.com>
Subject: Re: PERFORCE change 32698 for review
Message-ID: <20090624155045.70299a58@ripper.onstor.net>
References: <WEBMAILz4rlDQcpCqJf00093e1a@mail.onstor.com>
 <20090623104643.62c66597@ripper.onstor.net>
 <2779531E7C760D4491C96305019FEEB52AD1CA8780@exch1.onstor.net>
 <20090623132607.3640d6e2@ripper.onstor.net>
 <2779531E7C760D4491C96305019FEEB52AD1CA881B@exch1.onstor.net>
Organization: Onstor
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

That's only a small part of the criteria of the convention, carefully
chosen it would seem.  The convention also includes disrupting
development work by other developers, pointlessly and needlessly
overtaxing our IT resources, and probably a few other things.  This
massive directory doesn't yet belong in the dev branch.  Have you
disregarded the effect on development branches that periodically do
integrates from the dev branch?  Now they have to do a fiddly
hand-crafted integrate, as well as everyone having to modify their
clients.

Basically this needlessly makes everyone's life more difficult, for no
measureable gain.

On Tue, 23 Jun 2009 15:03:30 -0700 Maxim Kozlovsky
<maxim.kozlovsky@onstor.com> wrote:

> The convention is to make a branch for something that destabilizes
> the code for significant periods of time. Since in no way check-in of
> a baseline for ZFS work, or any subsequent work on ZFS for that
> matter, can make the code unstable, it should go on the dev branch.
> The GNS changes are going to the dev branch as well unless it happens
> that they are violating the criteria of keeping the code stable.
> 
> 
> >-----Original Message-----
> >From: Andy Sharp
> >Sent: Tuesday, June 23, 2009 1:26 PM
> >To: Maxim Kozlovsky
> >Cc: Larry Scheer
> >Subject: Re: PERFORCE change 32698 for review
> >
> >I agree these files belong in perforce as Max has stated, but
> >belonging in perforce is one thing, belonging in the dev branch is
> >quite another. Our convention, and I believe it's a good one, is to
> >do small-number-of-developers projects in a separate branch, like
> >gns and tuxrx and cougar and many others, until they become part of
> >the actual product.
> >
> >The projected delivery date for this work is way out in the future.
> >I don't think everyone should have to hack the snot out of their
> >client spec just to deal with this.  Normally I think the burden for
> >doing extra work should be on the projects' developers, not on the
> >rest of the staff. Additionally, it's a drain on company wide
> >resources and we need to be mindful of the effect we have on those
> >when this ostensibly could be checked out into many, many clients.
> >
> >Thanks,
> >
> >a
> >
> >
> > On Tue, 23 Jun 2009 10:54:17 -0700 Maxim Kozlovsky
> ><maxim.kozlovsky@onstor.com> wrote:
> >
> >> They do belong, we are doing the zfs port and unfortunately zfs is
> >> part of open solaris. If you don't want them in your development
> >> space, remove them from your client view.
> >>
> >> >-----Original Message-----
> >> >From: Andy Sharp
> >> >Sent: Tuesday, June 23, 2009 10:47 AM
> >> >To: Anand Mitra
> >> >Cc: Anurag Agarwal; Larry Scheer; Maxim Kozlovsky
> >> >Subject: Re: PERFORCE change 32698 for review
> >> >
> >> >On Tue, 23 Jun 2009 03:25:04 -0700 Anand Mitra
> >> ><anand.mitra@onstor.com> wrote:
> >> >
> >> >> Change 32698 by amitra@amitra-2 on 2009/06/23 03:23:36
> >> >>
> >> >>         Adding opensol source code snapshot onnv_111.
> >> >>         Reviewed by: maximk
> >> >>
> >> >> Affected files ...
> >> >>
> >> >> ... //depot/dev/opensol/.hg/00changelog.i#1 add
> >> >.
> >> >.
> >> >.
> >> >
> >> >These 97,500 files don't belong in the dev branch.  Please remove
> >> >them and check them into their own, separate branch.  Preferably
> >> >with the admin command that actually removes them from the
> >> >repository so they don't take up space in that branch as well as
> >> >the new one.  Larry might have to take care of it in that case.
> >> >
> >> >Thanks,
> >> >
> >> >a
> >>
