Hm, seemed the problem was caused by the syncing.
I've fixed it using a simple git pre-commit hook:
> #!/bin/sh
> if git rev-parse --verify HEAD >/dev/null 2>&1
> then
> ssh user@server "cd /repo; git pull &>/dev/null"
> fi
Thanks for the advice!
Tom De Vylder
On 18 Mar 2011, at 09:39, C
It might be worth trying to resolve the commit
(c8d50071517379635faf795e858fb3d52a190c19) on both your own clone and on the
one used by Review Board, just to make sure they're at least in sync.
Christian
--
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware,
That's right. It's a local git clone, RB has access to it.
It's in sync and the change should be pushed using a post-commit on my local
working copy.
Tom De Vylder
On 18 Mar 2011, at 08:44, Christian Hammond wrote:
> Ah, are you using a local Git clone that Review Board has access to? Is it
>
Ah, are you using a local Git clone that Review Board has access to? Is it
constantly in sync, and is that particular change accessible on it?
Christian
--
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com
On Thu, Mar 17, 201
Hi Christian,
Still no fish.
I've tried the diff --full-index, but then I receive this error:
> >>> Uploading diff, size: 3045
> >>> HTTP POSTing to
> >>> http://review.dev.inuits.be/api/review-requests/35/diffs/: {}
> >>> Got API Error 105 (HTTP code 400): One or more fields had errors
> >>> Er
Hi Tom,
Most likely, it's the short SHA1. If you generate the diff with
--full-index, you should be able to get a full SHA1, which may fix your
problem.
Christian
--
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com
On Wed,
Hi everyone,
I've been struggling trying to get post-review to post a patch to the web
interface.
Each time I try to submit a patch post-review complains that it can't find a
file
The file specified with --diff-filename is uploaded each time.
So my guess is that it can't find the file that need