1302: post-review on Solaris?

birb****@gmai***** (Google Code) (Is this you? Claim this profile.)
Sept. 9, 2009 
What version are you running?
reviewboard 1.0.1
post-review 0.8

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


What steps will reproduce the problem?
1.Install post-review on solaris via asy_install -U RBtools
2.Install the p4 counter to use the reviewboard server 
3.post-review -d -n 45

What is the expected output? What do you see instead?
The expected output should commit the change for review to review board.
Instead, the output is as follows:
>>> svn info
>>> hg root
>>> p4 info
>>> repository info: Path: perforce.test:1666, Base path: None, Supports
changesets: True
>>> p4 counters
>>> Generating diff for changenum 45
>>> p4 describe -s 45
>>> Processing add of //test/test
>>> Writing "//test/test" to "/tmp/tmpHOJ5aN"
>>> p4 print -q //test/test
>>> diff -urNp /tmp/tmpCca0S0 /tmp/tmpHOJ5aN
Unable to parse diff header: usage: diff [-bitw] [-c | -e | -f | -h | -n |
-u] file1 file2


What operating system are you using? What browser?
Solaris 10 U7

Please provide any additional information below.
chipx86
#1 chipx86 
This should be fixed in the nightlies. Please try the latest RBTools by doing:

easy_install -f http://downloads.review-board.org/nightlies/ -U RBTools
  • +NeedInfo
#2 birb****@gmai***** (Google Code) (Is this you? Claim this profile.) 
Thanks chipx86,
I grabbed the latest nightly using your instructions and I now see this line:
>>> gdiff -urNp /tmp/tmpwSUY0m /tmp/tmpABpFZ8

So I also need to have the gdiff package installed, right?
chipx86
#3 chipx86 
Right. We pretty specifically need GNU Diff's diff format and options (these are
subtley different between diff implementations).
#4 birb****@gmai***** (Google Code) (Is this you? Claim this profile.) 
That's clear.
One final thing. I'll be in the need to install post-review on multiple solaris
boxes. Is there any safer way to do this for now (so that I won't be using the
nighlty stable tarball)?
chipx86
#5 chipx86 
We'll be doing a release soon with these fixes. It's been delayed due to family
emergencies, but I'm hoping to get to it in a week or two. Basically, what's in the
nightly is more or less what will be in the release. There will be a few more
improvements, but mainly to Git support. I don't think svn will be touched.
  • -NeedInfo
    +Fixed
  • +Milestone-RBTools-Release1.0
    +Component-RBTools
    +OpSys-Solaris
#6 birb****@gmai***** (Google Code) (Is this you? Claim this profile.) 
Cool, thanks :)