It just doesn't sound like that file's being tracked properly by Subversion.
Is it a new file? Did you do 'svn add' on it? Also, can you show me how you're executing post-review? Christian -- Christian Hammond - chip...@chipx86.com Review Board - http://www.reviewboard.org Beanbag, Inc. - http://www.beanbaginc.com On May 5, 2013, at 10:16 PM, Prashima <prashimasha...@gmail.com> wrote: > I get the same response - > C:\Workspace\Kite\Server\kite_server>svn diff > C:\Workspace\Kite\Server\kite_server\Kite\src\main\java\edu\kite\controller\QuizController.java > svn: E150000: > 'C:\Workspace\Kite\Server\kite_server\Kite\src\main\java\edu\kite\controller\QuizController.java' > is not under version control > > >svn --version > svn, version 1.7.8 (r1419691) > compiled Dec 12 2012, 21:11:09 > > >post-review --version > RBTools 0.5 > > On Monday, 6 May 2013 12:56:43 UTC+8, Christian Hammond wrote: > Hi, > > What happens if you run that diff command without the --diff-cmd part. > > Can you also tell me what versions of SVN and RBTools you're using? > > Christian > > > On May 5, 2013, at 18:52, Prashima <prashim...@gmail.com> wrote: > >> Hey Christian, >> >> The file does exist on the exact path, as listed, on my local machine. I >> have set the path for svn repository to root folder. However, I had not >> synced to the root. I tried that also- svn repo path set to root and I >> synced local copy of the root. But this also didn't help and I am getting >> the same error. >> >> I am pretty new to svn + reviewboard setting, but from what I have >> understood till now, my concern is that how will the 'svn diff >> --diff-cmd=diff ..." command work for post-review when there is not >> DUMMY_FILE name and --new-file option is passed to compute the diff for a >> newly created file? >> >> >> >> On Monday, 6 May 2013 07:00:36 UTC+8, Christian Hammond wrote: >> Hi, >> >> That script is ancient and definitely not what you want. In fact, you don't >> want anything in reviewboard.googlecode.com. Look at >> http://github.com/reviewboard/rbtools/ >> >> When you ran the 'svn diff' command, it said that file isn't under version >> control. Does that file, as listed, exist on the filesystem, or is the >> computed path wrong? >> >> What do you have for the Path and Mirror Path settings on the repository >> side? Are they pointing to the root of the SVN repository, or inside a >> subdirectory of it? It needs to be the root. >> >> Christian >> >> -- >> Christian Hammond - chi...@chipx86.com >> Review Board - http://www.reviewboard.org >> Beanbag, Inc. - http://www.beanbaginc.com >> >> >> On Sun, May 5, 2013 at 8:30 AM, Prashima <prashim...@gmail.com> wrote: >> I looked into the issues little more. As per the debug output and referring >> to the postreview.py code - >> http://reviewboard.googlecode.com/svn/trunk/wxpostreview/postreview.py (Not >> sure if I am referring to the right code here, please correct me if I am >> wrong) >> >> def diff(self, files): >> """ >> Performs a diff across all modified files in a Subversion repository. >> """ >> return execute('svn diff --diff-cmd=diff %s' % ' '.join(files)) >> >> 'svn diff --diff-cmd=diff <filenames> ' is the command that is getting used >> to generate the diff for all the SVN files. But this command itself doesn't >> work for uncommitted files, which are not versioned in svn. As the output >> below also confirms. >> >> E:\Prashima\Kite\server\Kite>svn diff --diff-cmd=diff >> src\main\java\edu\kite\controller\QuizController.java >> svn: E150000: >> 'E:\Prashima\Kite_Work\server\Kite\src\main\java\edu\kite\controller\QuizController.java' >> is not under version control >> >> >> I could only generate diff for a newly added file using following command. >> But I don't know yet how can I generate diff for set of files which have >> both types of files - the ones that are under version control and the >> uncommitted ones, which are not under version control. >> >> Any help this point onward? >> >> >> On Sunday, May 5, 2013 5:01:09 PM UTC+8, Prashima wrote: >> Hi, >> >> Does anyone has any idea about this problem? Specially for the case where >> post-review fails to post review for uncommitted, unversioned, new svn file. >> >> >> Thanks >> >> -- >> 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...@googlegroups.com >> For more options, visit this group at >> http://groups.google.com/group/reviewboard?hl=en >> --- >> You received this message because you are subscribed to the Google Groups >> "reviewboard" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to reviewboard...@googlegroups.com. >> For more options, visit https://groups.google.com/groups/opt_out. >> >> >> -- 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 --- You received this message because you are subscribed to the Google Groups "reviewboard" group. To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/groups/opt_out.
