Hi,
Sorry for replying to an old post. But I am having problems for uploading
svn files, as described below:
- In my repository (SVN) there was a single svn add and commit (resulting
in a revision #1683), for a huge set of files.
- I am trying to help my team upload these files to RB, but cannot create a
diff with this command below:
post-review --revision-range 0:HEAD --output-diff --debug
--disable-proxy
which gives me the below error:
rvnath@rvnath:~/workpad/mohit/ResourceManager$ post-review --revision-range
> 0:HEAD --output-diff --debug --disable-proxy
> >>> RBTools 0.4
> >>> Home = /home/rvnath
> >>> Disabling HTTP(s) proxy support
> >>> HTTP GETting api/
> >>> HTTP GETting http://172.19.2.226/reviews/api/info/
> >>> Using the new web API
> Failed to execute command: ['svn', 'diff', '--diff-cmd=diff', '-r',
> '0:HEAD']
> ["svn: Unable to find repository location for '' in revision 0\n"]
>
I downloaded and installed RBTools version 0.4, but still same problem.
Here is the version of my RBTools...
rvnath@rvnath:~/workpad/mohit/ResourceManager$ post-review --version
> RBTools 0.4
> rvnath@rvnath:~/workpad/mohit/ResourceManager$
>
I also tried with the script posted by Vester. This also gives me errors,
as below:
rvnath@rvnath:~/workpad/mohit/ResourceManager$ python
> ~/post-review-initial.py .
> Traceback (most recent call last):
> File "/home/rvnath/post-review-initial.py", line 147, in <module>
> main(sys.argv[1:])
> File "/home/rvnath/post-review-initial.py", line 131, in main
> diff = '\n'.join([create_file_diff(file) for file in files])
> File "/home/rvnath/post-review-initial.py", line 69, in create_file_diff
> diff[3] = '+++ ' + url + '\t(revision %s)' % (rev)
> IndexError: list assignment index out of range
> rvnath@rvnath:~/workpad/mohit/ResourceManager$
>
What I really want is a simple way to upload a revision snapshot, instead
of a diff.
Thanks for all the help in this regard..
rv
On Saturday, August 21, 2010 7:01:54 AM UTC+5:30, Nathan wrote:
>
> It is indeed possible to use post-review to generate an appropriate diff
> following the post-commit review model. RBTools version 0.2.0 contains a
> fix that makes it possible to generate the initial review by passing
> "--revision-range 0:HEAD" to post-review. If you're stuck with an older
> version of RBTools, the appropriate changes can be found at
> http://reviews.reviewboard.org/r/1429/.
>
> Cheers,
> Nathan Heijermans
>
> On Fri, Aug 20, 2010 at 4:39 PM, Manjit Singh <manjit.ga...@gmail.com>wrote:
>
>> Your script did the job for me. This is exactly what I needed to do (I
>> needed a full diff).
>> Thanks a lot.
>> -MS
>>
>>
>> On Sun, Aug 15, 2010 at 6:26 AM, Vesterbaek <vesterb...@gmail.com> wrote:
>>
>>>
>>> > Correct me if I'm wrong, but your script is useful when you want to put
>>> > existing committed code as a whole up for review, right? Such as when
>>> you're
>>> > doing a post-commit review for the first time of some code?
>>>
>>> Yup, correct. Having re-read the original question, I agree that if
>>> you want to review new code that has just been added (but not
>>> committed), post-review does the job. My script is for generating a
>>> full review (diff with left side blank) of already checked in code.
>>>
>>> - Jeppe
>>>
>>> --
>>> Want to help the Review Board project? Donate today at
>>> http://www.reviewboard.org/donate/
>>> Happy user? Let us know at http://www.reviewboard.org/users/
>>> -~----------~----~----~----~------~----~------~--~---
>>> To unsubscribe from this group, send email to
>>> reviewboard+unsubscr...@googlegroups.com
>>> For more options, visit this group at
>>> http://groups.google.com/group/reviewboard?hl=en
>>>
>>
>> --
>> Want to help the Review Board project? Donate today at
>> http://www.reviewboard.org/donate/
>> Happy user? Let us know at http://www.reviewboard.org/users/
>> -~----------~----~----~----~------~----~------~--~---
>> To unsubscribe from this group, send email to
>> reviewboard+unsubscr...@googlegroups.com
>> For more options, visit this group at
>> http://groups.google.com/group/reviewboard?hl=en
>>
>
>
--
Want to help the Review Board project? Donate today at
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/reviewboard?hl=en
