Re: POST-REVIEW: viewdiff on xml give error

2012-08-22 Thread Nilesh Jaiswal
Hi All,

Please let us know how to resolve this viewdiff issue.

viewdiff reports patch failed error message but i checked the diff file i
saw as below

private void log(Exception e) {

@@ -86,4 +93,4 @@ public class ActionStatusEventHandlerThr

return ActionStatusEventHandlerThread;

}



-}

+}

*\ No newline at end of file*

So it seems that diff utility expect* **\ No newline at end of file* in
source file, how to override such behavior, Also we see patch diff issue
when we modify the xml file.

Appreciate your valuable comments.

Regards,
Nilesh

On Wed, Aug 15, 2012 at 7:11 PM, Nilesh Jaiswal nileshj...@gmail.comwrote:

 Hi All,

 We are facing issues when user clicks on viewdiff and the changed file is
 xml or if the changed file does have No new line at end of file

 Are we not supporting diff of xml file.

 Regards,
 Nilesh J



-- 
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: Got API Error 206 (HTTP code 400): The repository path specified is not in the list of known repositories

2012-08-22 Thread cmuser
thanks Christain for your inputs.

1) Is it necessary to have .reviewboardrc file? as all my developers wont 
be making it in there checkout wolud be little Add-on kinda so i can add 
this requirement in my commitinfo file like
post-review -dn --server=http://172.16.7.100:81 --username=admin 
--password=admin  -submit-as=cmuser  
--repository-url=:pserver:user:password@svn:/opt/cvsroot

can't this itself work?
2)what is this mean- when commit?

 Home = /
No supported repository could be accessed at the supplied url.

On Tuesday, August 21, 2012 11:59:14 PM UTC+5:30, Christian Hammond wrote:

 You don't want to be using --repository-url, if you have a .reviewboardrc 
 file.

 You also want to make sure you're running this from the checkout directory.

 Christian

 -- 
 Christian Hammond - chi...@chipx86.com javascript:
 Review Board - http://www.reviewboard.org
 VMware, Inc. - http://www.vmware.com


 On Tue, Aug 21, 2012 at 3:45 AM, cmuser kapila...@gmail.com javascript:
  wrote:


 I added this line in commitinfo..

 /usr/bin/post-review -dn --server=http://172.16.7.100:81   
 --submit-as=$name  --repository-url=:pserver:$user@svn:/opt/cvsroot

 -bash-3.2$ cvs ci -m BugID:47300 update   document.txt
 Post-review for /opt/cvsroot/test
  RBTools 0.4.1
  Home = /
 No supported repository could be accessed at the supplied url.
 A commit is occurring in /opt/cvsroot/test
 08/21/12 [16:13:16]: A commit is occurring in /opt/cvsroot/test
 08/21/12 [16:13:16]: Pre-commit check for document.txt
 cvs commit: Pre-commit check failed
 cvs [commit aborted]: correct above errors first!
 -bash-3.2$ cat .reviewboardrc

 REPOSITORY = 'svn:/opt/cvsroot'
 REVIEWBOARD_URL = 'http://172.16.7.100:81'
 -bash-3.2$

 reviewboard repo settings path- svn:/opt/cvsroot

 How can i debug this further?



 On Tuesday, August 21, 2012 4:11:45 PM UTC+5:30, cmuser wrote:

 My gratitude again for replying Christian.

 Versions-
 ReviewBoard==1.6.1
 RBTools==0.4.1





 On Tuesday, August 21, 2012 12:52:17 PM UTC+5:30, Christian Hammond 
 wrote:

 Oh, your RBTools is pretty old. You will need 0.4.x.

 What version of Review Board are you using?

 Christian

 -- 
 Christian Hammond - chi...@chipx86.com
 Review Board - http://www.reviewboard.org
 VMware, Inc. - http://www.vmware.com


 On Tue, Aug 21, 2012 at 12:09 AM, cmuser kapila...@gmail.com wrote:

 to show more error on m/c where cvs installed , reviewboard on another 
 m/c 

  cat .reviewboardrc
 REPOSITORY = 'svn:/opt/cvsroot'

 REVIEWBOARD_URL = 'http://172.16.7.100:81'
 -bash-3.2$ post-review   **
   ** 
 There was an error creating this review request.

 The repository path svn.mmt.com:/opt/cvsroot is not in the

 list of known repositories on the server.

 Ask the administrator to add this repository to the Review Board 
 server.
 For information on adding repositories, please read
 http://www.reviewboard.org/**docs/manual/dev/admin/**
 management/repositories/http://www.reviewboard.org/docs/manual/dev/admin/management/repositories/
 -bash-3.2$




 On Monday, August 20, 2012 4:38:36 PM UTC+5:30, cmuser wrote:


 I have reviewboard install on server named ABC(172.16.7.100)  my 
 cvs is installed on server DEF
 when i commit code from ser ABC it works but same command when i 
 try on ser-DEF it fails with below error. Can some1 help where i am 
 going 
 wrong?
 Not sure if .reviewboardrc file should also be there, but i have 
 tried with that also but still it failed

 cat .reviewboardrc

 REPOSITORY = ':pserver:cmsuer@def:/opt/**cvsr**oot'

 REVIEWBOARD_URL = 'http://172.16.7.100:81'

 command on changed files-

 $post-review --debug 
 --server=http://172.16.7.100:**8**1http://172.16.7.100:81--username=admin
  --password=admin  --repository-url=':pserver:
 **cmu**ser@def:/opt/cvsroot'
  RBTools 0.3
  svn info :pserver:kapilap@def:/opt/**cvsr**oot
  repository info: Path: svn.mmt.com:/opt/cvsroot, Base path: None, 
 Supports changesets: False
  cvs diff -uN
  HTTP GETting api/
  HTTP GETting 
  http://172.16.7.100:81/api/**inf**o/http://172.16.7.100:81/api/info/
  Using the new web API
  Attempting to create review request on 
 def.domain.com:/opt/cvsroot for None
  HTTP POSTing to 
  http://172.16.7.100:81/api/**rev**iew-requests/http://172.16.7.100:81/api/review-requests/:
   
 {'repository': 'def.domain.com:/opt/cvsroot'}
  Got API Error 206 (HTTP code 400): The repository path specified 
 is not in the list of known repositories
  Error data: {'stat': 'fail', 'repository': 
 'def.domain.com:/opt/cvsroot', 'err': {'msg': 'The repository path 
 specified is not in the list of known repositories', 'code': 206}}

 There was an error creating this review request.

 The repository path def.domain.com:/opt/cvsroot is not in the
 list of known repositories on the server.



  -- 
 Want to help the Review Board project? Donate today at 
 http://www.reviewboard.org/**donate/http://www.reviewboard.org/donate/
 Happy user? Let us know at 

Re: Trying to configure CVS repository but always gets A repository was not found at the specified path.

2012-08-22 Thread cmuser
Nopes no gain :(

Still the same error-

-bash-3.2$ cvs -d :pserver:kapilap@svn:/opt/cvsroot ci -m BugID:47300 
testing document.txt
Post-review for /opt/cvsroot/test
 RBTools 0.4.1
 Home = /
No supported repository could be accessed at the supplied url.


In commitinfo 
/usr/bin/post-review -dn --server=http://172.16.7.100:81   
--submit-as=$name  --repository-url=:pserver:kapilap@svn:/opt/cvsroot

reviewboard settings for cvs path-svn:/opt/cvsroot
.reviewboardrc =REVIEWBOARD_URL = 'http://172.16.37.170'




On Tuesday, August 21, 2012 9:37:36 PM UTC+5:30, iamagui wrote:

 cmuser,

 I am not sure about REPOSITORY in .reviewboardrc but could you try just 
 the IP for REVIEWBOARD_URL (drop the colon (:) and the port (81)), that is

 REVIEWBOARD_URL = http://172.16.17.100;


-- 
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: Trying to configure CVS repository but always gets A repository was not found at the specified path.

2012-08-22 Thread cmuser
correction-

.reviewboardrc =REVIEWBOARD_URL = 'http://172.16.7.100http://172.16.37.170
'

On Wednesday, August 22, 2012 2:39:25 PM UTC+5:30, cmuser wrote:

 Nopes no gain :(

 Still the same error-

 -bash-3.2$ cvs -d :pserver:kapilap@svn:/opt/cvsroot ci -m BugID:47300 
 testing document.txt
 Post-review for /opt/cvsroot/test
  RBTools 0.4.1
  Home = /
 No supported repository could be accessed at the supplied url.


 In commitinfo 
 /usr/bin/post-review -dn --server=http://172.16.7.100:81   
 --submit-as=$name  --repository-url=:pserver:kapilap@svn:/opt/cvsroot

 reviewboard settings for cvs path-svn:/opt/cvsroot
 .reviewboardrc =REVIEWBOARD_URL = 'http://172.16.37.170'




 On Tuesday, August 21, 2012 9:37:36 PM UTC+5:30, iamagui wrote:

 cmuser,

 I am not sure about REPOSITORY in .reviewboardrc but could you try just 
 the IP for REVIEWBOARD_URL (drop the colon (:) and the port (81)), that is

 REVIEWBOARD_URL = http://172.16.17.100;



-- 
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: post-review failing from post-commit hook, but not when run manually

2012-08-22 Thread cmuser
Thanks peter for your inputs tried your inputs but still failed.
Here what i did-
*[root@svn CVSROOT]# cat commitinfo*
ALL  /opt/cvsroot/CVSROOT/postreview.sh

*[root@svn CVSROOT]# cat /opt/cvsroot/CVSROOT/postreview.sh*
echo Post-review for $1
name=kapila
path=`echo $PATH`
export 
PATH=$path:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/javancss-32.53/bin:/opt/jdk1.7.0//bin:/usr/local/apache-ant-1.8.1//bin:/usr/local/apache-maven-2.1.0/bin:/usr/local/apr/bin:/usr/local/mysql/bin::/usr/kerberos/sbin:/usr/kerberos/bin:/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/apache2/bin:/root/bin:/home/kapilacvs
echo $PATH
export HOME=/home/kapilacvs
echo $HOME
export GROUP=cvsdev
/usr/bin/post-review -dn --server=http://172.16.37.170:81   
--submit-as=$name  --repository-url=:pserver:kapilap@svn:/opt/cvsroot

*[root@svn CVSROOT]# id kapilap*
id: kapilap: No such user (since its cvs pserver user)
~

then i tried to commit 
[kapila.narang@svn ]$ ll /home/kapilacvs/test
[kapila.narang@svn ]$ cvs -d :pserver:kapilap@svn:/opt/cvsroot ci -m 
BugID:47300 testing test/document.txt
Post-review for /opt/cvsroot/test
/bin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/javancss-32.53/bin:/opt/jdk1.7.0//bin:/usr/local/apache-ant-1.8.1//bin:/usr/local/apache-maven-2.1.0/bin:/usr/local/apr/bin:/usr/local/mysql/bin::/usr/kerberos/sbin:/usr/kerberos/bin:/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/apache2/bin:/root/bin:/home/kapilacvs
/home/kapilacvs
 RBTools 0.4.1
 Home = /home/kapilacvs
No supported repository could be accessed at the supplied url.
A commit is occurring in /opt/cvsroot/test
cvs commit: Pre-commit check failed
cvs [commit aborted]: correct above errors first!


Not able to understand what is causing error :(

On Tuesday, August 21, 2012 9:55:19 PM UTC+5:30, Peter Konneker wrote:

 So first I set the path, home, and group variable and exported them from 
 the svn hook.

 export 
 PATH=/sbin:/bin:/usr/sbin:/usr/bin:/usr/games:/usr/local/sbin:/usr/local/bin:/usr/X11R6/bin:/home/$AUTHOR/bin
 export HOME=/home/$AUTHOR
 export GROUP=mygroup


 If that doesn't work for you let me know. There's a ton of stuff I had to 
 try to get my hook working the way I wanted, haha.

 -Peter


-- 
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: Can't upgrade to 1.6.11

2012-08-22 Thread Igor Berger
I understand that first I have to upgrade Review Board then the site.

My attempt to upgrade Review Board is ignored, since easy_install is 
telling me that 1.6.9 is the latest version (which is what I already have 
installed).

But according to reviewboard.org, the latest version is 1.6.11.


On Tuesday, August 21, 2012 6:26:55 PM UTC-4, Christian Hammond wrote:

 I see the upgrade of Review Board, but not the site. Make sure you follow 
 the documentation on upgrading your Review Board site directory.

 Christian

 -- 
 Christian Hammond - chi...@chipx86.com javascript:
 Review Board - http://www.reviewboard.org
 VMware, Inc. - http://www.vmware.com


 On Tue, Aug 21, 2012 at 3:22 PM, Igor Berger codew...@gmail.comjavascript:
  wrote:

 Hello,

 I'm trying to upgrade Review Board 1.6.9 to 1.6.11.
 I get no errors, but doesn't upgrade either (please see below).

 Any ideas?

 Thank you,
 Igor.


 
 root@mybox:~# easy_install -U ReviewBoard
 Searching for ReviewBoard
 Reading http://pypi.python.org/simple/ReviewBoard/
 Reading http://www.reviewboard.org/
 Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.5/
 Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.6/
 Reading http://www.review-board.org/
 Reading http://downloads.review-board.org/releases/ReviewBoard/1.0/
 Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.0/
 Reading http://downloads.review-board.org/releases/
 Best match: ReviewBoard 1.6.9
 Processing ReviewBoard-1.6.9-py2.6.egg
 ReviewBoard 1.6.9 is already the active version in easy-install.pth
 Installing rb-site script to /usr/local/bin
 Installing rbssh script to /usr/local/bin

 Using /usr/local/lib/python2.6/dist-packages/ReviewBoard-1.6.9-py2.6.egg
 Reading http://downloads.reviewboard.org/mirror/
 Processing dependencies for ReviewBoard
 Finished processing dependencies for ReviewBoard
 

 --
 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 javascript:
 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: Trying to configure CVS repository but always gets A repository was not found at the specified path.

2012-08-22 Thread iamagui
What do you get when you click the 

1) http://172.16.7.100:81

2) http://172.16.7.100

Try: (Also try everything with OUT the --repository-url)

1) /usr/bin/post-review -dn --submit-as=$name  --repository-url=:pserver:
kapilap@svn:/opt/cvsroot

2) /usr/bin/post-review -dn --submit-as=$name  
--repository-url=svn:/opt/cvsroot

Then

Remove/rename .reviewboardrc and try

1) /usr/bin/post-review -dn --server=http://172.16.7.100:81   
--submit-as=$name  --repository-url=:pserver:
kapilap@svn:/opt/cvsroot

2) /usr/bin/post-review -dn --server=http://172.16.7.100   
--submit-as=$name  --repository-url=:pserver:
kapilap@svn:/opt/cvsroot

3) /usr/bin/post-review -dn --server=http://172.16.7.100:81   
--submit-as=$name  --repository-url=svn:/opt/cvsroot

4) /usr/bin/post-review -dn --server=http://172.16.7.100   
--submit-as=$name  --repository-url=svn:/opt/cvsroot

Hope one of this to work.

-- 
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: post-review failing from post-commit hook, but not when run manually

2012-08-22 Thread Peter Konneker
Is this a cvs or svn server?

I'm not familiar with this url format. So that is probably what it's
failing on.


 :pserver:kapilap@svn:/opt/cvsroot


How is your cvs/svn server setup? How do you access it remotely?

-Peter

On Wed, Aug 22, 2012 at 5:50 AM, cmuser kapila.ko...@gmail.com wrote:

 Thanks peter for your inputs tried your inputs but still failed.
 Here what i did-
 *[root@svn CVSROOT]# cat commitinfo*
 ALL  /opt/cvsroot/CVSROOT/postreview.sh

 *[root@svn CVSROOT]# cat /opt/cvsroot/CVSROOT/postreview.sh*
 echo Post-review for $1
 name=kapila
 path=`echo $PATH`
 export
 PATH=$path:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/javancss-32.53/bin:/opt/jdk1.7.0//bin:/usr/local/apache-ant-1.8.1//bin:/usr/local/apache-maven-2.1.0/bin:/usr/local/apr/bin:/usr/local/mysql/bin::/usr/kerberos/sbin:/usr/kerberos/bin:/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/apache2/bin:/root/bin:/home/kapilacvs
 echo $PATH
 export HOME=/home/kapilacvs
 echo $HOME
 export GROUP=cvsdev
 /usr/bin/post-review -dn --server=http://172.16.37.170:81
 --submit-as=$name  --repository-url=:pserver:kapilap@svn:/opt/cvsroot

 *[root@svn CVSROOT]# id kapilap*
 id: kapilap: No such user (since its cvs pserver user)
 ~

 then i tried to commit
 [kapila.narang@svn ]$ ll /home/kapilacvs/test
 [kapila.narang@svn ]$ cvs -d :pserver:kapilap@svn:/opt/cvsroot ci -m
 BugID:47300 testing test/document.txt
 Post-review for /opt/cvsroot/test

 /bin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/javancss-32.53/bin:/opt/jdk1.7.0//bin:/usr/local/apache-ant-1.8.1//bin:/usr/local/apache-maven-2.1.0/bin:/usr/local/apr/bin:/usr/local/mysql/bin::/usr/kerberos/sbin:/usr/kerberos/bin:/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/apache2/bin:/root/bin:/home/kapilacvs
 /home/kapilacvs
  RBTools 0.4.1
  Home = /home/kapilacvs
 No supported repository could be accessed at the supplied url.
 A commit is occurring in /opt/cvsroot/test
 cvs commit: Pre-commit check failed
 cvs [commit aborted]: correct above errors first!


 Not able to understand what is causing error :(

-- 
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: post-review failing from post-commit hook, but not when run manually

2012-08-22 Thread kapila . kohli
It is cvs though our linux m/c is called svn as earlier we had svn which we 
migrated to cvs :)
Sent from BlackBerry® on Airtel

-Original Message-
From: Peter Konneker pkonne...@wahoo.com
Sender: reviewboard@googlegroups.com
Date: Wed, 22 Aug 2012 11:01:22 
To: reviewboard@googlegroups.com
Reply-To: reviewboard@googlegroups.com
Subject: Re: post-review failing from post-commit hook, but not when run 
manually

Is this a cvs or svn server?

I'm not familiar with this url format. So that is probably what it's
failing on.


 :pserver:kapilap@svn:/opt/cvsroot


How is your cvs/svn server setup? How do you access it remotely?

-Peter

On Wed, Aug 22, 2012 at 5:50 AM, cmuser kapila.ko...@gmail.com wrote:

 Thanks peter for your inputs tried your inputs but still failed.
 Here what i did-
 *[root@svn CVSROOT]# cat commitinfo*
 ALL  /opt/cvsroot/CVSROOT/postreview.sh

 *[root@svn CVSROOT]# cat /opt/cvsroot/CVSROOT/postreview.sh*
 echo Post-review for $1
 name=kapila
 path=`echo $PATH`
 export
 PATH=$path:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/javancss-32.53/bin:/opt/jdk1.7.0//bin:/usr/local/apache-ant-1.8.1//bin:/usr/local/apache-maven-2.1.0/bin:/usr/local/apr/bin:/usr/local/mysql/bin::/usr/kerberos/sbin:/usr/kerberos/bin:/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/apache2/bin:/root/bin:/home/kapilacvs
 echo $PATH
 export HOME=/home/kapilacvs
 echo $HOME
 export GROUP=cvsdev
 /usr/bin/post-review -dn --server=http://172.16.37.170:81
 --submit-as=$name  --repository-url=:pserver:kapilap@svn:/opt/cvsroot

 *[root@svn CVSROOT]# id kapilap*
 id: kapilap: No such user (since its cvs pserver user)
 ~

 then i tried to commit
 [kapila.narang@svn ]$ ll /home/kapilacvs/test
 [kapila.narang@svn ]$ cvs -d :pserver:kapilap@svn:/opt/cvsroot ci -m
 BugID:47300 testing test/document.txt
 Post-review for /opt/cvsroot/test

 /bin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/local/javancss-32.53/bin:/opt/jdk1.7.0//bin:/usr/local/apache-ant-1.8.1//bin:/usr/local/apache-maven-2.1.0/bin:/usr/local/apr/bin:/usr/local/mysql/bin::/usr/kerberos/sbin:/usr/kerberos/bin:/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/apache2/bin:/root/bin:/home/kapilacvs
 /home/kapilacvs
  RBTools 0.4.1
  Home = /home/kapilacvs
 No supported repository could be accessed at the supplied url.
 A commit is occurring in /opt/cvsroot/test
 cvs commit: Pre-commit check failed
 cvs [commit aborted]: correct above errors first!


 Not able to understand what is causing error :(

-- 
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: ReviewBoard 1.6.3 and Subversion 1.7.x Compatibility

2012-08-22 Thread mxbraun
Yesterday, I updated our Reviewboard (1.5.x) to SVN 1.7.6 and pysvn-1.7.6.  
It did not have any effect on this issue--the problem is still there. So: 
updating to the latest pysvn does *not* solve the problem.

m@

On Tuesday, March 6, 2012 6:01:46 AM UTC-6, daniel@googlemail.com wrote:

 I believe there was an update to pysvn at the weekend - so it might be 
 worth checking that and seeing if that resolves the issues. 

 On Feb 14, 9:26 pm, David Trowbridge trowb...@gmail.com wrote: 
  The critical issue is that pysvn does not yet support SVN 1.7, and 
 generates 
  broken results. If someone can find a workaround we're happy to 
 integrate 
  it, 
  but otherwise I think we're going to wait for pysvn to update. 
  
  -David 
  
  
  
  
  
  
  
  On Tue, Feb 14, 2012 at 1:19 PM, phoke paul.w.h...@gmail.com wrote: 
   We upgraded to SVN 1.7 when it came out late last year. Unfortunately, 
   we realized later that ReviewBoard does not have all of the proper 
   support for SVN 1.7.  There are several other people encountering 
   similar issues/challenges with SVN 1.7 and ReviewBoard compatibility. 
   The main issue is due to the fact that the diff output that SVN 1.7 
   produces contains some new property entries in the diffi file that 
   ReviewBoard or RB Tools does not seem to understand. 
  
   Related thread: 
  http://code.google.com/p/reviewboard/issues/detail?id=2359 
  
   Is there any progress being made on SVN 1.7 compatibility support with 
   ReviewBoard? This issue seems to have been around for several months 
   now, but I have not seen any messages that indicate any progress or 
   when a fix might be be available in a new RB release. 
  
   Please let me know if you have any information on this topic. 
  
   Thanks for your help! 
   Paul 
  
   -- 
   Want to help the Review Board project? Donate today at 
  http://www.reviewboard.org/donate/ 
   Happy user? Let us know athttp://www.reviewboard.org/users/ 
   -~--~~~~--~~--~--~--- 
   To unsubscribe from this group, send email to 
   reviewboard...@googlegroups.com javascript: 
   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

Visual Studio 2010 integration

2012-08-22 Thread Ivan Gavryliuk
I wrote a concept preview of a Visual Studio 2010 extension. It works only with 
SVN and post-commit reviews and is a self-contained package (no python, svn 
cmdline, setuptools etc required).

Search for IronBoard in Extension Manager or go here 
http://ironboard.codeplex.com/. Feel free to comment so I can improve and 
stabilize.

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


Start new app

2012-08-22 Thread kvr
Hi,

Can we create new apps for reviewboard using manage.py?
I have tried but couldn't succeed.

Please tell me how to create new apps for reviewboard.

Regards,
kvr

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


Destination file path normalization for display

2012-08-22 Thread Tomasz Moń
Hi everyone,

Currently the templates use comment.filediff.dest_file directly in review 
emails and in the review comments. While this looks fairly good for VCSes 
like git or svn, it is pretty unreadable in case of Clear Case (it's pretty 
common for Clear Case extended paths to be over few hundred characters).

The scmtools have the normalize_path_for_display() function which, in case 
of Clear Case, indeed makes the extended paths readable.
Hence, I would like to know, whether there is any reason for it to be used 
only in diffutils.py? In my opinion, the paths should get normalized to 
display not only in diffviewer, but also in emails as well as comments.

Regards,
Tomasz

-- 
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: Start new app

2012-08-22 Thread Christian Hammond
What do you want to create these for?

Christian


On Aug 22, 2012, at 4:27, kvr kvr@gmail.com wrote:

 Hi,
 
 Can we create new apps for reviewboard using manage.py?
 I have tried but couldn't succeed.
 
 Please tell me how to create new apps for reviewboard.
 
 Regards,
 kvr
 
 -- 
 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: Can't upgrade to 1.6.11

2012-08-22 Thread Christian Hammond
Oh, I see.

Try:

easy_install ReviewBoard==1.6.11

Or download the egg file manually and pass that to easy_install.

Christian



On Aug 22, 2012, at 7:18, Igor Berger codewiz...@gmail.com wrote:

 I understand that first I have to upgrade Review Board then the site.
 
 My attempt to upgrade Review Board is ignored, since easy_install is telling 
 me that 1.6.9 is the latest version (which is what I already have installed).
 
 But according to reviewboard.org, the latest version is 1.6.11.
 
 
 On Tuesday, August 21, 2012 6:26:55 PM UTC-4, Christian Hammond wrote:
 I see the upgrade of Review Board, but not the site. Make sure you follow the 
 documentation on upgrading your Review Board site directory.
 
 Christian
 
 -- 
 Christian Hammond - chi...@chipx86.com
 Review Board - http://www.reviewboard.org
 VMware, Inc. - http://www.vmware.com
 
 
 On Tue, Aug 21, 2012 at 3:22 PM, Igor Berger codew...@gmail.com wrote:
 Hello,
 
 I'm trying to upgrade Review Board 1.6.9 to 1.6.11.
 I get no errors, but doesn't upgrade either (please see below).
 
 Any ideas?
 
 Thank you,
 Igor.
 
 
 
 root@mybox:~# easy_install -U ReviewBoard
 Searching for ReviewBoard
 Reading http://pypi.python.org/simple/ReviewBoard/
 Reading http://www.reviewboard.org/
 Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.5/
 Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.6/
 Reading http://www.review-board.org/
 Reading http://downloads.review-board.org/releases/ReviewBoard/1.0/
 Reading http://downloads.reviewboard.org/releases/ReviewBoard/1.0/
 Reading http://downloads.review-board.org/releases/
 Best match: ReviewBoard 1.6.9
 Processing ReviewBoard-1.6.9-py2.6.egg
 ReviewBoard 1.6.9 is already the active version in easy-install.pth
 Installing rb-site script to /usr/local/bin
 Installing rbssh script to /usr/local/bin
 
 Using /usr/local/lib/python2.6/dist-packages/ReviewBoard-1.6.9-py2.6.egg
 Reading http://downloads.reviewboard.org/mirror/
 Processing dependencies for ReviewBoard
 Finished processing dependencies for ReviewBoard
 
 
 --
 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
 
 -- 
 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: Destination file path normalization for display

2012-08-22 Thread Christian Hammond
Hey,

The normalization function is newer than the rendering code and was just never 
updated. I'd be happy to take a patch that uses it.

Christian


On Aug 22, 2012, at 1:49, Tomasz Moń deso...@gmail.com wrote:

 Hi everyone,
 
 Currently the templates use comment.filediff.dest_file directly in review 
 emails and in the review comments. While this looks fairly good for VCSes 
 like git or svn, it is pretty unreadable in case of Clear Case (it's pretty 
 common for Clear Case extended paths to be over few hundred characters).
 
 The scmtools have the normalize_path_for_display() function which, in case of 
 Clear Case, indeed makes the extended paths readable.
 Hence, I would like to know, whether there is any reason for it to be used 
 only in diffutils.py? In my opinion, the paths should get normalized to 
 display not only in diffviewer, but also in emails as well as comments.
 
 Regards,
 Tomasz
 -- 
 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

Issue 2714 in reviewboard: I'm not able to upload a diff file using ClearCase as Repository.

2012-08-22 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 2714 by gh.capon...@gmail.com: I'm not able to upload a diff file  
using ClearCase as Repository.

http://code.google.com/p/reviewboard/issues/detail?id=2714

* NOTE: Do not post confidential information in this bug report. *
*   If you need immediate support, please contact*
*   reviewbo...@googlegroups.com *

What version are you running?
1.6.11

What's the URL of the page containing the problem?
New Review Request

What steps will reproduce the problem?
1. Trying to upload a diff file using UI or post-review

What is the expected output? What do you see instead?
The expected output would be the diff file being uploaded properly.

What operating system are you using? What browser?
Windows 7 - Google Chrome

Please provide any additional information below.

When I set a Snapshot View as Repository, I can't upload directly the diff  
file.
The New Review Request UI complains that the diff file is not in a valid  
diff format (diff generated with cleartool).
So I tried with post-review and I get the error 207 - The file was not  
found in the repository.

With the --output-diff parameter I'm able to see the diff file.

When I try with a Dynamic View, I can't even see the repository on New  
Review Request.
It shows only “None - Graphics Only” for repository, but the repository was  
added and marked to be shown in administrator area.
And, of course, if I try using post-review I get The error 206 - The  
repository path specified is not in the list of known repositories

With the --output-diff parameter I'm able to see the diff file.

Any ideas?

Thanks in advance.

--
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.



Issue 2715 in reviewboard: Filter widget on admin page home-reviews-reviewrequest obscures the right side of the page

2012-08-22 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 2715 by tedmjohn...@gmail.com: Filter widget on admin  
page home-reviews-reviewrequest obscures the right side of the page

http://code.google.com/p/reviewboard/issues/detail?id=2715

* NOTE: Do not post confidential information in this bug report. *
*   If you need immediate support, please contact*
*   reviewbo...@googlegroups.com *

What version are you running?
1.6.3

What's the URL of the page containing the problem?
http://internal-server.com/admin/db/reviews/reviewrequest/

What steps will reproduce the problem?
1. Visit page with Chrome, Safari, or new versions of IE
2. Try to minimize or otherwise move the Filter so you can see the page
3.

What is the expected output? What do you see instead?
Expected: be able to minimize or remove the filter
Instead: I can't see the date column

What operating system are you using? What browser?
Win7 and OSX. Safari, Chrome, and IE


Please provide any additional information below.
I tried a Search on the RB bug tracker for various combinations of filter  
and review request and did not see anything, so I filed this new one.


Attached is what I see (with sensitive data crossed out). There is no  
apparent way to remove the filter which is obscuring the page.


Attachments:
Screen Shot 2012-08-22 at 10.57.42 AM.jpg  288 KB

--
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.



Re: Issue 2707 in reviewboard: Unable to publish a review with comments on a screenshot

2012-08-22 Thread reviewboard


Comment #1 on issue 2707 by matt.mul...@gmail.com: Unable to publish a  
review with comments on a screenshot

http://code.google.com/p/reviewboard/issues/detail?id=2707

I just tested with 1.7 beta 1 and this issue appears to be fixed there. The  
Open an Issue checkbox shows up on comments on images there.


--
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.



Issue 2716 in reviewboard: Error when çi was attaching a picture

2012-08-22 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 2716 by luifer.r...@gmail.com: Error when çi was attaching a  
picture

http://code.google.com/p/reviewboard/issues/detail?id=2716

* NOTE: Do not post confidential information in this bug report. *
*   If you need immediate support, please contact*
*   reviewbo...@googlegroups.com *

What version are you running?


What's the URL of the page containing the problem?


What steps will reproduce the problem?
1.
2.
3.

What is the expected output? What do you see instead?


What operating system are you using? What browser?


Please provide any additional information below.


--
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.



Re: Issue 2716 in reviewboard: Error when çi was attaching a picture

2012-08-22 Thread reviewboard

Updates:
Status: NeedInfo

Comment #1 on issue 2716 by trowb...@gmail.com: Error when çi was attaching  
a picture

http://code.google.com/p/reviewboard/issues/detail?id=2716

We need you to fill out the form, including the full contents of the error  
any anything applicable in the server logs.


--
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.



Issue 2717 in reviewboard: cannot use space in svn path setting

2012-08-22 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 2717 by zhouding...@gmail.com: cannot use space in svn path  
setting

http://code.google.com/p/reviewboard/issues/detail?id=2717

* NOTE: Do not post confidential information in this bug report. *
*   If you need immediate support, please contact*
*   reviewbo...@googlegroups.com *

What version are you running?
1.6.3

What's the URL of the page containing the problem?
none

What steps will reproduce the problem?
1.add a svn path with space, failed.(http://localhost/trunk/client 1.0)
2.add a svn path with %20 to replace space,  
ok,but.(http://localhost/trunk/client%201.0)

when create a new review request,failed forever.
3.add a svn path without space(http://localhost/trunk/),
create a new review request(base dir:/client 1.0),ok.

What is the expected output? What do you see instead?
add full svn path with space in setting.

What operating system are you using? What browser?
Windows 2008R2+chrome/IE/Firefox

Please provide any additional information below.


--
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.



Issue 2718 in reviewboard: the photo will be shown in landscape when it is taken in portrait

2012-08-22 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 2718 by zhihui.c...@yasofon.com: the photo will be shown in  
landscape when it is taken in portrait

http://code.google.com/p/reviewboard/issues/detail?id=2718

1 enter photosActivity
2 Add Photos
3 Take Photo
4 view the photo in the UploadPhotoActivity

--
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.



Re: Issue 2718 in reviewboard: the photo will be shown in landscape when it is taken in portrait

2012-08-22 Thread reviewboard

Updates:
Status: NotABug

Comment #1 on issue 2718 by trowb...@gmail.com: the photo will be shown in  
landscape when it is taken in portrait

http://code.google.com/p/reviewboard/issues/detail?id=2718

This is the bug tracker for Review Board, which doesn't appear to be what  
you wanted.


--
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.