Date: prev next · Thread: first prev next last
2018 Archives by date, by thread · List index


Hi Miklos,

On Wed, 17 Oct 2018 at 09:26:40 +0200, Miklos Vajna wrote:
On Wed, Oct 17, 2018 at 04:27:54AM +0200, Guilhem Moulin <guilhem@libreoffice.org> wrote:
Right now https://gerrit.libreoffice.org/#/c/$CHANGENUM/ has both gitweb
and Gitiles links, but unless we hear strong objection we'll disable
gitweb at the end of the month, and serve requests to
‘/gitweb?p=${REPOSITORY}.git;a=${ACTION};h=${REF}’ as 302-redirections
to the corresponding Gitiles paths.

The gitweb link takes me to a diff of the whole change, while in the
gitiles case I need one more additional click to the "diff" text to get
there. Would it be possible to keep showing the diff by default?

Ah, yeah.  There is even a custom setting in gerrit.config's “[gitweb]” section
in that effect: “revision = "?p=${project}.git;a=commitdiff;h=${commit}"” :-)

Done on our gerrit stage instance (https://vm178.documentfoundation.org/); will
do the same thing on the prod instance next time we have to restart the service.

A second point: gitweb supports searching by name, e.g.
<https://gerrit.libreoffice.org/gitweb?p=core.git&a=search&st=author&s=caolan>.
Does gitiles support something like this, or from now on one must clone
the repo to be able to do queries like this?

It does indeed, but it's a bit hidden :-P

    https://gerrit.libreoffice.org/plugins/gitiles/core/+log/?author=caolan

How attached are you to the search form?  We could definitely add one to
the template, but for now I only changed it (on the stage instance) to
make authors & committers clickable on the relevant views, and filter on
their email:

    commit        https://vm178.documentfoundation.org/plugins/gitiles/help/+/master
    commitdiff    https://vm178.documentfoundation.org/plugins/gitiles/help/+/master%5E%21
    log (oneline) https://vm178.documentfoundation.org/plugins/gitiles/help/+log/master/
    log (full)    https://vm178.documentfoundation.org/plugins/gitiles/help/+log/master/?pretty=full

Cheers,
-- 
Guilhem.

Attachment: signature.asc
Description: PGP signature


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.