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-only>svn 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
