Re: diff_view page applying patch to tip of branch (and finding conflicts)
Hello, On Wednesday 18 April 2012 10:30:34 pm kredler wrote: Thanks for confirming! Can you provide a more verbose CLI example/explanation, please? suppose a-b-c-d is public changeset d-e' is made and posted using hg reviewboard plugin What command should be used? hg postreview e' What info is sent to reviewboard? The patch e' which also contains a reference to the parent d. Now imagine that you send d-x-y-z where z is your patch and x and y are not public, two patches will be sent to RB: a parent patch which is the concatenation of x and y, which includes a reference to the public parent d, and the actual patch to review z. If d-f is pushed to the public repo from a different cloned repo where f conflicts with e', I see that conflict in the diff viewer page that worked prior to f being pushed. Then you will have to merge e' and f. The other solutions would be to rebase e' on top of f, or apply e' as a patch on top of f. Is there a way to choose the changeset referenced in the public repo manually? (via hg reviewboard plug-in and/or via GUI) It seems that the -m/--master option could do that. Never used it though. Note that the plugin will send a parent patch from this specified master to your patch. kr On Wednesday, April 18, 2012 2:43:55 PM UTC-5, morisgi wrote: On Wednesday 18 April 2012 07:46:19 pm kredler wrote: Using mercurial repo and 'hg postreview ...'. If a patch applies cleanly to the tip of the branch, the view_diff page works correctly. However, if the files change in the branch subsequent to the RB post, I get a patch collision and the ...didn't apply cleanly. message. 1. Does RB intend to always apply the patch to a static changeset in the mercurial repo or, contrastingly, always to the tip? 2. If RB applies patch to static changeset, what mechanism identifies the changeset upon which the patch applies? Thank you! Patch is applied to a static changeset. The Mercurial-Reviewboard walks the history up to find the first changeset known by the mercurial repository referenced from RB. The delta from this changeset to the parent of the patch is sent to RB along with the patch itself. Regards. Gilles. -- 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
Problem in setting up 'post-review' on win7 machine
Hi, I have a 64 bit win 7 machine. I am trying to set up 'post-review' by using http://www.reviewboard.org/docs/codebase/dev/getting-started/ I have installed the prerequisites and created a '.reviewboardrc' file in my trunk directory with value: REVIEWBOARD_URL = http://16.150.16.57/reviewboard; Now I am trying to use post-review by following command: post-review -r 980 --username=x --password= --diff-filename=C:\Users\shaashis\Desktop\ReviewBoardDiffs\11440.diff the console result is as follows: GNU diff is required in order to generate diffs. Make sure it is installed and in the path. On Windows, you can install this from: http://gnuwin32.sourceforge.net/packages/diffutils.htm this is confusing since i have installed diff and the path is correctly set. what is the problem?, please help thanks Ashish -- 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
Re: Problem in setting up 'post-review' on win7 machine
Christian, Following is my result of 'diff --version': Usage: diff [-befhHimnNrsw] [-c[n]] [-C n] [-Dname] [-M mark] file1 file2 thanks Ashish On Fri, Apr 20, 2012 at 1:46 PM, Christian Hammond chip...@chipx86.comwrote: If you're getting that error, then you either don't have GNU diff or it's not in the path. Can you run: diff --version And see what the results are? Also, you shouldn't ever need to use --diff-filename= unless you're doing something pretty custom. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Apr 20, 2012 at 12:34 AM, Ashish Sharma ashiish.sha...@gmail.comwrote: Hi, I have a 64 bit win 7 machine. I am trying to set up 'post-review' by using http://www.reviewboard.org/docs/codebase/dev/getting-started/ I have installed the prerequisites and created a '.reviewboardrc' file in my trunk directory with value: REVIEWBOARD_URL = http://16.150.16.57/reviewboard; Now I am trying to use post-review by following command: post-review -r 980 --username=x --password= --diff-filename=C:\Users\shaashis\Desktop\ReviewBoardDiffs\11440.diff the console result is as follows: GNU diff is required in order to generate diffs. Make sure it is installed and in the path. On Windows, you can install this from: http://gnuwin32.sourceforge.net/packages/diffutils.htm this is confusing since i have installed diff and the path is correctly set. what is the problem?, please help thanks Ashish -- 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
Re: Response code: 207 -- The file was not found in the repository
Here is the --debug output https://gist.github.com/356cc0b7d25b7748e9bb I didn't see my previous reply, so had to post it again. Sorry if this is getting posted twice. -- 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
Re: Problem in setting up 'post-review' on win7 machine
That doesn't look to be GNU diff. That's the problem. It must be exactly GNU diff and not some alternative. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Apr 20, 2012 at 1:25 AM, Ashish Sharma ashiish.sha...@gmail.comwrote: Christian, Following is my result of 'diff --version': Usage: diff [-befhHimnNrsw] [-c[n]] [-C n] [-Dname] [-M mark] file1 file2 thanks Ashish On Fri, Apr 20, 2012 at 1:46 PM, Christian Hammond chip...@chipx86.comwrote: If you're getting that error, then you either don't have GNU diff or it's not in the path. Can you run: diff --version And see what the results are? Also, you shouldn't ever need to use --diff-filename= unless you're doing something pretty custom. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Apr 20, 2012 at 12:34 AM, Ashish Sharma ashiish.sha...@gmail.com wrote: Hi, I have a 64 bit win 7 machine. I am trying to set up 'post-review' by using http://www.reviewboard.org/docs/codebase/dev/getting-started/ I have installed the prerequisites and created a '.reviewboardrc' file in my trunk directory with value: REVIEWBOARD_URL = http://16.150.16.57/reviewboard; Now I am trying to use post-review by following command: post-review -r 980 --username=x --password= --diff-filename=C:\Users\shaashis\Desktop\ReviewBoardDiffs\11440.diff the console result is as follows: GNU diff is required in order to generate diffs. Make sure it is installed and in the path. On Windows, you can install this from: http://gnuwin32.sourceforge.net/packages/diffutils.htm this is confusing since i have installed diff and the path is correctly set. what is the problem?, please help thanks Ashish -- 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 -- 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
Re: Response code: 207 -- The file was not found in the repository
How is your repository configured in Review Board? Is the path set to the root of the Subversion repository, or a subdirectory within it? It'll need to be the very root of the repository. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Apr 20, 2012 at 1:37 AM, CoderTR tramasamy...@gmail.com wrote: Here is the --debug output https://gist.github.com/356cc0b7d25b7748e9bb I didn't see my previous reply, so had to post it again. Sorry if this is getting posted twice. -- 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
Re: Response code: 207 -- The file was not found in the repository
There is a leading slash in path of the file in diff output, as the
reviewboard also complains with this repsonse --
Got API Error 207 (HTTP code 400): The file was not found in the repository
Error data: {u'stat': u'fail', u'file':
u'//trunk/src/main/java/com/anasoft/os/s4j/SeleniumInstanceFactory.java',
u'err': {u'msg': u'The file was not found in the repository', u'code':
207}, u'revision': u'54'}
Your review request still exists, but the diff is not attached.
Do you think this could be a reason for the failure ?
--
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
Re: Response code: 207 -- The file was not found in the repository
Yeah, I'm just not sure where that's coming from. Can you also show me what
'svn info' shows?
Christian
--
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com
On Fri, Apr 20, 2012 at 2:01 AM, CoderTR tramasamy...@gmail.com wrote:
There is a leading slash in path of the file in diff output, as the
reviewboard also complains with this repsonse --
Got API Error 207 (HTTP code 400): The file was not found in the
repository
Error data: {u'stat': u'fail', u'file':
u'//trunk/src/main/java/com/anasoft/os/s4j/SeleniumInstanceFactory.java',
u'err': {u'msg': u'The file was not found in the repository', u'code':
207}, u'revision': u'54'}
Your review request still exists, but the diff is not attached.
Do you think this could be a reason for the failure ?
--
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
New version on RPM
Hey guys, We just built an RPM for CentOS of RB-1.6.5 based on CentOS-5 mainstream RPM providing RB-1.5.6 We tested it on CentOS-5.6. Here is the updated spec file and patches, so it would be nice if someone test it, and give feedback. -- Hatem Ounis. -- 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 RH01-RHEL5-Do-not-use-GUI-configuration.patch Description: Binary data FED01-Disable-ez_setup-when-installing-by-RPM.patch Description: Binary data FED02-Notify-WSGI-users-that-config-changes-are-needed.patch Description: Binary data FED03-Change-default-cache-file-path.patch Description: Binary data ReviewBoard.spec Description: Binary data
Re: Response code: 207 -- The file was not found in the repository
Oh, where are you running it from? What happens when you run 'svn info' from that location? post-review expects to be run from within the source tree. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Apr 20, 2012 at 2:10 AM, CoderTR tramasamy...@gmail.com wrote: Ok. Also, I am *not* running post-review command from my svn working directory. Here is the svn info output C:\Workspace\workspace\selenium4junit-read-onlysvn info Path: . Working Copy Root Path: C:\Workspace\workspace\selenium4junit-read-only URL: http://selenium4junit.googlecode.com/svn/trunk Repository Root: http://selenium4junit.googlecode.com/svn Repository UUID: 733c2150-6652-11de-9c70-efa4e9705d34 Revision: 72 Node Kind: directory Schedule: normal Last Changed Author: vojtech.szocs Last Changed Rev: 72 Last Changed Date: 2010-08-26 07:45:21 -0700 (Thu, 26 Aug 2010) -- 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
Re: Response code: 207 -- The file was not found in the repository
Oh, where are you running it from? * * *I am running it from c:\ (whereas the project working directory is C:\Workspace\workspace\* *selenium4junit-read-only)* What happens when you run 'svn info' from that location? * C:\svn info svn: E155007: 'C:\' is not a working copy* post-review expects to be run from within the source tree. *I remember reading somewhere that its not mandatory to run it within the working directory. Also, I just ran the command from my working directory and got the same result (207 failure). * -- 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
Re: Response code: 207 -- The file was not found in the repository
Oh, I was missing the --repository-url= parameter. So, to compute the base path, we take the Repository Root value from svn info and the URL value and get the part in URL that is after Repository Root. Normally, this will be something like /trunk If this value is empty, it hard-codes /. Somewhere, this is conflicting with how it's generating the diff. So try your post-review command from the checkout, but don't pass --repository-url, and see if that gets any further. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Apr 20, 2012 at 2:19 AM, CoderTR tramasamy...@gmail.com wrote: Oh, where are you running it from? * * *I am running it from c:\ (whereas the project working directory is C:\Workspace\workspace\* ***selenium4junit-read-only)* What happens when you run 'svn info' from that location? * C:\svn info svn: E155007: 'C:\' is not a working copy* post-review expects to be run from within the source tree. *I remember reading somewhere that its not mandatory to run it within the working directory. Also, I just ran the command from my working directory and got the same result (207 failure). * -- 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
Re: Response code: 207 -- The file was not found in the repository
Christian, Thanks for all the help.
I tried without --repository-url and ran it within working directory, I get
the same problem, I checked out /trunk/ and for some reason, the file path
still have //trunk.
So, I checked out the entire repository (including trunk, branches and
tags), and ran the command within working directory, now that the file is
posted with correct relative path in diff. But reviewboard still won't
accept the request --
Got API Error 207 (HTTP code 400): The file was not found in the
repository
Error data: {u'stat': u'fail', u'file': u
'/trunk/src/main/java/com/anasoft/o
s/s4j/SeleniumInstanceFactory.java', u'err': {u'msg': u'The file was not
found i
n the repository', u'code': 207}, u'revision': u'54'}
Error uploading diff
Your review request still exists, but the diff is not attached.
C:\Workspace\workspace\selenium4junit-all-read-onlysvn info
Path: .
Working Copy Root Path: C:\Workspace\workspace\selenium4junit-all-read-only
URL: http://selenium4junit.googlecode.com/svn
Repository Root: http://selenium4junit.googlecode.com/svn
Repository UUID: 733c2150-6652-11de-9c70-efa4e9705d34
Revision: 72
Node Kind: directory
Schedule: normal
Last Changed Author: vojtech.szocs
Last Changed Rev: 72
Last Changed Date: 2010-08-26 07:45:21 -0700 (Thu, 26 Aug 2010)
*
However, closer look at the error message, seems like reviewboard
complaining that the file is not there in revision 54 ( the file was added
in revision 55) which makes sense. But, I don't know why RB would complain
abt this --- *
Index: src/main/java/com/anasoft/os/s4j/SeleniumInstanceFactory.java
===
--- src/main/java/com/anasoft/os/s4j/SeleniumInstanceFactory.java
(revision 0)
+++ src/main/java/com/anasoft/os/s4j/SeleniumInstanceFactory.java
(revision 55)
@@ -0,0 +1,24 @@
+package com.anasoft.os.s4j;
+
+import com.anasoft.os.s4j.annotation.SeleniumSession;
+import com.thoughtworks.selenium.Selenium;
+
+/**
+ * Factory for creating {@link Selenium} session instances.
+ *
+ * @see SeleniumRunnerLogic
+ *
+ * @author vojtech.szocs
+ */
+public interface SeleniumInstanceFactory {
+
+/**
+ * Creates new {@link Selenium} instance for the given session
+ * configuration.
+ *
+ * @param sessionConfig {@link Selenium} session configuration.
+ * @return New {@link Selenium} session instance.
+ */
+Selenium createSelenium(SeleniumSession sessionConfig);
+
+}
--
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
Re: Response code: 207 -- The file was not found in the repository
There is something special about this revision range -- 54:55 I tried with diff between 69:70 and the changes have been posted successfully, got a review request created. Yayy!! Does it mean that post-review/reviewboard can't handle the case where the file was added new ?? I will continue to do some research tomorrow. Thanks again for your help. -- 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
Re: New version on RPM
On Fri, 2012-04-20 at 09:30 +0100, hatem ounis wrote: Hey guys, We just built an RPM for CentOS of RB-1.6.5 based on CentOS-5 mainstream RPM providing RB-1.5.6 We tested it on CentOS-5.6. Here is the updated spec file and patches, so it would be nice if someone test it, and give feedback. I maintain the EPEL packages for ReviewBoard. The reason that EPEL's copy of ReviewBoard is still on 1.5.6 is that RHEL 5 and CentOS 5 only have Django 1.2 available, while the minimum requirement for ReviewBoard 1.6 is 1.3. I'm assuming that this spec and patches you've submitted are based on a different Django package that's not generally-available. However, EPEL 6 recently released Django 1.3.1, so I now have ReviewBoard 1.6.5 built for RHEL/CentOS 6 in the epel-testing repository. -- 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
Re: New version on RPM
I'm interested in the use of RB on CentOS 5 not CentOS 6. And for Django I installed an RPM of version 1.3 we can find here http://code.vpac.org/trac/karaage/wiki/CentOSInstall As you can see in the spec file I set Django = 1.3 as a requirement. I had more trouble installing Django-1.3.1 on CentOS 5 -- Hatem Ounis. -- 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
Private Github / company account
Hi - we' were going to install reviewboard but have no access to our ApiV2 token. What's the ETA for getting RB working with the V3 github API? Thanks! Florian -- 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
Re: Private Github / company account
Hi Florian, I'm working every day on it. There's a bit to do. However, I think by this weekend I'll at the very least have a patch up for review that you can try out, if you want to be an early tester. I'm shooting for a release by early next week (hopefully). GitHub tells me they're looking into bringing API Tokens back to the Account page soon, but I don't quite know when that will happen. Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Apr 20, 2012 at 8:01 AM, Florian Leibert f...@leibert.de wrote: Hi - we' were going to install reviewboard but have no access to our ApiV2 token. What's the ETA for getting RB working with the V3 github API? Thanks! Florian -- 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
Re: Response code: 207 -- The file was not found in the repository
It shouldn't have a problem with new files. I'm not sure why it is in your case. Perhaps it's something that changed in how your version of Subversion represents new files in diffs? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org VMware, Inc. - http://www.vmware.com On Fri, Apr 20, 2012 at 3:28 AM, CoderTR tramasamy...@gmail.com wrote: There is something special about this revision range -- 54:55 I tried with diff between 69:70 and the changes have been posted successfully, got a review request created. Yayy!! Does it mean that post-review/reviewboard can't handle the case where the file was added new ?? I will continue to do some research tomorrow. Thanks again for your help. -- 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
Re: Issue 800 in reviewboard: Please add ability to replace the screenshot
Comment #6 on issue 800 by ecirce...@gmail.com: Please add ability to replace the screenshot http://code.google.com/p/reviewboard/issues/detail?id=800 At a minimum, it would be nice to be able to re-order the screenshots so that a sequence can be preserved. That way, i can add a replacement, remove the original, and drag the replacement to the same location where the original was, thus preserving the sequence of screenshots. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post to this group, send email to reviewboard-issues@googlegroups.com. To unsubscribe from this group, send email to reviewboard-issues+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/reviewboard-issues?hl=en.
