Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
Received-SPF: error (athena.apache.org: local policy)
References: 
 <CACOnxCg-z83G4WOba2CQDDYRCSNMoOhSSDfi0urcG7ZyJ1WVWQ@mail.gmail.com>
 <DB6FA658-51BE-41AD-8A4A-1D545766D03D@gmail.com>
 <CACOnxCh2-y_dy6Wz1Ov2b03hBNJjHfoUqNN=+nyBnzPa22ipvw@mail.gmail.com>
 <CACOnxCinMHqXDV3DWGZAxpF-VjDn4RcjqRaaRE+4w4Mbu5fORA@mail.gmail.com>
From: Chip Childers <chip.childers@sungard.com>
In-Reply-To: 
 <CACOnxCinMHqXDV3DWGZAxpF-VjDn4RcjqRaaRE+4w4Mbu5fORA@mail.gmail.com>
Mime-Version: 1.0 (1.0)
Date: Mon, 27 May 2013 20:24:41 -0400
Message-ID: <9061577063239463257@unknownmsgid>
Subject: Re: Question about Review Request
To: "<dev@cloudstack.apache.org>" <dev@cloudstack.apache.org>
Content-Type: text/plain; charset=windows-1252
Content-Transfer-Encoding: quoted-printable

Can you send the patch to the list as a txt attachment with a subject
line that indicates you want to have it reviewed and committed?
Perhaps we have to do this the old fashioned way.

On May 27, 2013, at 7:46 PM, Mike Tutkowski
<mike.tutkowski@solidfire.com> wrote:

> Regardless of those whitespace messages, it looks like my new branch
> (solidfire_plugin2) does have the changes in the patch file (they show up
> as modified, but not staged files or - in the case of new files - untrack=
ed
> files).
>
>
> On Mon, May 27, 2013 at 5:36 PM, Mike Tutkowski <
> mike.tutkowski@solidfire.com> wrote:
>
>> Thanks for the reply, Sebastien!
>>
>> So, I went ahead and updated from upstream again and merged
>> upstream/master to master, then merged master to solidfire_plugin.
>>
>> I created another patch file and tried to upload it to Review Board, but
>> received the same error message.
>>
>> I then tried to apply the patch file to a new branch, called
>> solidfire_plugin2, that I created off of master.
>>
>> There was no error output when I did an apply --check, but when I actual=
ly
>> tried to apply the patch I received the following output (any thoughts?)=
:
>>
>> mtutkowski-LT:cloudstack mtutkowski$ git apply solidfire_plugin.patch
>> solidfire_plugin.patch:66: trailing whitespace.
>>
>> solidfire_plugin.patch:68: trailing whitespace.
>>
>> solidfire_plugin.patch:70: trailing whitespace.
>>
>> solidfire_plugin.patch:72: trailing whitespace.
>>
>> solidfire_plugin.patch:74: trailing whitespace.
>>
>> warning: squelched 708 whitespace errors
>> warning: 713 lines add whitespace errors.
>>
>>
>> On Mon, May 27, 2013 at 1:45 AM, Sebastien Goasguen <runseb@gmail.com>wr=
ote:
>>
>>>
>>> On May 27, 2013, at 1:55 AM, Mike Tutkowski <mike.tutkowski@solidfire.c=
om>
>>> wrote:
>>>
>>>> Hi everyone,
>>>>
>>>> I'm trying to submit code for the first time.
>>>>
>>>> I'm following the instructions here:
>>> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Review+Board+Gui=
delines
>>>>
>>>> When it says to upload my diff, I assume it means my .patch file. Is
>>> that
>>>> correct?
>>>>
>>>> I generate such a file this way:
>>>>
>>>> git format-patch upstream/master --stdout > solidfire_plugin.patch
>>>
>>> That should work. I usually just do:
>>> git format-patch master --stdout > my.patch
>>>
>>>> It works just fine, but when I try to upload it by clicking on the
>>> Create
>>>> Review Request button, I get the following error:
>>>>
>>>> The file
>>> 'plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixRe=
sourceBase.java'
>>>> (r5f45a62) could not be found in the repository
>>>> *
>>>> *
>>>
>>> Create a fresh branch out of your local master and try to see if your
>>> patch applies cleanly:
>>> git --check apply ( I think).
>>>
>>> Maybe it's a relative path issue (just a wild guess).
>>>
>>>
>>>> I'm not sure why it says this because this file is a part of the curre=
nt
>>>> repository.
>>>>
>>>> Would someone be able to explain what I might be doing wrong here?
>>>>
>>>> I did update from the ACS repo and merge its master (my upstream/maste=
r)
>>>> into my solidfire_plugin branch recently. After doing this, I committe=
d
>>> the
>>>> changes and made my .patch file.
>>>
>>> Did you check your patch making sure that all changed files were staged=
 ?
>>>
>>> I always do a git status -s , it's very helpful.
>>>
>>>> Thanks!
>>>>
>>>> --
>>>> *Mike Tutkowski*
>>>> *Senior CloudStack Developer, SolidFire Inc.*
>>>> e: mike.tutkowski@solidfire.com
>>>> o: 303.746.7302
>>>> Advancing the way the world uses the
>>>> cloud<http://solidfire.com/solution/overview/?video=3Dplay>
>>>> *=99*
>>
>>
>> --
>> *Mike Tutkowski*
>> *Senior CloudStack Developer, SolidFire Inc.*
>> e: mike.tutkowski@solidfire.com
>> o: 303.746.7302
>> Advancing the way the world uses the cloud<http://solidfire.com/solution=
/overview/?video=3Dplay>
>> *=99*
>
>
>
> --
> *Mike Tutkowski*
> *Senior CloudStack Developer, SolidFire Inc.*
> e: mike.tutkowski@solidfire.com
> o: 303.746.7302
> Advancing the way the world uses the
> cloud<http://solidfire.com/solution/overview/?video=3Dplay>
> *=99*