Re: Mail thread references in commits

From: Heikki Linnakangas <hlinnaka(at)iki(dot)fi>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Stephen Frost <sfrost(at)snowman(dot)net>, Magnus Hagander <magnus(at)hagander(dot)net>, Joshua Drake <jd(at)commandprompt(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com>
Subject: Re: Mail thread references in commits
Date: 2016-11-21 09:58:18
Message-ID: 7c19b141-66d1-8e17-fa68-a049db548acd@iki.fi
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 11/19/2016 06:38 PM, Tom Lane wrote:
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
>> Every proposal along these lines strikes me as making more unobvious
>> hoops for people to jump through. We're just reinventing the wheel here
>> because we don't like the color of the standard wheels that some email
>> MUAs produce.
>
> Yeah, anything like this would be going very very far out of our way to
> keep these lines in commit messages under an arbitrary limit (which we're
> already exceeding anyway in many cases). I'm inclined to think we should
> just straight up decide that having the message link be a clickable URL is
> worth the longer lines, or that it is not. Personally I vote for "not",
> but I recognize that I might be in the minority.

Agreed. I'm happy with the status quo. But if we want to make them
clickable, let's just bite the bullet and start doing:

Discussion: https://www.postgresql.org/message-id/<message-id>

The point of keeping the lines short is to make the text in the commit
more pleasant to read, but that hardly applies to a URL.

> BTW, before we give up completely on reconciling the two objectives,
> is anyone aware of a line-continuation convention that would be likely
> to work? If we could do something like
>
> Discussion: https://www.postgresql.org/message-id/\
> CAF4Au4w6rrH_j1bvVhzpOsRiHCog7sGJ3LSX0tY8ZdwhHT88LQ(at)mail(dot)gmail(dot)com
>
> it would improve matters all around.

That would make it more difficult to parse programmatically, as you
couldn't just grep lines beginning with "Discussion: ".

Unless we decide to switch to using clickable URLs here, and perhaps
even if we do, because we already have a lot of current-format
"Discussion: " lines in the git history, it would be nice to change the
message archive web interface to know about "Discussion: ". The web
interface could view those as clickable links. At the moment, not only
are they not clickable, but the interface even obfuscates them, thinking
that the message-ids are email addresses, making copy-pasting harder too:

Discussion: <26681(dot)1477940227(at)sss(dot)pgh(dot)pa(dot)us>

- Heikki

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Magnus Hagander 2016-11-21 10:04:05 Re: Mail thread references in commits
Previous Message Oleksandr Shulgin 2016-11-21 09:55:24 Re: Danger of automatic connection reset in psql