On Wed, Sep 19, 2012 at 8:27 PM, Spencer Oliver <s...@spen-soft.co.uk> wrote:
> On 19/09/12 18:36, Austin, Alex wrote:
>> What output does the following command produce?
>>
>> $ git config remote.review.push
>>
>> The ref you're trying to push to looks a little odd.
>> I think it should be pushing to refs/for/master.
>>
>
> either is valid - refs/for has been deprecated in favour of
> refs/publish/ in newer versions of git.

Newer versions of gerrit, not git.

And I think the order of the Signed-off-by and the Change-Id is not so
important. However the Change-Id must be in the last paragraph and [1]
indicates that the same applies to the Signed-off-by. It looks like
there's a newline after it in the OP's commit message.

/Andreas

[1] http://code.google.com/p/gerrit/issues/detail?id=606

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to