Mailing-List: contact gora-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: gora-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of lewis.mcgibbney@gmail.com
 designates 74.125.82.175 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAGaRif2-niptZ2gAycbt9VpCYH8ZDUM5dDYPx4xxVLUMSWzCAA@mail.gmail.com>
References: 
 <CAGaRif3UiibQFk+EHbXV4qU6mNiP4dQ8eVVoEL5J40bKZ5gcPQ@mail.gmail.com>
	<D5C4DC98-012C-4F58-A96E-24308F60AC8B@jpl.nasa.gov>
	<CAGaRif2q1OfwPCMYqx6jbSpwE7Hm-jezfj-sW0-TOCUpSwe9HA@mail.gmail.com>
	<CAGaRif0LDnCK0Dp+K9DsctgbMO-xfqOda=mVG7th1GuJW9cBLg@mail.gmail.com>
	<CAMUu0w8ZkOFzCYjHGe3zMG5Eh29d-qR6jPCyjHYhNOeHeBU-Hw@mail.gmail.com>
	<CAGaRif2TD6Tt9Ea__x5E+R0A93M=uEZK7_=Dnh577zeBKbTgZA@mail.gmail.com>
	<CAMUu0w-dGjRqz8fMWB65nBZBf_=zxTA-FWMUpL_Tm5D8gR2TWw@mail.gmail.com>
	<CAGaRif2-niptZ2gAycbt9VpCYH8ZDUM5dDYPx4xxVLUMSWzCAA@mail.gmail.com>
Date: Wed, 4 Jan 2012 19:13:43 +0000
Message-ID: 
 <CAGaRif3E1kD0jXbKN1VKBz3NGewJOvBcD=8rpHFoe_S60GpqKw@mail.gmail.com>
Subject: Re: Taking notice of community comments
From: Lewis John Mcgibbney <lewis.mcgibbney@gmail.com>
To: gora-dev@incubator.apache.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Hi Enis,

.gitignore files have now been added back into SVN area. Apologies for
taking so long to get this reverted.

Committed @ r1227267 in trunk

Thanks

Lewis
On Thu, Dec 15, 2011 at 12:26 PM, Lewis John Mcgibbney
<lewis.mcgibbney@gmail.com> wrote:
> OK so to clarify.
>
> I've removed the extra jar files which were littering the place. I've
> also removed the .gitignore files whereever I saw them. I'll revert
> the .gitignore files for the reasons you mention as they are more than
> justified Enis.
>
> Thanks
>
> Lewis
>
> On Wed, Dec 14, 2011 at 8:01 PM, Enis S=F6ztutar <enis.soz@gmail.com> wro=
te:
>> Thanks Lewis for taking care of this. I agree that we should remove the
>> extra jars, and cleanup the unnecessary files. The only thing is that,
>> since we should continue supporting the git repo, when you check out the
>> code from git://git.apache.org/gora.gitthat we should be able to compile
>> and run the unit tests.
>>
>> We may opt to keep the .gitignore (and similar) files in the repo, but n=
ot
>> release them in the tarball, but it is too much of a hassle to maintain
>> this. Besides I think of .gitignore files as a part of the code (since i=
t
>> configures your build structure), so we can release them together with t=
he
>> code in the tarball.
>>
>> Thanks.
>> Enis
>>
>> On Tue, Dec 13, 2011 at 3:22 AM, Lewis John Mcgibbney <
>> lewis.mcgibbney@gmail.com> wrote:
>>
>>> Hi Enis,
>>>
>>> No I'm not using Git, but realise that a few of you were/are,
>>> therefore I am more than happy for you to revert the changes I made to
>>> suit those Git repos users.
>>>
>>> If there is anything else I can do then please say and I will help to
>>> make the necessary changes.
>>>
>>> Thanks
>>>
>>> On Mon, Dec 12, 2011 at 10:56 PM, Enis S=F6ztutar <enis.soz@gmail.com>
>>> wrote:
>>> > Hi,
>>> >
>>> > I think we should keep the .gitignore files, and revert the commit fo=
r
>>> > this. The git/svn flame wars is still very hot in members@, but even =
if
>>> we
>>> > will not have read/write git access, we will always have at least
>>> read-only
>>> > git access at git.apache.org. Some people (like me) prefer to use git=
 as
>>> > their main tool, and we have a git repo at
>>> > git://git.apache.org/gora.gitthat we should support. Besides, the
>>> > release CAN and SHOULD include the
>>> > .gitignore files for git to work correctly.
>>> >
>>> > Normally, we do not need more than the top level .gitignore, but sinc=
e
>>> for
>>> > the build to work with ant, we needed those empty dirs at
>>> > (gora-xxx/lib-ext, etc), and since git does not track empty dirs, the
>>> > default way to add them to repo is to add an empty .gitignore. Now th=
at
>>> we
>>> > have switched to maven, we probably do not need some of those dirs, s=
o I
>>> > propose to revert the commit, and selectively do remove the ones not
>>> > needed. One more reason to revert the change is that the patch for is=
sue
>>> > GORA-63 seems to be accidentally committed at
>>> > gora-core/src/main/java/org/apache/gora/compiler/GoraCompiler.java.
>>> >
>>> > Lewis, are you using the git repo. If not, I can do the changes if yo=
u
>>> > prefer.
>>> >
>>> > Thanks,
>>> > Enis
>>> >
>>> > On Sun, Dec 11, 2011 at 1:01 PM, Lewis John Mcgibbney <
>>> > lewis.mcgibbney@gmail.com> wrote:
>>> >
>>> >> Hi,
>>> >>
>>> >> You should be able to see this has now been addressed and Committed =
@
>>> >> revision 1213062 in trunk.
>>> >>
>>> >> Thanks
>>> >>
>>> >> Lewis
>>> >>
>>> >> On Thu, Dec 8, 2011 at 10:16 PM, Lewis John Mcgibbney
>>> >> <lewis.mcgibbney@gmail.com> wrote:
>>> >> > OK please see below
>>> >> >>
>>> >> >>
>>> >> >> Yeah we can remove the .gitnore files, feel free to svn remove th=
em,
>>> +1.
>>> >> >
>>> >> > will do
>>> >> >
>>> >> >>
>>> >> >>
>>> >> >> I'd like to move away from having any of these files in trunk sin=
ce
>>> we
>>> >> >> don't
>>> >> >> need them as they are all available on Central right? Let's get r=
id
>>> of
>>> >> >> them...
>>> >> >
>>> >> > Yeah, we've moved completely away from the cassandra and hector
>>> versions
>>> >> and
>>> >> > now support 1.X, in addition this thrift jar looks pretty our of
>>> place as
>>> >> > I've upgraded the HBase dependency to 0.90.4 (latest stable as wel=
l).
>>> >> >
>>> >> > I'll look into it and get a Jira logged and patch submitted.
>>> >> >
>>> >> > ta
>>> >> >
>>> >> >>
>>> >> >>
>>> >> >> Cheers,
>>> >> >> Chris
>>> >> >>
>>> >> > Thanks
>>> >> > Lewis
>>> >> >
>>> >> >
>>> >> > --
>>> >> > Lewis
>>> >> >
>>> >>
>>> >>
>>> >>
>>> >> --
>>> >> Lewis
>>> >>
>>>
>>>
>>>
>>> --
>>> Lewis
>>>
>
>
>
> --
> Lewis


--=20
Lewis