From pgollucci@frieza.p6m7g8.net  Thu May 27 20:55:42 2010
Return-Path: <pgollucci@frieza.p6m7g8.net>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 7AA3B106564A
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 27 May 2010 20:55:42 +0000 (UTC)
	(envelope-from pgollucci@frieza.p6m7g8.net)
Received: from frieza.p6m7g8.net (wsip-174-79-184-239.dc.dc.cox.net [174.79.184.239])
	by mx1.freebsd.org (Postfix) with ESMTP id 3752B8FC0C
	for <FreeBSD-gnats-submit@freebsd.org>; Thu, 27 May 2010 20:55:41 +0000 (UTC)
Received: from frieza.p6m7g8.net (localhost [127.0.0.1])
	by frieza.p6m7g8.net (8.14.4/8.14.4) with ESMTP id o4RKr8R2099573;
	Thu, 27 May 2010 16:53:08 -0400 (EDT)
	(envelope-from pgollucci@frieza.p6m7g8.net)
Received: (from pgollucci@localhost)
	by frieza.p6m7g8.net (8.14.4/8.14.4/Submit) id o4RKr8Jn099559;
	Thu, 27 May 2010 16:53:08 -0400 (EDT)
	(envelope-from pgollucci)
Message-Id: <201005272053.o4RKr8Jn099559@frieza.p6m7g8.net>
Date: Thu, 27 May 2010 16:53:08 -0400 (EDT)
From: "Philip M. Gollucci" <pgollucci@p6m7g8.com>
To: FreeBSD-gnats-submit@freebsd.org
Cc: bfoz@bfoz.net
Subject: [PATCH] www/mod_log_dbd: cleanups
X-Send-Pr-Version: 3.113
X-GNATS-Notify: bfoz@bfoz.net

>Number:         147147
>Category:       ports
>Synopsis:       [PATCH] www/mod_log_dbd: cleanups
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    apache
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Thu May 27 21:00:07 UTC 2010
>Closed-Date:    Tue Aug 31 01:16:08 UTC 2010
>Last-Modified:  Tue Aug 31 01:16:08 UTC 2010
>Originator:     Philip M. Gollucci
>Release:        FreeBSD 9.0-CURRENT amd64
>Organization:
RideCharge Inc / TaxiMagic
>Environment:
System: FreeBSD frieza.p6m7g8.net 9.0-CURRENT FreeBSD 9.0-CURRENT #0: Mon Apr 26 16:20:00 EDT 2010
>Description:
- pet portlint
- use apxs not sed in pkg-plist
- apxs -a is install -A is uninistall
- bump portrevision

Port maintainer (bfoz@bfoz.net) is cc'd.

Generated with FreeBSD Port Tools 0.99
>How-To-Repeat:
>Fix:

--- ap22-mod_log_dbd-0.2_4.patch begins here ---
Index: Makefile
===================================================================
RCS file: /home/ncvs/ports/www/mod_log_dbd/Makefile,v
retrieving revision 1.7
diff -u -u -r1.7 Makefile
--- Makefile	25 May 2010 20:17:32 -0000	1.7
+++ Makefile	27 May 2010 20:36:17 -0000
@@ -6,7 +6,7 @@
 
 PORTNAME=	mod_log_dbd
 PORTVERSION=	0.2
-PORTREVISION=	3
+PORTREVISION=	4
 CATEGORIES=	www databases
 MASTER_SITES=	http://bfoz.net/projects/${PORTNAME}/release/
 PKGNAMEPREFIX=	${APACHE_PKGNAMEPREFIX}
@@ -14,7 +14,7 @@
 MAINTAINER=	bfoz@bfoz.net
 COMMENT=	Uses APR DBD to store Apache access logs in a database
 
-MAKE_JOBS_SAFE=  yes
+MAKE_JOBS_SAFE=	yes
 
 USE_APACHE=	2.2+
 USE_BZIP2=	yes
Index: pkg-plist
===================================================================
RCS file: /home/ncvs/ports/www/mod_log_dbd/pkg-plist,v
retrieving revision 1.2
diff -u -u -r1.2 pkg-plist
--- pkg-plist	15 Jul 2007 19:10:58 -0000	1.2
+++ pkg-plist	27 May 2010 20:36:49 -0000
@@ -1,4 +1,4 @@
+@exec %D/sbin/apxs -e -a -n %%AP_NAME%% %D/%f
 %%APACHEMODDIR%%/%%AP_MODULE%%
-@exec %D/sbin/apxs -e -A -n %%AP_NAME%% %D/%%APACHEMODDIR%%/%%AP_MODULE%%
-@unexec /usr/bin/sed -i '' '/LoadModule %%AP_NAME%%_module/d' %D/%%APACHEETCDIR%%/httpd.conf
+@unexec %D/sbin/apxs -e -A -n %%AP_NAME%% %D/%f
 @unexec echo "Don't forget to remove all mod_%%AP_NAME%% related directives in your httpd.conf"
--- ap22-mod_log_dbd-0.2_4.patch ends here ---

>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->apache 
Responsible-Changed-By: edwin 
Responsible-Changed-When: Thu May 27 21:00:16 UTC 2010 
Responsible-Changed-Why:  
apache@ wants this port PRs (via the GNATS Auto Assign Tool) 

http://www.freebsd.org/cgi/query-pr.cgi?pr=147147 
State-Changed-From-To: open->feedback 
State-Changed-By: edwin 
State-Changed-When: Thu May 27 21:00:20 UTC 2010 
State-Changed-Why:  
Awaiting maintainers feedback (via the GNATS Auto Assign Tool) 

http://www.freebsd.org/cgi/query-pr.cgi?pr=147147 

From: Edwin Groothuis <edwin@FreeBSD.org>
To: bfoz@bfoz.net
Cc: bug-followup@FreeBSD.org
Subject: Re: ports/147147: [PATCH] www/mod_log_dbd: cleanups
Date: Thu, 27 May 2010 21:00:19 UT

 Maintainer of www/mod_log_dbd,
 
 Please note that PR ports/147147 has just been submitted.
 
 If it contains a patch for an upgrade, an enhancement or a bug fix
 you agree on, reply to this email stating that you approve the patch
 and a committer will take care of it.
 
 The full text of the PR can be found at:
     http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/147147
 
 -- 
 Edwin Groothuis via the GNATS Auto Assign Tool
 edwin@FreeBSD.org

From: Brandon Fosdick <bfosdick@gmail.com>
To: bug-followup@FreeBSD.ORG
Cc:  
Subject: Re: ports/147147: [PATCH] www/mod_log_dbd: cleanups
Date: Fri, 28 May 2010 21:42:27 +0200

 --Apple-Mail-4-415804560
 Content-Transfer-Encoding: quoted-printable
 Content-Type: text/plain;
 	charset=us-ascii
 
 I used sed for @unexec to avoid polluting http.conf with a commented out =
 LoadModule line, which IIRC was the recommended practice at the time. If =
 using 'apxs -A' is the will of the ports team then I'm fine with the =
 change.
 Using 'apxs -A' for @exec prevents enabling the module before the admin =
 has had a chance to properly configure it. If the module were enabled =
 before properly configuring a target database there would be nowhere to =
 write future log events to and they would consequently be discarded. It =
 would appear to the admin that the server had mysteriously stopped =
 logging. However, if it's the will of the ports team that modules be =
 immediately enabled...
 
 On May 27, 2010, at 23:00 , Edwin Groothuis wrote:
 
 > Maintainer of www/mod_log_dbd,
 >=20
 > Please note that PR ports/147147 has just been submitted.
 >=20
 > If it contains a patch for an upgrade, an enhancement or a bug fix
 > you agree on, reply to this email stating that you approve the patch
 > and a committer will take care of it.
 >=20
 > The full text of the PR can be found at:
 >    http://www.freebsd.org/cgi/query-pr.cgi?pr=3Dports/147147
 >=20
 > --=20
 > Edwin Groothuis via the GNATS Auto Assign Tool
 > edwin@FreeBSD.org
 
 
 --Apple-Mail-4-415804560
 Content-Disposition: attachment;
 	filename=smime.p7s
 Content-Type: application/pkcs7-signature;
 	name=smime.p7s
 Content-Transfer-Encoding: base64
 
 MIAGCSqGSIb3DQEHAqCAMIACAQExCzAJBgUrDgMCGgUAMIAGCSqGSIb3DQEHAQAAoIIOeDCCBo4w
 ggV2oAMCAQICAwDVfTANBgkqhkiG9w0BAQUFADCBjDELMAkGA1UEBhMCSUwxFjAUBgNVBAoTDVN0
 YXJ0Q29tIEx0ZC4xKzApBgNVBAsTIlNlY3VyZSBEaWdpdGFsIENlcnRpZmljYXRlIFNpZ25pbmcx
 ODA2BgNVBAMTL1N0YXJ0Q29tIENsYXNzIDEgUHJpbWFyeSBJbnRlcm1lZGlhdGUgQ2xpZW50IENB
 MB4XDTA5MTIyMjA3MjkzMloXDTEwMTIyMzE5MjYyOVowgYsxIDAeBgNVBA0TFzExNzkwNi1BYlB5
 cGQ1NTFseU1WeDFwMR4wHAYDVQQKExVQZXJzb25hIE5vdCBWYWxpZGF0ZWQxKTAnBgNVBAMTIFN0
 YXJ0Q29tIEZyZWUgQ2VydGlmaWNhdGUgTWVtYmVyMRwwGgYJKoZIhvcNAQkBFg1iZm96QGJmb3ou
 bmV0MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA38vfMRFJ4cVBCHWE68GP3vw+nlUq
 p+u7vmLoMvrLwKo3d1UhAIbKfk/ikWp4RSnv7maVbcbGA5OqJQbAMTPg4GhSJKOuYFx8NqeB4yM3
 B5bZ2J6rMaGc0WyafcOiUYizAjUUxuFIf2KzT17k53F8WJgH4/RLTxKUnQqH/GAUiDhgdoVNgqHu
 B5/sisKDnN8RPa5176rj3n+fO5w42hJF80ksztHCWGlCCKwOcdV75tDSyzBpL2SShLrE+7xO6OTB
 0khednBrmzMJQt5RkvNgeGunRZ62uoOf5KAgnlPo2zL7nVXxnsGt73rBuf+Kn1Ozb2kUjBfX9G9d
 Gun0j5/61QIDAQABo4IC9jCCAvIwCQYDVR0TBAIwADALBgNVHQ8EBAMCBLAwHQYDVR0lBBYwFAYI
 KwYBBQUHAwIGCCsGAQUFBwMEMB0GA1UdDgQWBBRV3kCTwLBLjYw8tTiZE3SJxI0hEDAfBgNVHSME
 GDAWgBRTcu2SnODaywFcfH6WNU7y1LhRgjAYBgNVHREEETAPgQ1iZm96QGJmb3oubmV0MIIBQgYD
 VR0gBIIBOTCCATUwggExBgsrBgEEAYG1NwECATCCASAwLgYIKwYBBQUHAgEWImh0dHA6Ly93d3cu
 c3RhcnRzc2wuY29tL3BvbGljeS5wZGYwNAYIKwYBBQUHAgEWKGh0dHA6Ly93d3cuc3RhcnRzc2wu
 Y29tL2ludGVybWVkaWF0ZS5wZGYwgbcGCCsGAQUFBwICMIGqMBQWDVN0YXJ0Q29tIEx0ZC4wAwIB
 ARqBkUxpbWl0ZWQgTGlhYmlsaXR5LCBzZWUgc2VjdGlvbiAqTGVnYWwgTGltaXRhdGlvbnMqIG9m
 IHRoZSBTdGFydENvbSBDZXJ0aWZpY2F0aW9uIEF1dGhvcml0eSBQb2xpY3kgYXZhaWxhYmxlIGF0
 IGh0dHA6Ly93d3cuc3RhcnRzc2wuY29tL3BvbGljeS5wZGYwYwYDVR0fBFwwWjAroCmgJ4YlaHR0
 cDovL3d3dy5zdGFydHNzbC5jb20vY3J0dTEtY3JsLmNybDAroCmgJ4YlaHR0cDovL2NybC5zdGFy
 dHNzbC5jb20vY3J0dTEtY3JsLmNybDCBjgYIKwYBBQUHAQEEgYEwfzA5BggrBgEFBQcwAYYtaHR0
 cDovL29jc3Auc3RhcnRzc2wuY29tL3N1Yi9jbGFzczEvY2xpZW50L2NhMEIGCCsGAQUFBzAChjZo
 dHRwOi8vd3d3LnN0YXJ0c3NsLmNvbS9jZXJ0cy9zdWIuY2xhc3MxLmNsaWVudC5jYS5jcnQwIwYD
 VR0SBBwwGoYYaHR0cDovL3d3dy5zdGFydHNzbC5jb20vMA0GCSqGSIb3DQEBBQUAA4IBAQCelMgD
 nwlNQlHccWFHUtCz8ZHVLHvWqb9yIa+vKuxMvOIM9qz2zswfxqj3didguE0fJPGDRjJardTSrv36
 uFtXj5Rt1nE26kUoJHYEwau463QeNTY3ONWUmHnKX5QYTWbQXij6BaerMAcqYzcdxtplL66uxFsr
 iDpl8ccniT4cuDFwWTpR5PZ7dyaCHsZ3ENXiZDD+d6IXzHaWiGRB9yxlXPQYopcwn904RAnfUIDV
 R8UmyuKAe1YXJhI1Rc9XptUMgKnGbh1TCe9V3eV9bzQxp56oHivQ1wlc/DxjDulWMNrTzwBkUCP7
 q0Nar8JTeLgM0DsDNLyqBg0c1zG4alg9MIIH4jCCBcqgAwIBAgIBDTANBgkqhkiG9w0BAQUFADB9
 MQswCQYDVQQGEwJJTDEWMBQGA1UEChMNU3RhcnRDb20gTHRkLjErMCkGA1UECxMiU2VjdXJlIERp
 Z2l0YWwgQ2VydGlmaWNhdGUgU2lnbmluZzEpMCcGA1UEAxMgU3RhcnRDb20gQ2VydGlmaWNhdGlv
 biBBdXRob3JpdHkwHhcNMDcxMDI0MjEwMTU0WhcNMTIxMDIyMjEwMTU0WjCBjDELMAkGA1UEBhMC
 SUwxFjAUBgNVBAoTDVN0YXJ0Q29tIEx0ZC4xKzApBgNVBAsTIlNlY3VyZSBEaWdpdGFsIENlcnRp
 ZmljYXRlIFNpZ25pbmcxODA2BgNVBAMTL1N0YXJ0Q29tIENsYXNzIDEgUHJpbWFyeSBJbnRlcm1l
 ZGlhdGUgQ2xpZW50IENBMIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEAxwmDzM4t2Bqx
 KaQuE6uWvooyg4ymiEGWVUet1G8SD+rqvyNH4QrvnEIaFHxOhESip7vMz39ScLpNLbL1QpOlPW/t
 FIzNHS3qd2XRNYG5Sv9RcGE+T4qbLtsjjJbi6sL7Ls/f/X9ftTyhxvxWkf8KW37iKrueKsxw2Hqo
 lH7GM6FX5UfNAwAu4ZifkpmZzU1slBhyWwaQPEPPZRsWoTb7q8hmgv6Nv3Hg9rmA1/VPBIOQ6SKR
 kHXG0Hhmq1dOFoAFI411+a/9nWm5rcVjGcIWZ2v/43Yksq60jExipA4l5uv9/+Hm33mbgmCszdj/
 Dthf13tgAv2O83hLJ0exTqfrlwIDAQABo4IDWzCCA1cwDAYDVR0TBAUwAwEB/zALBgNVHQ8EBAMC
 AaYwHQYDVR0OBBYEFFNy7ZKc4NrLAVx8fpY1TvLUuFGCMIGoBgNVHSMEgaAwgZ2AFE4L7xqkQFul
 F2mHMMo0aEPQQa7yoYGBpH8wfTELMAkGA1UEBhMCSUwxFjAUBgNVBAoTDVN0YXJ0Q29tIEx0ZC4x
 KzApBgNVBAsTIlNlY3VyZSBEaWdpdGFsIENlcnRpZmljYXRlIFNpZ25pbmcxKTAnBgNVBAMTIFN0
 YXJ0Q29tIENlcnRpZmljYXRpb24gQXV0aG9yaXR5ggEBMAkGA1UdEgQCMAAwPQYIKwYBBQUHAQEE
 MTAvMC0GCCsGAQUFBzAChiFodHRwOi8vd3d3LnN0YXJ0c3NsLmNvbS9zZnNjYS5jcnQwYAYDVR0f
 BFkwVzAsoCqgKIYmaHR0cDovL2NlcnQuc3RhcnRjb20ub3JnL3Nmc2NhLWNybC5jcmwwJ6AloCOG
 IWh0dHA6Ly9jcmwuc3RhcnRzc2wuY29tL3Nmc2NhLmNybDCCAV0GA1UdIASCAVQwggFQMIIBTAYL
 KwYBBAGBtTcBAQQwggE7MC8GCCsGAQUFBwIBFiNodHRwOi8vY2VydC5zdGFydGNvbS5vcmcvcG9s
 aWN5LnBkZjA1BggrBgEFBQcCARYpaHR0cDovL2NlcnQuc3RhcnRjb20ub3JnL2ludGVybWVkaWF0
 ZS5wZGYwgdAGCCsGAQUFBwICMIHDMCcWIFN0YXJ0IENvbW1lcmNpYWwgKFN0YXJ0Q29tKSBMdGQu
 MAMCAQEagZdMaW1pdGVkIExpYWJpbGl0eSwgcmVhZCB0aGUgc2VjdGlvbiAqTGVnYWwgTGltaXRh
 dGlvbnMqIG9mIHRoZSBTdGFydENvbSBDZXJ0aWZpY2F0aW9uIEF1dGhvcml0eSBQb2xpY3kgYXZh
 aWxhYmxlIGF0IGh0dHA6Ly9jZXJ0LnN0YXJ0Y29tLm9yZy9wb2xpY3kucGRmMBEGCWCGSAGG+EIB
 AQQEAwIABzBQBglghkgBhvhCAQ0EQxZBU3RhcnRDb20gQ2xhc3MgMSBQcmltYXJ5IEludGVybWVk
 aWF0ZSBGcmVlIFNTTCBFbWFpbCBDZXJ0aWZpY2F0ZXMwDQYJKoZIhvcNAQEFBQADggIBAKqa4eBb
 jM4dG/wdxiwwIKC3kyb98QK2zREovyn/xzDP/4H/Bc8FFDTgoJR+nX2Li0EP3U7TsjG+CaIi90+8
 YlShADpkPrfm/8SzjGtJtfM6EaluJOhpcqMr3OyzK3aYGJP5RIeZ6vLT3fQaDZsIooXl6YSFR/0H
 pU4FJDc0wuyFaZmFbCrjTp8RNYyRWTTX6mWSv+TraOwuj3zrrddSpgUEi2WqwM9G/5o4IXQbGHx7
 oXTvL6zrw9IOYO3QOKZDgFNhHeKUgqMAUiLcg/+WhcGe+Y4umKuxghtwaYsgD/bLfIfop3NC/u5J
 qwDCWizAJruhmbOV4LG859MFCb2w/YeY55zDPVGmQ3MZdriwdOKrhlFjOjYihmm28UHOvND2G3kK
 0LvnuieLqjQMc6GuUcZAQOWv96pW4BfbiQXpAqibMMeb0PZISa7PFEzGiBc2xAuVRkM4kB9/+iie
 A1D/OTiRJwsf6rkoVgOsN9fCw522tzOmuVfiqDS4bFYv00sX/dFGwasHUUf3DsLhpDSYdejb74SK
 jtuqLDIOuAm2bA1axA6+7kjFeNIngSU6OPSMre+xAjoc/6coaMGthFD+mimr/i/8F8wDwdyzas7o
 xkdCtaW8hVir8mJnbp4CbckllDMPkeQ6qQNmxSDhOeqX1jyx2cTi/vPq+/TyxV/stlehMYIDbzCC
 A2sCAQEwgZQwgYwxCzAJBgNVBAYTAklMMRYwFAYDVQQKEw1TdGFydENvbSBMdGQuMSswKQYDVQQL
 EyJTZWN1cmUgRGlnaXRhbCBDZXJ0aWZpY2F0ZSBTaWduaW5nMTgwNgYDVQQDEy9TdGFydENvbSBD
 bGFzcyAxIFByaW1hcnkgSW50ZXJtZWRpYXRlIENsaWVudCBDQQIDANV9MAkGBSsOAwIaBQCgggGv
 MBgGCSqGSIb3DQEJAzELBgkqhkiG9w0BBwEwHAYJKoZIhvcNAQkFMQ8XDTEwMDUyODE5NDIyOFow
 IwYJKoZIhvcNAQkEMRYEFJ1LRHQVGQUr6SH2yiZzaXdyoTV2MIGlBgkrBgEEAYI3EAQxgZcwgZQw
 gYwxCzAJBgNVBAYTAklMMRYwFAYDVQQKEw1TdGFydENvbSBMdGQuMSswKQYDVQQLEyJTZWN1cmUg
 RGlnaXRhbCBDZXJ0aWZpY2F0ZSBTaWduaW5nMTgwNgYDVQQDEy9TdGFydENvbSBDbGFzcyAxIFBy
 aW1hcnkgSW50ZXJtZWRpYXRlIENsaWVudCBDQQIDANV9MIGnBgsqhkiG9w0BCRACCzGBl6CBlDCB
 jDELMAkGA1UEBhMCSUwxFjAUBgNVBAoTDVN0YXJ0Q29tIEx0ZC4xKzApBgNVBAsTIlNlY3VyZSBE
 aWdpdGFsIENlcnRpZmljYXRlIFNpZ25pbmcxODA2BgNVBAMTL1N0YXJ0Q29tIENsYXNzIDEgUHJp
 bWFyeSBJbnRlcm1lZGlhdGUgQ2xpZW50IENBAgMA1X0wDQYJKoZIhvcNAQEBBQAEggEAATqNYSBy
 5+UXhr1UJnErzOv6sm/Kdz+46AHB4s/p28hpkVF5kZxVN0qMA/SxLpHE/8cqqS3ovJg2SH/ObW2f
 /L8uuOwehEQi+8OLcyG9E8MypQx+3BIQNKvl0oiN6xpvXHaelAziJ2PVLZUGCZXILBSUinx869Hu
 NnZL4zHv+dzmI3L/o2HVNKOzWqplCfEDc6VvblrDZMaI58ZM4QF51wCHFgSyhc7nqKJCxEsiEUAO
 asnpkJrdGak4XVglVpVPt9l1o0yjarJ3qVCsvcyRmWplYxOglMFbiuhS4+Xu0rdXFj1b2OC3YQCw
 vorkr/7xmFZRxmrtJorvqbm0g3HpIwAAAAAAAA==
 
 --Apple-Mail-4-415804560--
State-Changed-From-To: feedback->open 
State-Changed-By: pgollucci 
State-Changed-When: Wed Jun 9 23:38:59 UTC 2010 
State-Changed-Why:  
maintainer timeout 14 days 

http://www.freebsd.org/cgi/query-pr.cgi?pr=147147 
State-Changed-From-To: open->closed 
State-Changed-By: pgollucci 
State-Changed-When: Tue Aug 31 01:16:06 UTC 2010 
State-Changed-Why:  
since this is logging related, I agree with maintainer for now 

http://www.freebsd.org/cgi/query-pr.cgi?pr=147147 
>Unformatted:
