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): > > http://qt.gitorious.org/qtqa/gerrit/commit/737400d1bad4fa8bfd39cb326636a

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 http://lists

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 ve

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 con