Mailing-List: contact dev-help@ignite.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ignite.apache.org
MIME-Version: 1.0
In-Reply-To: 
 <CAPJKmXk+myACNd1pPeY4ty2hLTJpwzipoo3wjCxTZf36G9uinw@mail.gmail.com>
References: 
 <CAEHhZ=mCGEOuMwvHFZMeQfLqoC-GJ7rn-Ms72qCGP5Evv7Zkxg@mail.gmail.com>
	<CAPJKmXk2yM8cm3xnbAsa9fWqoTxjbD6t8imh2UQ7RqyPEEcggA@mail.gmail.com>
	<5645D847.7060405@gridgain.com>
	<6D2F12AB-6E90-4102-BF8B-F185BA3FC337@gridgain.com>
	<CAPJKmXk+myACNd1pPeY4ty2hLTJpwzipoo3wjCxTZf36G9uinw@mail.gmail.com>
Date: Mon, 16 Nov 2015 15:02:11 +0300
Message-ID: 
 <CAEHhZ==UMaa3X_oxp-iisEA3BNPXDDD+wFo6XpK_tWBmrt2fsg@mail.gmail.com>
Subject: Re: IGNITE-1227 patch available
From: Amir Akhmedov <amir.akhmedov@gmail.com>
To: dev@ignite.apache.org
Content-Type: multipart/alternative; boundary=001a1145ace4ea37300524a72d87

--001a1145ace4ea37300524a72d87
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Denis, Artem,

Thank you guys for clarification. Latter I'll provide a pull-request for
this ticket.

I think the page [1] should also be updated, cause I followed the steps
from there.

[1] https://ignite.apache.org/community/contribute.html#contribute

2015-11-16 14:26 GMT+03:00 Artem Shutak <ashutak@gridgain.com>:

> Denis, Amir,
>
> First of all, need to understand how it works on TC. TC downloads a patch
> file and applies it by 'git am <patch-file>' to the latest master. That i=
s
> mentioned on How to Contribute
> <https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute> -
> see "2. Create a Patch-file"/"Applying"
>
> As I see in build logs (
>
> http://ci.ignite.apache.org/viewLog.html?buildId=3D75417&buildTypeId=3DIg=
nite_IgniteCacheFullApi&tab=3DbuildLog&state=3D&expand=3Dnone#_state=3D78&f=
ocus=3D78
> )
> the patch-file couldn't be applied.
>
> Step 3/7: Apply patch (if exist) (Gradle)
> ...
> :patchapply
> ...
> [10:32:08]Trying to apply patch.
> [10:32:08]Executing command 'git am
> dev-tools/IGNITE-1227-12772149.patch'...
> [10:32:08]Command output:
> [10:32:08]
> [10:32:08]Return code: 1
> [10:32:08]Patch was not applied successfully. Aborting patch applying.
>
> I did it locally and got "Patch format detection failed.". According to a
> head of the content of the file, the patch has been created by IDEA - it'=
s
> wrong way. To create patch manually you need to use "git format-patch" or=
 I
> recommend you to use "<ignite_home>/scripts/git-format-patch.sh". See How
> to Contribute
> <https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute> -
> "2.
> Create a Patch-file".
>
> Also, I want to say +1 for an Denis's advice to use a pull-request way.
> This way has no minuses and has a lots of pluses by my opinion against
> patch-way.
>
> Thanks,
> -- Artem --
>
> On Mon, Nov 16, 2015 at 1:39 PM, Denis Magda <dmagda@gridgain.com> wrote:
>
> > Amir, before making a review I want to be sure that all Ignite test
> suites
> > are green.
> >
> > However, seems that the patch based way contribution is broken. When I
> > open a test run info for any suite I see the following
> > http://ci.ignite.apache.org/viewLog.html?buildId=3D75417
> >
> > FAILURE: Build failed with an exception.
> >
> > * What went wrong:
> > Execution failed for task ':patchapply'.
> > > Process 'command '/usr/lib/jvm/java-7-oracle/bin/java'' finished with
> > non-zero exit value 1
> >
> >
> > Artem, do we still support the patch based contribution?
> >
> > Amir, until we=E2=80=99re resolving issue with the patches please follo=
w the
> > pull-request way. This way of contribution is a preferred one and works
> > fine:
> >
> >
> https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute#Howt=
oContribute-1.CreateGitHubpull-request
> > <
> >
> https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute#Howt=
oContribute-1.CreateGitHubpull-request
> > >
> > Before you push your pull request please get to know our coding
> guidelines
> > (identations, brackets usage, etc.). You have some issues in your code
> > related to guidelines. You=E2=80=99ll see them when familiarize yoursel=
f with
> this
> > doc:
> > https://cwiki.apache.org/confluence/display/IGNITE/Coding+Guidelines <
> > https://cwiki.apache.org/confluence/display/IGNITE/Coding+Guidelines>
> >
> >
> > Regards,
> > Denis
> >
> > > On 13 =D0=BD=D0=BE=D1=8F=D0=B1. 2015 =D0=B3., at 15:32, Denis Magda <=
dmagda@gridgain.com>
> wrote:
> > >
> > > Amir, great thanks!
> > >
> > > If you don't mind I'll review your changes in a couple of days.
> > >
> > >
> > > However, probably someone else will do this earlier:)
> > >
> > > --
> > > Denis
> > >
> > >
> > > From: Amir Akhmedov <amir.akhmedov@gmail.com <mailto:
> > amir.akhmedov@gmail.com>>
> > > Date: Fri, Nov 13, 2015 at 11:28 AM
> > > Subject: IGNITE-1227 patch available
> > > To: dev@ignite.apache.org <mailto:dev@ignite.apache.org>
> > >
> > >
> > > Hi Igniters,
> > >
> > > I submitted patch for IGNITE-1227
> > > <https://issues.apache.org/jira/browse/IGNITE-1227 <
> > https://issues.apache.org/jira/browse/IGNITE-1227>>, could review it
> > > whenever it is possible.
> > >
> > > Thanks.
> > >
> > > --
> > > Sincerely Yours Amir Akhmedov
> > >
> > >
> >
> >
>


--=20
Sincerely Yours Amir Akhmedov

--001a1145ace4ea37300524a72d87--