[Kmymoney-devel] Reviewboard and Git

[Alvaro Soliverez]

Hello Allan,

To create a patch, you can simply run:

git format-patch -n   <--- n is the number of commits you want to output as
patches.

So, if you want the last 3 commits in patch format, just run:

git format-patch -3


About the KMM group for Git Reviewboard, I've already requested it as part
of the git migration, and it's going to be done soon.

Regards,
Alvaro

On Tue, Nov 15, 2011 at 9:26 AM, Allan <agander93 at gmail.com> wrote:

> On 15/11/11 11:58, Cristian Oneț wrote:
>
>> 2011/11/15 Allan<agander93 at gmail.com>:
>>
>>> On 15/11/11 00:20, Allan wrote:
>>>
>>>>
>>>> On 14/11/11 22:16, Cristian Oneț wrote:
>>>>
>>>>>
>>>>> On Monday 14 November 2011 22:11:32 you wrote:
>>>>>
>>>>>>
>>>>>> Just finishing adding all the new files, etc. Then, see where I'm up
>>>>>> to
>>>>>> in this new world. What am I to do with the new patches? Is it a case
>>>>>> of resubmitting under git?
>>>>>>
>>>>>
>>>>> You can commit them to a local branch in your repository and after
>>>>> testing
>>>>> that everything is OK (you've ported the patch correctly) submit the
>>>>> patch
>>>>> obtained by running 'git format-patch master --stdout>  csv.patch' to
>>>>> review.
>>>>>
>>>>>
>>>> All seems OK and the patches are ready. I'm assuming that when KMM is
>>>> added to git reviewboard, that's where they need to go, but do I need to
>>>> copy the original svn description too? I'm assuming I do.
>>>>
>>>> The original review was submitted over two weeks go. Do I need a go
>>>> ahead to commit them?
>>>>
>>>> Allan
>>>>
>>>
>>> I'm having a couple of problems, I'm afraid.
>>>
>>> 1) I'd originally submitted a review for the csv plugin wizard on svn
>>> reviewboard and want now to proceed.  In the meantime, Git has come
>>> along so
>>> I've created a new patch file under Git.  Looking at the patch, I find it
>>> contains a reference to a test file I initially created and committed and
>>> then apparently removed successfully, but it still shows in the patch.
>>>  So,
>>> I deleted that from the file.
>>>
>>
>> Well if it appears in the patch then it must be there, how are you
>> exactly trying to create the patch?
>> Editing the patch file by hand is not a good idea AFAIK.
>>
>
> From Git/kmymoney, I ran 'git format-patch master --stdout>  csv.patch' as
> you suggested.
>
> The file (new.txt) does not exist in my /home folder.  However, I've just
> created a new patch via qgit and that seems OK.
>
>
>
>>  2) I found I couldn't login to git reviewboard with my original
>>> ID/password
>>> although it suggests I should be able to, so I registered a new id.
>>>
>>
>> Logging in with you identity.kde.org account should work, there si no
>> need for a new account.
>>
>>
> I don't have an identity.kde.org account, just the original svn
> reviewboard id/password.
>
> How do I contact admin to get my access back?
>
> Allan
>
>
>  Now logged in, I tried to submit the patch, but it gives an error - a
>>> missing terminator in the header.
>>>
>>>
>> That is probably because you manually changed the patch file.
>>
>>  Now, though, I can't sign in as my id/password is rejected, possibly
>>> because
>>> I'm still logged in somewhere else, but can't find where.
>>>
>>> Please, how do I get out of this mess?
>>>
>>> Thanks.
>>>
>>> Allan
>>>
>>
>
> ______________________________**_________________
> KMyMoney-devel mailing list
> KMyMoney-devel at kde.org
> https://mail.kde.org/mailman/**listinfo/kmymoney-devel<https://mail.kde.org/mailman/listinfo/kmymoney-devel>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20111115/7db41818/attachment.html>