Re: Patching our gerrit instance [was Re: [ANN] Please use Gerrit from now on for Patch Review]

2012-07-19 Thread Lionel Elie Mamane
On Wed, Jul 18, 2012 at 10:10:38PM +0200, David Ostrovsky wrote: https://gerrit.libreoffice.org/#/c/328/ The gitweb link on that page says 404 no such project. -- Lionel ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org

Re: Patching our gerrit instance [was Re: [ANN] Please use Gerrit from now on for Patch Review]

2012-07-19 Thread Bjoern Michaelsen
Hi, On Wed, Jul 18, 2012 at 07:11:53PM +0200, Jan Holesovsky wrote: Luckily, the Qt guys apparently had the same problem, and have a solution (the entire diff on one page, without abandoning the inline commenting):

Re: Patching our gerrit instance [was Re: [ANN] Please use Gerrit from now on for Patch Review]

2012-07-18 Thread David Ostrovsky
Hi Kendy, On 18.07.2012 19:11, Jan Holesovsky wrote: Another thing are the mail templates - can you please commit the current mail templates that we are using on gerrit.libreoffice.org to dev-tools/gerrit/gerrit_site/etc/mail, so that we can tweak them [eg. to get rid of the . line], in a

Patching our gerrit instance [was Re: [ANN] Please use Gerrit from now on for Patch Review]

2012-07-18 Thread Jan Holesovsky
Hi Bjoern, On 2012-07-02 at 18:44 +0200, Bjoern Michaelsen wrote: When I click Diff All Side-by-Side (or Diff All Unified), it shows me only the first file with a link to the (diff) of the next file. I expected to see the diff for *all* files on one page. That is much more convenient to