Date: Tue, 26 Feb 2019 08:56:36 +0000 From: bugzilla-noreply@freebsd.org To: ports-bugs@FreeBSD.org Subject: [Bug 233458] net-mgmt/nagiosql: remove redundant regex escapes Message-ID: <bug-233458-7788-TDY6KOddAT@https.bugs.freebsd.org/bugzilla/> In-Reply-To: <bug-233458-7788@https.bugs.freebsd.org/bugzilla/> References: <bug-233458-7788@https.bugs.freebsd.org/bugzilla/>
next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D233458 --- Comment #1 from commit-hook@freebsd.org --- A commit references this bug: Author: tobik Date: Tue Feb 26 08:56:31 UTC 2019 New revision: 493951 URL: https://svnweb.freebsd.org/changeset/ports/493951 Log: net-mgmt/nagiosql: Remove redundant regex escapes nagiosql's Makefile currently has some redundant escaping of single quotes in its Makefile. These get passed through as escaped single quotes to regex(3), which is an escape of an ordinary character. Such escapes will be forbidden in a future version of regex(3), so we'll get ahead of the game and remove said escapes now. While here, there are two questionable expressions: -e "s|\>/etc/nagiosql|\>${ETCDIR}|g" and the one immediately following it. \> is a word delimiter (see reformat(7)), and will only match as such. If this were to match, it would replace <word delimiter>/etc/nagiosql with >${ETCDIR} -- this likely works as intended now, but I think it was intended that it matches a literal >/etc/nagiosql, so the included patch treats it accordingly. - While here reset MAINTAINER after a timeout of 3 months and repeated previous timeouts PR: 233458 Submitted by: kevans Approved by: chris@smartt.com (maintainer timeout, 3 months) Changes: head/net-mgmt/nagiosql/Makefile --=20 You are receiving this mail because: You are the assignee for the bug.=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-233458-7788-TDY6KOddAT>