[Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Spencer Oliver
Hi, We are now testing Gerrit for use within OpenOCD. A Gerrit server has been setup at the following url: http://openocd.zylin.com/ To keep loading down on the server clone as usual: git clone git://openocd.git.sourceforge.net/gitroot/openocd/openocd for example The change happens we want to su

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Spencer Oliver
On 11 October 2011 13:40, Spencer Oliver wrote: > Hi, > > We are now testing Gerrit for use within OpenOCD. > A Gerrit server has been setup at the following url: > http://openocd.zylin.com/ > > To keep loading down on the server clone as usual: > git clone git://openocd.git.sourceforge.net/gitroo

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Peter Stuge
Spencer Oliver wrote: > One addition i forgot to mention. > Gerrit uses a Change-Id to track the change, thsi is generated client > side using a hook. > > You will need to install this hook, we will look into a better solution > scp -p -P 29418 usern...@openocd.zylin.com:hooks/commit-msg .git/hook

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Peter Stuge
Spencer Oliver wrote: > It is advised to keep each patch in its own branch. I'll try to explain this a little. If multiple commits are pushed, ie. if the branch being pushed has more than one commit which is not in the main openocd.git repo, then Gerrit will record that these commits form a chain

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Peter Stuge
Spencer Oliver wrote: > git remote add review ssh://usern...@openocd.zylin.com:29418/openocd.git > git config remote.review.push HEAD:refs/for/master When pushing, run: git push review Thanks to the remote.review.push setting this will push whatever is currently checked out. As always, use git

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Laurent Gauch
Hi Spen, Why using a commercial server and a commercial url for an open source project? Regards, Laurent ___ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Peter Stuge
Laurent Gauch wrote: > Why using a commercial server and a commercial url for an open > source project? What is "a commercial server"? A company has donated resources to the project for running Gerrit. I don't think the URL that is used really matters. Do you? Really? Perhaps you can focus on t

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Tormod Volden
On Tue, Oct 11, 2011 at 4:14 PM, Laurent Gauch wrote: > Why using a commercial server and a commercial url for an open source > project? heh, like sourceforge.net, code.google.com, github.com or launchpad.net? So that the open-source crowd can spend their valuable time on producing free and open-

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Øyvind Harboe
On Tue, Oct 11, 2011 at 5:59 PM, Tormod Volden wrote: > On Tue, Oct 11, 2011 at 4:14 PM, Laurent Gauch wrote: >> Why using a commercial server and a commercial url for an open source >> project? > > heh, like sourceforge.net, code.google.com, github.com or launchpad.net? > > So that the open-sourc

[Openocd-development] Documentation for gerrit usage

2011-10-11 Thread Øyvind Harboe
If someone wants to contribute a patch with documentation on how to use gerrit, that would be most welcome! -- Øyvind Harboe - Can Zylin Consulting help on your project? US toll free 1-866-980-3434 http://www.zylin.com/ ___ Openocd-development mailing l

[Openocd-development] how to push patch series to gerrit?

2011-10-11 Thread Øyvind Harboe
How do I push a patch series to gerrit? Does gerrit have a concept of patch series or are all patches islands? -- Øyvind Harboe - Can Zylin Consulting help on your project? US toll free 1-866-980-3434 http://www.zylin.com/ ___ Openocd-development maili

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Jason
On Tue, Oct 11, 2011 at 01:40:28PM +0100, Spencer Oliver wrote: > We are now testing Gerrit for use within OpenOCD. Just a comment from a lurker. Is there any intent to integrate gerrit back into the ML? See [1] for an existing gripe. The main difference I see between gerrit and a mailinglist i

Re: [Openocd-development] how to push patch series to gerrit?

2011-10-11 Thread Peter Stuge
Øyvind Harboe wrote: > How do I push a patch series to gerrit? Put the commits in a branch rooted in the main openocd.git repo master branch, then push that branch into refs/for/master in gerrit. (With Spencer's default config you will push it if it is currently checked out and you run git push re

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Øyvind Harboe
Hi Jason, I do share your concern that we're loosing something when we're not using the mailing list to discuss patches. However, I certainly still welcome discussion of patches in the mailing list prior to pushing them to Gerrit. In fact I think that if the entire discussion was in the mailing l

Re: [Openocd-development] how to push patch series to gerrit?

2011-10-11 Thread Øyvind Harboe
>> Does gerrit have a concept of patch series or are all patches >> islands? > > If you push a series of commits to gerrit as per above then gerrit > will record dependency between the commits. I did git am -s on two patches and tried using Spencer's config: git push review Counting objects: 16

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Spencer Oliver
On 11/10/2011 20:54, Øyvind Harboe wrote: Hi Jason, I do share your concern that we're loosing something when we're not using the mailing list to discuss patches. However, I certainly still welcome discussion of patches in the mailing list prior to pushing them to Gerrit. In fact I think that i

Re: [Openocd-development] how to push patch series to gerrit?

2011-10-11 Thread Øyvind Harboe
Ah gotit. I need a unique id per patch, I thought the id had to be unique per series. I thought that patches would be displayed as series within the same patch id rather than this "depends on" scheme. -- Øyvind Harboe - Can Zylin Consulting help on your project? US toll free 1-866-980-3434 http

Re: [Openocd-development] OpenOCD Gerrit Review

2011-10-11 Thread Alain Mouette
+1 if there were to bo no more list, I will never more know what happens... I simply don have the time to browse through Forums seekng out what might have been going on :) :) Alain Em 11-10-2011 14:21, Jason escreveu: On Tue, Oct 11, 2011 at 01:40:28PM +0100, Spencer Oliver wrote: We are no