From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | Andrew Dunstan <andrew(at)dunslane(dot)net>, Andres Freund <andres(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Commit subject line |
Date: | 2013-05-06 17:12:38 |
Message-ID: | 20130506171238.GF26481@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, May 6, 2013 at 06:41:53PM +0200, Magnus Hagander wrote:
> > In practice, something else must be further truncating it, at about 64 chars
> > by the look of it - see for example
> > <http://www.postgresql.org/message-id/E1UVTFj-00079k-2r@gemulon.postgresql.org>
>
> Ha. Good point. There's actually a bit of a bug in the code there :)
> What it does is limit the length to 80-length("pgsql: $shortmsg"),
> which is 64. It is supposed to limit it to 80-length("pgsql: ")..
> (Since it substitutes the actual commit message where $shortmsg is
> found).
>
> That's fixable though :)
>
>
> > Re your other point, github at least seems to elide at about 70 chars - see
> > <https://github.com/postgres/postgres/commit/b42ea7981ce1e7484951a22662937541066d8647>
> > - where Joe used a very long first sentence rather than a show summary line.
> > I don't know if gitweb could be induced to elide after a greater length - I
> > bet it could fairly easily. There does seem to be lots of spare screen real
> > estate on the commit summary and history pages, which I think is where this
> > occurs.
>
> Possibly. I can never find my way around that one though, and making
> any modifications also has us ending up maintaining what's basically a
> fork - unless there's always a config argument for it somewhere.
So what should our goal length be?
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
From | Date | Subject | |
---|---|---|---|
Next Message | Josh Berkus | 2013-05-06 17:23:40 | Re: pg_dump versus materialized views |
Previous Message | Tom Lane | 2013-05-06 17:07:17 | Re: pg_dump --snapshot |