Mailing-List: contact clerezza-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: clerezza-dev@incubator.apache.org
Received-SPF: neutral (nike.apache.org: local policy)
Subject: Re: Size of issues/patches
Mime-Version: 1.0 (Apple Message framework v1084)
Content-Type: multipart/alternative; boundary=Apple-Mail-40-484900089
From: Henry Story <henry.story@bblfish.net>
In-Reply-To: <BANLkTik0L34x3JWpdLKMkMF9pViXhEXqxQ@mail.gmail.com>
Date: Fri, 1 Jul 2011 09:16:21 +0200
Cc: clerezza-dev@incubator.apache.org
Message-Id: <87D4186D-9877-4AF7-AC77-0F6F63BE7093@bblfish.net>
References: <BANLkTik0L34x3JWpdLKMkMF9pViXhEXqxQ@mail.gmail.com>
To: Reto Bachmann-Gmuer <reto.bachmann@trialox.org>

--Apple-Mail-40-484900089
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain;
	charset=us-ascii


On 1 Jul 2011, at 08:16, Reto Bachmann-Gmuer wrote:

> On Fri, Jul 1, 2011 at 1:58 AM, Henry Story <henry.story@bblfish.net> =
wrote:
>=20
> You say in an earlier mail that my  patches are big. That is quite =
wrong. My patches were very small and constantly improved with real =
working examples over a period of a few months to make something that =
works in practice.=20
>=20
> This might point us straight to the core of the difficulties we're =
facing now. You worked for several month on an issue changing 3 =
different projects with commits associated to that issue.

Those three projects were projects I was working on myself. The code was =
quite isolated from anything anyone else was working on namely:

 - foaf-ssl tests
 - new features in account control panel=20
 - and of course new files in the rdf.scala package that did not affect =
anything else.

I don't think anyone was complaining of their code not working due to =
these new things coming up.

> During this time others were hindered from working on the affected =
projects.

Nonsense. Your a drama queen.=20

> Now you're close to a resolution of the issue with 711 new lines in =
the main branch of the main project. The patch I'm proposing to resolve =
the issue is less than 200 lines (again not including tests) of length.

You're a genius Reto. You know that! Don't be shy.

Henry


> While imho it offers some improvements compared with your patch it is =
obviously less comprehensive and I've omitted parts of what you =
implemented. But I think it is a suitable single step of an iterative =
development process, so it can be proposed and the community can decide =
if this is an improvement and can be accepted.  I think the parts I =
omitted can be part of other issues without disadvantages and thus =
should be parts of other issues.


>=20
> Cheers,
> Reto
>=20

Social Web Architect
http://bblfish.net/


--Apple-Mail-40-484900089--