From owner-freebsd-ports-bugs@FreeBSD.ORG Thu Oct 4 19:40:03 2012 Return-Path: Delivered-To: freebsd-ports-bugs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id CDDE01065670 for ; Thu, 4 Oct 2012 19:40:03 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 99EA98FC0C for ; Thu, 4 Oct 2012 19:40:03 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.5/8.14.5) with ESMTP id q94Je3uo008181 for ; Thu, 4 Oct 2012 19:40:03 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.5/8.14.5/Submit) id q94Je3lj008180; Thu, 4 Oct 2012 19:40:03 GMT (envelope-from gnats) Resent-Date: Thu, 4 Oct 2012 19:40:03 GMT Resent-Message-Id: <201210041940.q94Je3lj008180@freefall.freebsd.org> Resent-From: FreeBSD-gnats-submit@FreeBSD.org (GNATS Filer) Resent-To: freebsd-ports-bugs@FreeBSD.org Resent-Reply-To: FreeBSD-gnats-submit@FreeBSD.org, Greg Larkin Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 0DC51106566B; Thu, 4 Oct 2012 19:36:59 +0000 (UTC) (envelope-from jlr@sourcehosting.net) Received: from mail1.sourcehosting.net (mail1.sourcehosting.net [74.205.51.45]) by mx1.freebsd.org (Postfix) with ESMTP id D51AC8FC0C; Thu, 4 Oct 2012 19:36:58 +0000 (UTC) Received: from 24-181-237-39.dhcp.oxfr.ma.charter.com ([24.181.237.39] helo=portsdev.shn.prv) by mail1.sourcehosting.net with esmtps (TLSv1:AES256-SHA:256) (Exim 4.73 (FreeBSD)) (envelope-from ) id 1TJrE7-00035D-1h; Thu, 04 Oct 2012 15:36:57 -0400 Received: from portsdev.shn.prv (portsdev.shn.prv [172.16.93.9]) by portsdev.shn.prv (8.14.4/8.14.4) with ESMTP id q94Jara6086604; Thu, 4 Oct 2012 15:36:53 -0400 (EDT) (envelope-from root@portsdev.shn.prv) Received: (from root@localhost) by portsdev.shn.prv (8.14.4/8.14.4/Submit) id q94JZhRP086127; Thu, 4 Oct 2012 15:35:43 -0400 (EDT) (envelope-from root) Message-Id: <201210041935.q94JZhRP086127@portsdev.shn.prv> Date: Thu, 4 Oct 2012 15:35:43 -0400 (EDT) From: Greg Larkin To: FreeBSD-gnats-submit@FreeBSD.org X-Send-Pr-Version: 3.113 Cc: marcus@FreeBSD.org Subject: ports/172335: [PATCH] ports-mgmt/portlint: Remove spurious warning about *_CPPFLAGS/*_LDFLAGS variables X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 04 Oct 2012 19:40:03 -0000 >Number: 172335 >Category: ports >Synopsis: [PATCH] ports-mgmt/portlint: Remove spurious warning about *_CPPFLAGS/*_LDFLAGS variables >Confidential: no >Severity: non-critical >Priority: low >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: change-request >Submitter-Id: current-users >Arrival-Date: Thu Oct 04 19:40:03 UTC 2012 >Closed-Date: >Last-Modified: >Originator: Greg Larkin >Release: FreeBSD 8.3-RELEASE-p3 amd64 >Organization: The FreeBSD Project >Environment: System: FreeBSD portsdev.shn.prv 8.3-RELEASE-p3 FreeBSD 8.3-RELEASE-p3 #0: Tue Jun 12 00:39:29 UTC 2012 >Description: portlint tests if the port Makefile adds CPPFLAGS and LDFLAGS to the CONFIGURE_ENV variable and warns against it since it's already done in bsd.port.mk. The regexp used in the test also matches variables such as FOO_CPPFLAGS and FOO_LDFLAGS, as in this example from devel/gearmand/Makefile: CONFIGURE_ENV+= LIBTOKYOCABINET_CPPFLAGS=-I${LOCALBASE}/include \ LIBTOKYOCABINET_LDFLAGS=-L${LOCALBASE}/lib The patch adds a word boundary check to the beginning of each regexp to avoid warning about constructs like the one above. Port maintainer (marcus@FreeBSD.org) is cc'd. Generated with FreeBSD Port Tools 0.99_6 (mode: change, diff: ports) >How-To-Repeat: Run "portlint -C" in /usr/ports/devel/gearmand >Fix: --- portlint-2.13.12.patch begins here --- --- /usr/ports/ports-mgmt/portlint/src/portlint.pl 2012-08-05 18:56:23.000000000 -0400 +++ ./src/portlint.pl 2012-10-04 15:27:26.000000000 -0400 @@ -16,7 +16,7 @@ # This code now mainly supports FreeBSD, but patches to update support for # OpenBSD and NetBSD will be accepted. # # $FreeBSD: ports/ports-mgmt/portlint/src/portlint.pl,v 1.130 2012/08/05 22:56:23 marcus Exp $ # $MCom: portlint/portlint.pl,v 1.252 2012/08/05 22:18:57 marcus Exp $ # @@ -2186,14 +2186,14 @@ "instead with $1=..."); } - if ($configure_env =~ /(CPPFLAGS)=/) { + if ($configure_env =~ /(\bCPPFLAGS)=/) { &perror("FATAL", $file, -1, "$1 is already ". "passed in CONFIGURE_ENV via bsd.port.mk. If you need to ". "override the default value, alter $1 in the Makefile ". "instead with $1+=..."); } - if ($configure_env =~ /(LDFLAGS)=/) { + if ($configure_env =~ /(\bLDFLAGS)=/) { &perror("FATAL", $file, -1, "$1 is already passed in ". "CONFIGURE_ENV via bsd.port.mk. If you need to ". "override the default value, alter $1 in the Makefile ". --- portlint-2.13.12.patch ends here --- >Release-Note: >Audit-Trail: >Unformatted: