[PATCH] dev-tools patch for gerrit script (was Re: gerrit daily )

Bjoern Michaelsen bjoern.michaelsen at canonical.com
Tue Jun 11 02:34:59 PDT 2013


Hello Mat,

On Tue, Jun 11, 2013 at 01:58:10AM +0200, Mat M wrote:
> my patch is this:
> Sanitize the age query parameters with --
> Improve layout of mail like:
> -----8<-------------------------------------------------------------------
> Moin!
> 
> * Open changes on master for project core changed in the last 25 hours:
> 
> Rework code after OUString'ification
>   in https://gerrit.libreoffice.org/4214 from Matteo Casalin
> Simplify SwPaM::GetTxt
>   in https://gerrit.libreoffice.org/4213 from Matteo Casalin
> groupdialog.ui widget
>   in https://gerrit.libreoffice.org/4208 from Manal Alhassoun
> fdo#64977 Adding Tibetan Language Support
>   in https://gerrit.libreoffice.org/4197 from Élie Roux
> ...
> ----------------------------------------------->8-------------------------
> 
> without going beyond 75 chars and with no wrap neither.
> 
> Since urls have fixed-length, I put them at start to ahve ~aligned things.
> 
> Attached is the patch, because dev-tools does not have the Change-Id
> hook (apparently).

Pushed as:

 https://gerrit.libreoffice.org/gitweb?p=dev-tools.git;a=commit;h=45f825621e3c40065e88052e916ec602d45c016d

and deployed.

As for the hook -- it needs to be setup manually as we have no canonical "make"
that does this in hiding, but then you can use gerrit on it just like any other
repo (or you can just push to refs/for/master, which will reject the change but
give you a proposal for a change-id that you can 'git commit --amend' then
quickly)

Best,

Bjoern

P.S.: As you can see from the commit lazy me forgot the hook too :/


More information about the LibreOffice mailing list