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

2014-05-25 Thread OPL X
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
merge-tools.tortoisemerge.diffargs=/base:$parent /mine:$child 
/basename:'$plabel1' /minename:'$clabel'
merge-tools.ecmerge.args=$base $local $other --mode=merge3 --title0=base 
--title1=local --title2=other --to=$output
merge-tools.ecmerge.regkey=Software\Elli\xc3\xa9 Computing\Merge
merge-tools.ecmerge.regkeyalt=Software\Wow6432Node\Elli\xc3\xa9 
Computing\Merge
merge-tools.ecmerge.gui=True
merge-tools.ecmerge.diffargs=$parent $child --mode=diff2 
--title1='$plabel1' --title2='$clabel'
merge-tools.filemerge.executable=/Developer/Applications/Utilities/FileMerge.app/Contents/MacOS/FileMerge
merge-tools.filemerge.args=-left $other -right $local -ancestor $base 
-merge $output
merge-tools.filemerge.gui=True
merge-tools.beyondcompare3.args=$local $other $base $output /ro 
/lefttitle=local /centertitle=base /righttitle=other /automerge 
/reviewconflicts /solo
merge-tools.beyondcompare3.regkey=Software\Scooter Software\Beyond Compare 3
merge-tools.beyondcompare3.regname=ExePath
merge-tools.beyondcompare3.gui=True
merge-tools.beyondcompare3.priority=-2
merge-tools.beyondcompare3.diffargs=/lro /lefttitle='$plabel1' 
/righttitle='$clabel' /solo /expandall $parent $child
merge-tools.bcompare.args=$local $other $base -mergeoutput=$output -ro 
-lefttitle=parent1 -centertitle=base -righttitle=parent2 
-outputtitle=merged -automerge -reviewconflicts -solo
merge-tools.bcompare.premerge=False
merge-tools.bcompare.gui=True
merge-tools.bcompare.priority=-1
merge-tools.bcompare.diffargs=-lro -lefttitle='$plabel1' 
-righttitle='$clabel' -solo -expandall $parent $child
merge-tools.winmerge.args=/e /x /wl /ub /dl other /dr local $other $local 
$output
merge-tools.winmerge.regkey=Software\Thingamahoochie\WinMerge
merge-tools.winmerge.regkeyalt=Software\Wow6432Node\Thingamahoochie\WinMerge\
merge-tools.winmerge.regname=Executable
merge-tools.winmerge.check=changed
merge-tools.winmerge.gui=True
merge-tools.winmerge.priority=-10
merge-tools.winmerge.diffargs=/r /e /x /ub /wl /dl '$plabel1' /dr '$clabel' 
$parent $child

Re: Issue 3370 in reviewboard: condense diffs function runs out of memory on larger installs

2014-05-25 Thread reviewboard


Comment #1 on issue 3370 by christia...@gmail.com: condense diffs function  
runs out of memory on larger installs

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

I just migrated the database from MySQL to postgres (on the same machine).  
I ran the condensediffs command again and this time it completed  
successfully. The rb-site script was maxing out at around 2.1GB memory  
usage in this case.


Not sure if the dumpdb/loaddb commands perform some sort of sanitation on  
the data or if somehow on postgres the behaviour is different.


--
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 3329 in reviewboard: Show proper error message when attempting to publish review request with duplicate commit ID

2014-05-25 Thread reviewboard

Updates:
Status: Confirmed
Labels: Component-Reviews

Comment #4 on issue 3329 by chip...@gmail.com: Show proper error message  
when attempting to publish review request with duplicate commit ID

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

This is certainly harder to hit now, but not fully fixed.

The following repro case results in two review requests with the same  
commit ID:


1. Run `rbt post`
2. Update the branch, run `rbt post` again, creating a new review request.
3. Update the original review request with `rbt post -r ID`.
4. Publish both review requests.

While it doesn't seem to break anymore (fortunately), it does result in two  
review requests with the same commit ID.


We need to check more thoroughly that publishing a draft wouldn't result in  
a duplicate commit ID, before any actual publishing takes place.


--
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 3370 in reviewboard: condense diffs function runs out of memory on larger installs

2014-05-25 Thread reviewboard

Updates:
Status: Fixed
Owner: chip...@gmail.com
Labels: Milestone-Release2.0.x Component-RB-Site

Comment #2 on issue 3370 by chip...@gmail.com: condense diffs function runs  
out of memory on larger installs

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

We're releasing 2.0.1 this Sunday or Monday, and that should address the  
memory issues. Or, at least improve upon them.


This should be fixed now on release-2.0.x (84d17a3)

--
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 3329 in reviewboard: Show proper error message when attempting to publish review request with duplicate commit ID

2014-05-25 Thread reviewboard


Comment #5 on issue 3329 by chip...@gmail.com: Show proper error message  
when attempting to publish review request with duplicate commit ID

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

(Actually, I imagine it just doesn't crash because it's SQLite3, which I  
think is less strict?)


--
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 3375 in reviewboard: Diff viewewr cuts off long lines on Firefox

2014-05-25 Thread reviewboard

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

New issue 3375 by christia...@gmail.com: Diff viewewr cuts off long lines  
on Firefox

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


What version are you running?
2.0

What's the URL of the page containing the problem?
http://myreviewboardserver/r/1234/diff/#

What steps will reproduce the problem?
1. Looking at a diff of a file which has long lines

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

The diff viewer shows the old and new file side by side but on the new file  
part the lines get cut off if they are too long for the browser window (see  
attached screenshot). On Chrome and IE the lines get wrapped. On Firefox I  
would expect the same or having a horizontal scrollbar to appear would  
suffice as well



What operating system are you using? What browser?
Windows 8.1, FF 29.0.1, works ok on Chrome 35.0 and IE11


Please provide any additional information below.


Attachments:
rb_ui_bug_2.png  20.0 KB

--
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 3350 in reviewboard: No support for rbt status for non-group submitters

2014-05-25 Thread reviewboard

Updates:
Status: Fixed

Comment #2 on issue 3350 by trowb...@gmail.com: No support for rbt status  
for non-group submitters

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

Fixed in rbtools git release-0.6.x (ddaa655). Thanks!

--
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 3367 in reviewboard: ImportError: No module named fcntl Djblets 0.8, Windows

2014-05-25 Thread reviewboard

Updates:
Status: Fixed

Comment #2 on issue 3367 by trowb...@gmail.com: ImportError: No module  
named fcntl Djblets 0.8, Windows

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

Fixed in djblets release-0.8.x (db2f6ce). Thanks!

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