From owner-svn-ports-head@freebsd.org Thu Dec 7 11:23:49 2017 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 993B2E83BE4; Thu, 7 Dec 2017 11:23:49 +0000 (UTC) (envelope-from marcus@freebsd.org) Received: from aer-iport-1.cisco.com (aer-iport-1.cisco.com [173.38.203.51]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (Client CN "iport.cisco.com", Issuer "HydrantID SSL ICA G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 6DAC311FF; Thu, 7 Dec 2017 11:23:47 +0000 (UTC) (envelope-from marcus@freebsd.org) X-Files: signature.asc : 195 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0DBAwBfIyla/xbLJq1dHAEBAQQBAQoBA?= =?us-ascii?q?YQkchQThAKLFJAGmR4HAyOFGAKGGRQBAQEBAQEBAQFrKIUjAQUjVhALDgYEKgI?= =?us-ascii?q?CVwYNBgIBAQWKHhCoBoInilgBAQEBAQEBAQEBAQEBAQEBAQEBEA+DVIVJKYMCg?= =?us-ascii?q?ySCHYJ1gmMFimGIPI9khEuCK4ECjzuJdodQikCCRX2IV4E7NiKBT0wwCIJjCYR?= =?us-ascii?q?qIzeKBQEBAQ?= X-IronPort-AV: E=Sophos;i="5.45,372,1508803200"; d="asc'?scan'208";a="744408" Received: from aer-iport-nat.cisco.com (HELO aer-core-2.cisco.com) ([173.38.203.22]) by aer-iport-1.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 Dec 2017 11:22:34 +0000 Received: from [10.230.72.116] ([10.230.72.116]) by aer-core-2.cisco.com (8.14.5/8.14.5) with ESMTP id vB7BMXZL010572; Thu, 7 Dec 2017 11:22:34 GMT Subject: Re: svn commit: r447981 - in head/ports-mgmt/portlint: . src To: Pietro Cerutti Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org References: <201708151243.v7FChEAF043279@repo.freebsd.org> <20171207080421.qijzbigqkbfbezqe@ptrcrt.ch> <20171207081055.xtssqrp5okethfeu@ptrcrt.ch> <20171207081308.5bxt5mp2o5wr33un@ptrcrt.ch> From: Joe Marcus Clarke Organization: FreeBSD, Inc. Message-ID: Date: Thu, 7 Dec 2017 06:22:30 -0500 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.13; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: <20171207081308.5bxt5mp2o5wr33un@ptrcrt.ch> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="C7ju7B9pBgpTPoVJn7h7RGsm8xCSPJaUL" X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 07 Dec 2017 11:23:49 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --C7ju7B9pBgpTPoVJn7h7RGsm8xCSPJaUL Content-Type: multipart/mixed; boundary="msHMxnKmIxRLjbM9S1LGAfLKof52PGS6S"; protected-headers="v1" From: Joe Marcus Clarke To: Pietro Cerutti Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Message-ID: Subject: Re: svn commit: r447981 - in head/ports-mgmt/portlint: . src References: <201708151243.v7FChEAF043279@repo.freebsd.org> <20171207080421.qijzbigqkbfbezqe@ptrcrt.ch> <20171207081055.xtssqrp5okethfeu@ptrcrt.ch> <20171207081308.5bxt5mp2o5wr33un@ptrcrt.ch> In-Reply-To: <20171207081308.5bxt5mp2o5wr33un@ptrcrt.ch> --msHMxnKmIxRLjbM9S1LGAfLKof52PGS6S Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 12/7/17 03:13, Pietro Cerutti wrote: > On Dec 7 17 08:10, Pietro Cerutti wrote: >> On Dec 7 17 08:04, Pietro Cerutti wrote: >>> On Aug 15 17 12:43, Joe Marcus Clarke wrote: >>>> Author: marcus >>>> Date: Tue Aug 15 12:43:14 2017 >>>> New Revision: 447981 >>>> URL: https://svnweb.freebsd.org/changeset/ports/447981 >>>> >>>> Log: >>>> Update to 2.17.12. >>> >>>> @@ -3400,6 +3437,11 @@ sub abspathname { >>>> =20 >>>> if ($pre =3D~ /MASTER_SITE_SUBDIR/) { >>>> # MASTER_SITE_SUBDIR lines are ok. >>>> + $i =3D ''; >>>> + } >>>> + if ($s =3D~ /\$\{[^}]*?$i/) { >>>> + # If we're inside a make variable, we probably do not have >>>> + # an absolute path. >>>> $i =3D ''; >>>> } >>>> } >>>> >>> >>> Unmatched ) in regex; marked by <-- HERE in m/\$\{[^}]*?/dev/null 2>&= 1 )=20 >>> <-- HERE / at /usr/local/bin/portlint line 3443. >>> >>> I think the ? doesn't belong there. >> >> No, my analysis was wrong. It's about the contents of the $i variable.= =20 >> Here's why. You can reproduce by portlint -AC in net-im/openfire. >> >> i is /var/log/openfire >> WARN: Makefile: possible use of absolute pathname "/var/log/openfire".= >> >> i is /var/db/openfire >> WARN: Makefile: possible use of absolute pathname "/var/db/openfire". >> >> i is /dev/null 2>&1 ) >> Unmatched ) in regex; marked by <-- HERE in m/\$\{[^}]*?/dev/null 2>&1= )=20 >> <-- HERE / at /usr/local/bin/portlint line 3444. >=20 > Surrounding $i with \Q and \E in the regex solves the problem. >=20 >=20 Thanks. I'll commit a quick fix ahead of the next release. Joe --=20 Joe Marcus Clarke FreeBSD GNOME Team :: gnome@FreeBSD.org FreeNode / #freebsd-gnome http://www.FreeBSD.org/gnome --msHMxnKmIxRLjbM9S1LGAfLKof52PGS6S-- --C7ju7B9pBgpTPoVJn7h7RGsm8xCSPJaUL Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EARECAB0WIQTMiWQHc8wChijkr7lvaI+K/hTPhwUCWikkeQAKCRBvaI+K/hTP h18AAJoCx0ZbOcfLAlkE+IH0+2OHI9fAXwCgkja36MkgVtNEQ5JPIoCYvm/wyOo= =Gqer -----END PGP SIGNATURE----- --C7ju7B9pBgpTPoVJn7h7RGsm8xCSPJaUL--