From owner-cvs-all@FreeBSD.ORG  Thu Jul 10 04:26:47 2008
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 8F807106567B;
	Thu, 10 Jul 2008 04:26:47 +0000 (UTC)
	(envelope-from jdc@parodius.com)
Received: from mx01.sc1.parodius.com (mx01.sc1.parodius.com [72.20.106.3])
	by mx1.freebsd.org (Postfix) with ESMTP id 7465B8FC28;
	Thu, 10 Jul 2008 04:26:47 +0000 (UTC)
	(envelope-from jdc@parodius.com)
Received: by mx01.sc1.parodius.com (Postfix, from userid 1000)
	id 536601CC077; Wed,  9 Jul 2008 21:16:17 -0700 (PDT)
Date: Wed, 9 Jul 2008 21:16:17 -0700
From: Jeremy Chadwick <koitsu@FreeBSD.org>
To: "Philip M. Gollucci" <pgollucci@p6m7g8.com>
Message-ID: <20080710041617.GA34651@eos.sc1.parodius.com>
References: <200807092308.m69N8DUm089133@repoman.freebsd.org>
	<48756EE4.2090206@p6m7g8.com> <20080710020932.GA60785@k7.mavetju>
	<487570DA.7080005@p6m7g8.com>
	<20080710035109.GY62764@server.vk2pj.dyndns.org>
	<487588CC.2040708@p6m7g8.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <487588CC.2040708@p6m7g8.com>
User-Agent: Mutt/1.5.18 (2008-05-17)
Cc: cvs-all@freebsd.org, ports-committers@freebsd.org,
	Edwin Groothuis <edwin@mavetju.org>, cvs-ports@freebsd.org,
	Edwin Groothuis <edwin@freebsd.org>
Subject: Re: cvs commit: CVSROOT modules
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 10 Jul 2008 04:26:47 -0000

On Wed, Jul 09, 2008 at 11:58:04PM -0400, Philip M. Gollucci wrote:
> Peter Jeremy wrote:
>> On 2008-Jul-09 22:15:54 -0400, "Philip M. Gollucci" <pgollucci@p6m7g8.com> wrote:
>>> For example a recent cvs-ports mail process with my ~/.procmailrc 
>>> which invokes cvsmail.  Your commits to CVSROOT/modules contain no 
>>> such diff in the e-mail.
>>
>> The original mail sent from FreeBSD.org does not contain any patches.
>> The actual commit message looks normal so the problem would appear
>> to be that your local configuration is not correctly parsing the
>> commit message.  One way in which it is different to the update you
>> reported is that it relates to a CVSROOT file - possibly you are not
>> correctly handling CVSROOT commits.
>>
> Anything wrong with this ?
>
> :0
> * ^Sender:.owner-cvs-\/[^@]+@FreeBSD.ORG
> {
>     LISTNAME=${MATCH}
>     :0
>     * LISTNAME??^\/[^@]+
>     | /usr/local/bin/cvsmail -d -c -g -f freebsd.org/cvs-${MATCH}
> }
>
> :0
> * ^Sender:.owner-freebsd-\/[^@]+@FreeBSD.ORG
> {
>     LISTNAME=${MATCH}
>     :0
>     * LISTNAME??^\/[^@]+
>     freebsd.org/${MATCH}
> }

The lack of file locking on your delivery rules is a bit disheartening
for starters (unless you use Maildir, which I don't believe you do,
since there's no trailing "/" on your delivery boxes).

The rule I've used for quite some time, at least for lists:

:0
* ^List-Id:.*<\/[^.]+\.freebsd.org>
{
  LISTNAME=${MATCH}

  :0:
  * LISTNAME ?? ^\/[^.]+
  freebsd/${MATCH}
}

-- 
| Jeremy Chadwick                                jdc at parodius.com |
| Parodius Networking                       http://www.parodius.com/ |
| UNIX Systems Administrator                  Mountain View, CA, USA |
| Making life hard for others since 1977.              PGP: 4BD6C0CB |