On 09/25/2014 02:57 PM, Bozo Dragojevic wrote:
As a workaround, I've applied the patch to a svn checkout and uploaded
That works, but is a bit clumsy...
yum install RBTools
Then use this script:
BASEDIR=$((cd $(git rev-parse --show-toplevel) && git svn info --url) | sed
rbt post -g -d --username=aconway --repository=qpid --basedir=$BASEDIR "$@"
That works in git-svn checkouts of in qpid, proton and dispatch. I don't use
plain SVN anymore so dont know about that. To do it by hand AFAIK what you did
is the only way that works - must be svn diff in an svn checkout or reviewboard
throws up all over it.
On 25. 09. 14 20:50, Bozo Dragojevic wrote:
Can somebody help me with the reviewboard, please?
I wanted to stuff this patch to reviewboard, but it gets rejected with:
The specified diff file could not be parsed.
Line 19: No valid separator after the filename was found in the diff header
I selected repository 'qpid'
I uploaded the patch below
I selected base directory /proton/trunk (I have also tried
/qpid/proton/trunk with same effect)
fwiw, I'm doing this from a mac using chrome
On 25. 09. 14 20:44, Bozo Dragojevic (JIRA) wrote:
Bozo Dragojevic updated PROTON-660:
Attachment: (was: 0001-PROTON-660-Fix-openssl.c-build-on-windows.patch)
Fix openssl.c build on windows
Project: Qpid Proton
Issue Type: Bug
Affects Versions: 0.8
Environment: Windows 7, VS2013
Reporter: Bozo Dragojevic
Compiled openssl-1.0.1i from source
Proton finds it, but openssl.c does not compile without small adjustments.
This message was sent by Atlassian JIRA