From owner-freebsd-ports@FreeBSD.ORG  Fri Feb 18 18:58:22 2011
Return-Path: <owner-freebsd-ports@FreeBSD.ORG>
Delivered-To: ports@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 3B83B106564A
	for <ports@freebsd.org>; Fri, 18 Feb 2011 18:58:22 +0000 (UTC)
	(envelope-from wxs@atarininja.org)
Received: from syn.atarininja.org (syn.csh.rit.edu [129.21.49.45])
	by mx1.freebsd.org (Postfix) with ESMTP id 158378FC13
	for <ports@freebsd.org>; Fri, 18 Feb 2011 18:58:21 +0000 (UTC)
Received: by syn.atarininja.org (Postfix, from userid 1001)
	id 01A2B5C39; Fri, 18 Feb 2011 13:59:09 -0500 (EST)
Date: Fri, 18 Feb 2011 13:59:08 -0500
From: Wesley Shields <wxs@FreeBSD.org>
To: Dean Freeman <wfreeman@sourcefire.com>
Message-ID: <20110218185908.GB36275@atarininja.org>
References: <AANLkTinpFQ9PVPp9UYY7kj4EPCrL8EJx3MXVAmhUX6af@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <AANLkTinpFQ9PVPp9UYY7kj4EPCrL8EJx3MXVAmhUX6af@mail.gmail.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
Cc: ports@freebsd.org
Subject: Re: port revision for Snort
X-BeenThere: freebsd-ports@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Porting software to FreeBSD <freebsd-ports.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-ports>,
	<mailto:freebsd-ports-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-ports>
List-Post: <mailto:freebsd-ports@freebsd.org>
List-Help: <mailto:freebsd-ports-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-ports>,
	<mailto:freebsd-ports-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 18 Feb 2011 18:58:22 -0000

On Fri, Feb 18, 2011 at 01:05:56PM -0500, Dean Freeman wrote:
> I've submitted a patch to fix two issues, including a potential segfault in
> HttpInspect.  This change will bump the port revision to 2.

I'll commit this.

Two quick things for the next time you submit a PR though.

1. The patch is reversed.
2. You probably don't want to have all the unnecessary stuff in the
patches to the port (? cflags.out, etc).

I'll try and get this in the tree as quickly as I can.

-- WXS