DragonFly commits List (threaded) for 2008-12
DragonFly BSD
DragonFly commits List (threaded) for 2008-12
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

Re: commit mail subject format

From: "Simon 'corecode' Schubert" <corecode@xxxxxxxxxxxx>
Date: Wed, 03 Dec 2008 14:13:23 +0100

Aggelos Economopoulos wrote:
Simon 'corecode' Schubert wrote:
For anybody confused about the mail subject, let me explain the thing:

DragonFly- master test/test README
^^^^^^^^^ ^^^^^ ^ ^^^^^^ ^^^^^^ ^^^^^^^^^ ^^^^^^
|           |   |  |     ^branch  ^directory  ^file in directory
|           |   |  abbreviated commit id
that's us   |   topological distance to the release, here 5 commits
          last "release"

The string "" is actually the output of git-describe and
probably will be used in more places in the future (kernel version, ISO
build rev, etc)

Why do we need all this stuff in the message Subject? The "DragonFly-" part is essentially wasted space (gee, like I need to be reminded which folder I'm reading), space that could be used for something useful. Also, if I want any of the rest of the info, all I need is the commit id and I can use git to get it. Maaaybe having the commit id appended (so it won't intrude too much) to the Subject would be useful so you can speedily search for any discussions relevant to a commit, but that's also debatable.

Why can't we use the commit summary or just the branch, directory and
files changed? The whole idea behind the Subject: line is that you can
tell at a glance if you're interested in reading the mail body and I
don't see how this format accomplishes it... All the extra info could go
in the message body if people still want it to be instantly available.
Personally, I only care about the commitid and diffstat, but that's just me.

Another idea would be to stuff the extra info in the headers so that
each subscriber can select which fields to view, but I'd prefer a sane
format that more or less works for everybody.


Exactly my thoughts. Matt insisted on the DragonFly and the commit id-at-the-beginning for search/subject alignment reasons.


PS: Don't hate the player, hate the game.

  <3 the future  +++  RENT this banner advert  +++   ASCII Ribbon   /"\
  rock the past  +++  space for low €€€ NOW!1  +++     Campaign     \ /
Party Enjoy Relax   |   http://dragonflybsd.org      Against  HTML   \
Dude 2c 2 the max   !   http://golden-apple.biz       Mail + News   / \

[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]