[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-devel
Subject: Re: Uploading .diff files on reviewboard
From: Rahul Sharma <rahulveera2009 () gmail ! com>
Date: 2012-03-01 11:57:11
Message-ID: CAC3EFMgQhc_9w5Bnp2Q-zMpZ=0ePzKrO-CmifWaWSDWEwJ6M6Q () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
Hi Thomas,
I created the diff the same way as you said and perhaps reviewboard
accepted that. But I encountered another error. It says:
*The file 'home/rahul/kmplot/kmplot/ksliderwindow.cpp' (r7ca8503) could not
be found in the repository. *after I have uploaded my diff.
And I don't know what does that mean. So, could you please tell me?
Regards,
Rahul Sharma.
On Mon, Feb 27, 2012 at 2:18 AM, Thomas L=FCbking
<thomas.luebking@gmail.com>wrote:
> Am 26.02.2012, 20:30 Uhr, schrieb Rahul Sharma <rahulveera2009@gmail.com>=
:
>
>
> Hi,
>> I am trying to upload my .diff file on the reviewboard through the web U=
I
>> for reviewboard, but I am getting error.
>> It always says that the file I uploaded is not .diff file even I am
>> uploading the file of the .diff extension.
>>
>> Any suggestions??
>>
>
> The reviewboard is pretty picky about diffs.
> Create them using "git diff <hash>" of "git format-patch <hash>", eg
> git format-patch HEAD~3
> will create diffs for the 3 latest commits
> while
> git diff HEAD~2
> will create one patch against the branch as it was two commits ago.
>
> The reviewboard will also reject diffs against hashes it doesn't know
> (because they only exist in your local branch) - for this purpose you can
> create a diff agains a common hash and supply that as parenting patch.
>
>
> Cheers,
> Thomas
>
>
> Visit http://mail.kde.org/mailman/**listinfo/kde-devel#unsub<http://mail=
.kde.org/mailman/listinfo/kde-devel#unsub>to unsubscribe <<
>>>
>>
[Attachment #5 (text/html)]
Hi Thomas,<br>I created the diff the same way as you said and perhaps reviewboard \
accepted that. But I encountered another error. It says:<br><b><u>The file \
'home/rahul/kmplot/kmplot/ksliderwindow.cpp' (r7ca8503) could not be found in \
the repository</u>. </b>after I have uploaded my diff.<br>
<br>And I don't know what does that mean. So, could you please tell \
me?<br><br>Regards,<br>Rahul Sharma.<br><br><div class="gmail_quote">On Mon, Feb 27, \
2012 at 2:18 AM, Thomas Lübking <span dir="ltr"><<a \
href="mailto:thomas.luebking@gmail.com">thomas.luebking@gmail.com</a>></span> \
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex">Am 26.02.2012, 20:30 Uhr, schrieb Rahul Sharma <<a \
href="mailto:rahulveera2009@gmail.com" \
target="_blank">rahulveera2009@gmail.com</a>>:<div>
<div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"> Hi,<br>
I am trying to upload my .diff file on the reviewboard through the web UI<br>
for reviewboard, but I am getting error.<br>
It always says that the file I uploaded is not .diff file even I am<br>
uploading the file of the .diff extension.<br>
<br>
Any suggestions??<br>
</blockquote>
<br></div></div>
The reviewboard is pretty picky about diffs.<br>
Create them using "git diff <hash>" of "git format-patch \
<hash>", eg<br> git format-patch HEAD~3<br>
will create diffs for the 3 latest commits<br>
while<br>
git diff HEAD~2<br>
will create one patch against the branch as it was two commits ago.<br>
<br>
The reviewboard will also reject diffs against hashes it doesn't know (because \
they only exist in your local branch) - for this purpose you can create a diff agains \
a common hash and supply that as parenting patch.<br>
<br>
<br>
Cheers,<br>
Thomas<div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 \
.8ex;border-left:1px #ccc solid;padding-left:1ex"> Visit <a \
href="http://mail.kde.org/mailman/listinfo/kde-devel#unsub" \
target="_blank">http://mail.kde.org/mailman/<u></u>listinfo/kde-devel#unsub</a> to \
unsubscribe <<<br> </blockquote></blockquote>
</div></div></blockquote></div><br>
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic