Re: CRITICAL: tuple index out of range error with RBTools 0.6 and Mercurial

2014-10-06 Thread OPL X
Hello,

After much trial and error, I managed to get ReviewBoard working!  I 
believe there was an issue with how I had set up the paths and permissions 
on the test machine I had.

One thing I noticed is that when I make any changes locally and execute:  
rbt post, I get 'CRITICAL: There are no outgoing changes'.  This only 
happens if I do not issue an 'hg commit'.

I had thought that I should be able to 'rbt post' without having to 
commit.  Am I missing something?  Thank you again.

On Friday, June 6, 2014 11:25:32 AM UTC+9, OPL X wrote:

 Hello again,

 I tried:

 rbt post --tracking-branch=default

 and came up with the same error.  I will try to take a bit of a deeper 
 look as to what may be going on with Mercurial.  On the same test machine 
 that I am using, Subversion works without any problems.

 If I discover the reason, I will be sure to let you know.  Thanks again!

 On Thursday, June 5, 2014 11:13:58 AM UTC+9, David Trowbridge wrote:

 I believe you need to use = when specifying long options:

 rbt post --tracking-branch=default

 That said, if your upstream branch is named 'default', it should be able 
 to find it. I'm not sure why there are no paths.branch entries in your 
 'hg showconfig'

 -David


 On Wed, Jun 4, 2014 at 6:35 PM, OPL X oplx...@gmail.com wrote:

 Hello,

 Thank you for taking a look at the problem.  When I issue:  rbt post 
 --tracking-branch default for on the designated Mercurial repository, the 
 same error message is reported.  Shall I just wait for version 0.6.1?  
 Thank you.


 On Wednesday, June 4, 2014 7:25:45 AM UTC+9, David Trowbridge wrote:

 The issue here is that rbt can't figure out which branch is your 
 upstream (that is, which one is also available on the repository 
 configured 
 in review board). You can use --tracking-branch to specify this manually.

 We'll get a fix for the terrible error message in for 0.6.1

 -David


 On Sun, May 25, 2014 at 6:31 PM, OPL X oplx...@gmail.com wrote:

 Hello,

 Here's the output from 'hg showconfig'

 ---

 bundle.mainreporoot=/home/oplx/rb_repo/hg0
 merge-tools.kdiff3.args=--auto --L1 base --L2 local --L3 other $base 
 $local $other -o $output
 merge-tools.kdiff3.regkey=Software\KDiff3
 merge-tools.kdiff3.regkeyalt=Software\Wow6432Node\KDiff3
 merge-tools.kdiff3.regappend=\kdiff3.exe
 merge-tools.kdiff3.fixeol=True
 merge-tools.kdiff3.gui=True
 merge-tools.kdiff3.diffargs=--L1 '$plabel1' --L2 '$clabel' $parent 
 $child
 merge-tools.gvimdiff.args=--nofork -d -g -O $local $other $base
 merge-tools.gvimdiff.regkey=Software\Vim\GVim
 merge-tools.gvimdiff.regkeyalt=Software\Wow6432Node\Vim\GVim
 merge-tools.gvimdiff.regname=path
 merge-tools.gvimdiff.priority=-9
 merge-tools.vimdiff.args=$local $other $base -c 'redraw | echomsg hg 
 merge conflict, type \:cq\ to abort vimdiff'
 merge-tools.vimdiff.check=changed
 merge-tools.vimdiff.priority=-10
 merge-tools.merge.check=conflicts
 merge-tools.merge.priority=-100
 merge-tools.gpyfm.gui=True
 merge-tools.meld.gui=True
 merge-tools.meld.args=--label='local' $local --label='merged' $base 
 --label='other' $other -o $output
 merge-tools.meld.check=changed
 merge-tools.meld.diffargs=-a --label='$plabel1' $parent 
 --label='$clabel' $child
 merge-tools.tkdiff.args=$local $other -a $base -o $output
 merge-tools.tkdiff.gui=True
 merge-tools.tkdiff.priority=-8
 merge-tools.tkdiff.diffargs=-L '$plabel1' $parent -L '$clabel' $child
 merge-tools.xxdiff.args=--show-merged-pane --exit-with-merge-status 
 --title1 local --title2 base --title3 other --merged-filename $output 
 --merge $local $base $other
 merge-tools.xxdiff.gui=True
 merge-tools.xxdiff.priority=-8
 merge-tools.xxdiff.diffargs=--title1 '$plabel1' $parent --title2 
 '$clabel' $child
 merge-tools.diffmerge.regkey=Software\SourceGear\SourceGear DiffMerge\
 merge-tools.diffmerge.regkeyalt=Software\Wow6432Node\SourceGear\SourceGear
  
 DiffMerge\
 merge-tools.diffmerge.regname=Location
 merge-tools.diffmerge.priority=-7
 merge-tools.diffmerge.args=-nosplash -merge -title1=local 
 -title2=merged -title3=other $local $base $other -result=$output
 merge-tools.diffmerge.check=changed
 merge-tools.diffmerge.gui=True
 merge-tools.diffmerge.diffargs=--nosplash --title1='$plabel1' 
 --title2='$clabel' $parent $child
 merge-tools.p4merge.args=$base $local $other $output
 merge-tools.p4merge.regkey=Software\Perforce\Environment
 merge-tools.p4merge.regkeyalt=Software\Wow6432Node\Perforce\
 Environment
 merge-tools.p4merge.regname=P4INSTROOT
 merge-tools.p4merge.regappend=\p4merge.exe
 merge-tools.p4merge.gui=True
 merge-tools.p4merge.priority=-8
 merge-tools.p4merge.diffargs=$parent $child
 merge-tools.tortoisemerge.args=/base:$base /mine:$local 
 /theirs:$other /merged:$output
 merge-tools.tortoisemerge.regkey=Software\TortoiseSVN
 merge-tools.tortoisemerge.regkeyalt=Software\Wow6432Node\TortoiseSVN
 merge-tools.tortoisemerge.check=changed
 merge-tools.tortoisemerge.gui=True
 merge-tools.tortoisemerge.priority=-8
 

stuck in last step - configuring email

2014-10-06 Thread neel roy
Hi,

Probably last step but I am stuck there anyway :)

So, I configured the email and found that account I am using does not have 
privilege of sending mails on behalf of other users. Asked IT guys to fix 
it. They have but now I have problem with plain authentication method. The 
diagnosis is based on following log from reviewboard:

- Error sending e-mail notification with subject 'Review request description' 
on behalf of 'user1' to 'user1,user2,user3': *No suitable 
authentication method found.*
Traceback (most recent call last):
  File 
/usr/local/lib/python2.7/dist-packages/ReviewBoard-2.0.8-py2.7.egg/reviewboard/notifications/email.py,
 line 279, in send_review_mail
message.send()
  File 
/usr/local/lib/python2.7/dist-packages/Django-1.6.7-py2.7.egg/django/core/mail/message.py,
 line 276, in send
return self.get_connection(fail_silently).send_messages([self])
  File 
/usr/local/lib/python2.7/dist-packages/Django-1.6.7-py2.7.egg/django/core/mail/backends/smtp.py,
 line 87, in send_messages
new_conn_created = self.open()
  File 
/usr/local/lib/python2.7/dist-packages/Django-1.6.7-py2.7.egg/django/core/mail/backends/smtp.py,
 line 54, in open
self.connection.login(self.username, self.password)
  File /usr/lib/python2.7/smtplib.py, line 611, in login
raise SMTPException(No suitable authentication method 
found.)*SMTPException: No suitable authentication method found.*


*As per question 
**http://stackoverflow.com/questions/1684534/python-and-sll-exchange-2007-no-suitable-authentication-method-found
 I think we require Plain Text Authentication method enabled on exchange 
server, which might be a no-no (don't know; haven't asked those IT guys yet). 
But before I ask them I wanted to know whether there is any other workaround?*


*Is there a way I can use non-plain text authentication method?*


*Appreciate any help,*

*-Neel.*

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
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/d/optout.


Re: CRITICAL: tuple index out of range error with RBTools 0.6 and Mercurial

2014-10-06 Thread christ...@beanbaginc.com
Hi,

For Git, Mercurial, etc., the workflow is:

1) Commit changes to a branch.
2) Run rbt post

The commit messages will then be used as the default summary and description 
for the review request.

Christian

--  
Christian Hammond - christ...@beanbaginc.com  
Review Board - https://www.reviewboard.org  
Beanbag, Inc. - https://www.beanbaginc.com

-Original Message-
From: OPL X oplx.0...@gmail.com
Reply: reviewboard@googlegroups.com reviewboard@googlegroups.com
Date: October 5, 2014 at 11:51:44 PM
To: reviewboard@googlegroups.com reviewboard@googlegroups.com
Subject:  Re: CRITICAL: tuple index out of range error with RBTools 0.6 and 
Mercurial

 Hello,
  
 After much trial and error, I managed to get ReviewBoard working! I
 believe there was an issue with how I had set up the paths and permissions
 on the test machine I had.
  
 One thing I noticed is that when I make any changes locally and execute:
 rbt post, I get 'CRITICAL: There are no outgoing changes'. This only
 happens if I do not issue an 'hg commit'.
  
 I had thought that I should be able to 'rbt post' without having to
 commit. Am I missing something? Thank you again.
  
 On Friday, June 6, 2014 11:25:32 AM UTC+9, OPL X wrote:
 
  Hello again,
 
  I tried:
 
  rbt post --tracking-branch=default
 
  and came up with the same error. I will try to take a bit of a deeper
  look as to what may be going on with Mercurial. On the same test machine
  that I am using, Subversion works without any problems.
 
  If I discover the reason, I will be sure to let you know. Thanks again!
 
  On Thursday, June 5, 2014 11:13:58 AM UTC+9, David Trowbridge wrote:
 
  I believe you need to use = when specifying long options:
 
  rbt post --tracking-branch=default
 
  That said, if your upstream branch is named 'default', it should be able
  to find it. I'm not sure why there are no paths. entries in your
  'hg showconfig'
 
  -David
 
 
  On Wed, Jun 4, 2014 at 6:35 PM, OPL X wrote:
 
  Hello,
 
  Thank you for taking a look at the problem. When I issue: rbt post
  --tracking-branch default for on the designated Mercurial repository, the
  same error message is reported. Shall I just wait for version 0.6.1?
  Thank you.
 
 
  On Wednesday, June 4, 2014 7:25:45 AM UTC+9, David Trowbridge wrote:
 
  The issue here is that rbt can't figure out which branch is your
  upstream (that is, which one is also available on the repository 
  configured
  in review board). You can use --tracking-branch to specify this manually.
 
  We'll get a fix for the terrible error message in for 0.6.1
 
  -David
 
 
  On Sun, May 25, 2014 at 6:31 PM, OPL X wrote:
 
  Hello,
 
  Here's the output from 'hg showconfig'
 
  ---
 
  bundle.mainreporoot=/home/oplx/rb_repo/hg0
  merge-tools.kdiff3.args=--auto --L1 base --L2 local --L3 other $base
  $local $other -o $output
  merge-tools.kdiff3.regkey=Software\KDiff3
  merge-tools.kdiff3.regkeyalt=Software\Wow6432Node\KDiff3
  merge-tools.kdiff3.regappend=\kdiff3.exe
  merge-tools.kdiff3.fixeol=True
  merge-tools.kdiff3.gui=True
  merge-tools.kdiff3.diffargs=--L1 '$plabel1' --L2 '$clabel' $parent
  $child
  merge-tools.gvimdiff.args=--nofork -d -g -O $local $other $base
  merge-tools.gvimdiff.regkey=Software\Vim\GVim
  merge-tools.gvimdiff.regkeyalt=Software\Wow6432Node\Vim\GVim
  merge-tools.gvimdiff.regname=path
  merge-tools.gvimdiff.priority=-9
  merge-tools.vimdiff.args=$local $other $base -c 'redraw | echomsg hg
  merge conflict, type \:cq\ to abort vimdiff'
  merge-tools.vimdiff.check=changed
  merge-tools.vimdiff.priority=-10
  merge-tools.merge.check=conflicts
  merge-tools.merge.priority=-100
  merge-tools.gpyfm.gui=True
  merge-tools.meld.gui=True
  merge-tools.meld.args=--label='local' $local --label='merged' $base
  --label='other' $other -o $output
  merge-tools.meld.check=changed
  merge-tools.meld.diffargs=-a --label='$plabel1' $parent
  --label='$clabel' $child
  merge-tools.tkdiff.args=$local $other -a $base -o $output
  merge-tools.tkdiff.gui=True
  merge-tools.tkdiff.priority=-8
  merge-tools.tkdiff.diffargs=-L '$plabel1' $parent -L '$clabel' $child
  merge-tools.xxdiff.args=--show-merged-pane --exit-with-merge-status  
  --title1 local --title2 base --title3 other --merged-filename $output
  --merge $local $base $other
  merge-tools.xxdiff.gui=True
  merge-tools.xxdiff.priority=-8
  merge-tools.xxdiff.diffargs=--title1 '$plabel1' $parent --title2
  '$clabel' $child
  merge-tools.diffmerge.regkey=Software\SourceGear\SourceGear DiffMerge\  
  merge-tools.diffmerge.regkeyalt=Software\Wow6432Node\SourceGear\SourceGear

  DiffMerge\
  merge-tools.diffmerge.regname=Location
  merge-tools.diffmerge.priority=-7
  merge-tools.diffmerge.args=-nosplash -merge -title1=local
  -title2=merged -title3=other $local $base $other -result=$output
  merge-tools.diffmerge.check=changed
  merge-tools.diffmerge.gui=True
  merge-tools.diffmerge.diffargs=--nosplash --title1='$plabel1'
  --title2='$clabel' $parent $child
  

Support for subversion file:/// URLs?

2014-10-06 Thread Kevyn Pair
Hi all,

Does any have any experience adding an SVN repository using the file:/// 
URL?

I get the following errors:
Unable to connect to a repository at URL 
'file:///Z:/Subversion%20repository/ blah'
Unable to open an ra_local session to URL
Unable to open repository 'file:///Z:/Subversion%20repository/blah'

Note: blah == rest of the path

In the past, I have used the http://; URL successfully, but I'm now trying 
to connect to a repository that doesn't have a web server.

Regards,
Kevyn

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
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/d/optout.


relaying email through ReviewBoard

2014-10-06 Thread Stefan Seefeld
Hello,

I have successfully set up a RB instance prototype, and can receive mail
notification on request creation as well as comments posted via the web
interface.
However, any replies sent to those messages go directly to the
author(s), and are not routed through the RB instance.
What settings am I missing that would allow me to do all RB
communication via email, i.e. that would allow replies to be routed back
through the RB instance (so they are properly recorded there) ?

The online docs on
https://www.reviewboard.org/docs/manual/2.0/admin/configuration/email-settings/#email-settings
are talking**about a sender e-mail address, but what exactly I need to
put there isn't entirely clear to me.**

Thanks,
Stefan

-- 

  ...ich hab' noch einen Koffer in Berlin...

-- 
Get the Review Board Power Pack at http://www.reviewboard.org/powerpack/
---
Sign up for Review Board hosting at RBCommons: https://rbcommons.com/
---
Happy user? Let us know at http://www.reviewboard.org/users/
--- 
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/d/optout.


Re: Issue 3529 in reviewboard: Incorrect escaping sequence with markdown

2014-10-06 Thread reviewboard

Updates:
Status: Duplicate
Mergedinto: 3602

Comment #1 on issue 3529 by chip...@gmail.com: Incorrect escaping sequence  
with markdown

https://code.google.com/p/reviewboard/issues/detail?id=3529

Created a master bug for all Markdown issues.

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Issue 3602 in reviewboard: Master bug: Plain text is rendered incorrectly when escaping for Markdown.

2014-10-06 Thread reviewboard


Comment #1 on issue 3602 by chip...@gmail.com: Master bug: Plain text is  
rendered incorrectly when escaping for Markdown.

https://code.google.com/p/reviewboard/issues/detail?id=3602

Issue 3529 has been merged into this issue.

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Issue 3604 in reviewboard: error at install time

2014-10-06 Thread reviewboard

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

New issue 3604 by makowski...@gmail.com: error at install time
https://code.google.com/p/reviewboard/issues/detail?id=3604

*** READ THIS BEFORE POSTING!
***
*** You must complete this form in its entirety, or your bug report will be
*** rejected.
***
*** If you have a security issue to report, please send it confidentially
to
*** secur...@reviewboard.org. Posting security-related issues to this bug
*** tracker causes us to have to do an emergency release.
***
*** For customer support, please post to reviewbo...@googlegroups.com
***
*** If you have a patch, please submit it to
http://reviews.reviewboard.org/
***
*** This bug tracker is public. Please check that any logs or other
information
*** that you include has been stripped of confidential information.


What version are you running?
0.8.11

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

What steps will reproduce the problem?
1. run setup.py install

What is the expected output? What do you see instead?
expect install, but get :
running build_media
Traceback (most recent call last):
  File contrib/internal/build-media.py, line 24, in module
ret = call_command('collectstatic', interactive=False, verbosity=2)
   
File /usr/lib/python2.7/site-packages/django/core/management/__init__.py,  
line 95, in call_command

raise CommandError(Unknown command: %r % name)
django.core.management.base.CommandError: Unknown command: 'collectstatic'


What operating system are you using? What browser?
Mageia Cauldron

Please provide any additional information below.

this occurs when trying to build the package for Mageia


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Issue 3605 in reviewboard: Reviewboard displays or ignores lines restored after deleting incorrectly

2014-10-06 Thread reviewboard

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

New issue 3605 by glind...@dynamiccontrols.com: Reviewboard displays or  
ignores lines restored after deleting incorrectly

https://code.google.com/p/reviewboard/issues/detail?id=3605

*** READ THIS BEFORE POSTING!
***
*** You must complete this form in its entirety, or your bug report will be
*** rejected.
***
*** If you have a security issue to report, please send it confidentially
to
*** secur...@reviewboard.org. Posting security-related issues to this bug
*** tracker causes us to have to do an emergency release.
***
*** For customer support, please post to reviewbo...@googlegroups.com
***
*** If you have a patch, please submit it to
http://reviews.reviewboard.org/
***
*** This bug tracker is public. Please check that any logs or other
information
*** that you include has been stripped of confidential information.


What version are you running?
2.0.6

What's the URL of the page containing the problem?
N/A

What steps will reproduce the problem?
1. delete a few lines from a file
2. add the diff to Reviewboard
3. publish the review
4. restore the deleted lines with a few small changes to the lines
5. add the updated diff to Reviewboard

What is the expected output? What do you see instead?
When I look at the changes between revision 1 and revision 2, I expect to  
see the restored lines as a change, but instead, Reviewboard shows no  
changes. If I expand the file, I do see the differences but they are not  
marked as changes by Reviewboard.


What operating system are you using? What browser?
$ uname -a
Linux 3.13.0-35-generic #62-Ubuntu SMP Fri Aug 15 01:58:42 UTC 2014 x86_64  
x86_64 x86_64 GNU/Linux

Google Chrome Version 36.0.1985.125

Please provide any additional information below.


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Issue 3604 in reviewboard: error at install time

2014-10-06 Thread reviewboard

Updates:
Status: SetupIssue

Comment #1 on issue 3604 by trowb...@gmail.com: error at install time
https://code.google.com/p/reviewboard/issues/detail?id=3604

If you want to install from git (or a tar.gz), you'll need to install a  
handful of other dependencies (django-pipeline, uglifyjs, lesscss). None of  
this is needed if you easy_install the .egg.


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Issue 3606 in reviewboard: Edit button appears next to image titles that the user doesn't have permission to edit

2014-10-06 Thread reviewboard

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

New issue 3606 by buck.a...@gmail.com: Edit button appears next to image  
titles that the user doesn't have permission to edit

https://code.google.com/p/reviewboard/issues/detail?id=3606

What version are you running?
2.0.6. This also reproduces with the version deployed on the demo site (2.1  
alpha 0).


What's the URL of the page containing the problem?
http://demo.reviewboard.org/r//

What steps will reproduce the problem?
1. Visit a review not posted by you that has images attached. (e.g.  
http://demo.reviewboard.org/r/1/ assuming you're not logged in as 'chipx86')
2. Note that an edit button appears next to the title of each image  
attachment, even though you are lacking permissions to edit the titles.


What is the expected output? What do you see instead?
The edit button should not appear.

What operating system are you using? What browser?
Linux; Firefox 29.0 and Firefox 30.0 both show this issue.

Please provide any additional information below.
Actually attempting to edit the image titles first shows the new title you  
input, but the edit is rejected server-side. Refreshing the page will  
restore the title of the image to whatever it was before.


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Issue 3607 in reviewboard: 4-tab indentation reduced to 3-tabs in diff viewer

2014-10-06 Thread reviewboard

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

New issue 3607 by christop...@delphix.com: 4-tab indentation reduced to  
3-tabs in diff viewer

https://code.google.com/p/reviewboard/issues/detail?id=3607

*** READ THIS BEFORE POSTING!
***
*** You must complete this form in its entirety, or your bug report will be
*** rejected.
***
*** If you have a security issue to report, please send it confidentially
to
*** secur...@reviewboard.org. Posting security-related issues to this bug
*** tracker causes us to have to do an emergency release.
***
*** For customer support, please post to reviewbo...@googlegroups.com
***
*** If you have a patch, please submit it to
http://reviews.reviewboard.org/
***
*** This bug tracker is public. Please check that any logs or other
information
*** that you include has been stripped of confidential information.


What version are you running?
2.0.8

What's the URL of the page containing the problem?
https://reviews.csiden.org/r/51/diff/#

What steps will reproduce the problem?
1. Look at the link in usr/src/cmd/zstreamdump/zstreamdump.c in the new  
version of the file lines 322 and 323.


What is the expected output? What do you see instead?
Those lines should be indented by 4 tabs in the diff viewer, as they are in  
the patch file. However when viewed on OSX with chrome the diff viewer only  
displays three tabs, as seen here:

https://reviews.csiden.org/r/111/file/5/

What operating system are you using? What browser?
I have only seen this issue with Chrome 37.0.2062.124 on OSX.
The issue was not reproducible with Safari on the same OSX machine or  
Chrome 37.0.2062.120 on Linux.


Please provide any additional information below.


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.