From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17316 invoked by alias); 25 Oct 2013 01:56:12 -0000 Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org Received: (qmail 17176 invoked by uid 89); 25 Oct 2013 01:56:10 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,WEIRD_PORT autolearn=ham version=3.3.2 X-HELO: rock.gnat.com Received: from rock.gnat.com (HELO rock.gnat.com) (205.232.38.15) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Fri, 25 Oct 2013 01:56:09 +0000 Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id DEC441168A9; Thu, 24 Oct 2013 21:56:32 -0400 (EDT) Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id t7p1wRmG5TbY; Thu, 24 Oct 2013 21:56:32 -0400 (EDT) Received: from joel.gnat.com (localhost.localdomain [127.0.0.1]) by rock.gnat.com (Postfix) with ESMTP id 7B5241168A5; Thu, 24 Oct 2013 21:56:32 -0400 (EDT) Received: by joel.gnat.com (Postfix, from userid 1000) id ADDC9E10A4; Fri, 25 Oct 2013 05:56:03 +0400 (RET) Date: Fri, 25 Oct 2013 01:56:00 -0000 From: Joel Brobecker To: asmwarrior Cc: gdb@sourceware.org Subject: Re: suggestion: the title of mails in the cvs mailllist should show the changed location instead a hash value Message-ID: <20131025015603.GK4688@adacore.com> References: <5269B846.3090301@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <5269B846.3090301@gmail.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-SW-Source: 2013-10/txt/msg00136.txt.bz2 > Hi, after the git migration, I see that the email title changed in the cvs maillist like: > > gdb and binutils branch master updated. 5c2955e2ec5ca2e80098763d6fdda317dc4b2f39 > gdb and binutils branch master updated. 7283b932234f3501eb19c1750d79c20fc339f32a > ... > > > Compared to the original name before git migration: > src/gdb ChangeLog infrun.c testsuite/ChangeLog ... > src/gdb ChangeLog c-varobj.c > > > I think the old style email title is still better than the current one, it at least showed some changed files/locations, so a reader(like me, I use the news://www.gmane.org:119/gmane.comp.gdb.cvs to receive the updated emails) can briefly see where the commit modifies by reading the title. With the current name scheme, you know nothing about this. > > Any ideas? True, but eventually, I think an even better approach is to have one email per commit, with the commit's subject as the email subject. It's something I have implemented at AdaCore for our internal git repositories, and works pretty well. I'd be happy to help improve the hooks, but it'll have to be sometime next year. In the meantime, I think I see in the script where we could change the subject to add the list of files being changed. It should be fairly easy to do so, but we'd need to agree on the new format. For instance: '[' project ['/' branch] ']' updated-files Eg: [gdb] gdb/infrun.c gdb/ChangeLog <-- update of the master branch [gdb/gdb-7.7] gdb/NEWS gdb/ChangeLog <-- update of the gdb-7.7 branch For pure binutils changes, you'd get the same, but replacing "gdb" by "binutils". And for changes spanning both projects, you'd get: [gdb+binutils] bfd/configure.host bfd/ChangeLog gdb/ChangeLog [gdb+binutils/gdb-7.7] bfd/configure.host bfd/ChangeLog gdb/ChangeLog -- Joel