ports/150476: PORTREVISION of courier-authlib*

[TAOKA Fumiyoshi]

>Number:         150476
>Category:       ports
>Synopsis:       PORTREVISION of courier-authlib*
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Fri Sep 10 15:40:01 UTC 2010
>Closed-Date:
>Last-Modified:
>Originator:     TAOKA Fumiyoshi
>Release:        
>Organization:
>Environment:
>Description:

As shown below both courier-authlib and courier-authlib-ldap, 
which are master and slave ports, define PORTREVISION and
they conflict.

# grep PORTREVISION /usr/ports/*/courier-authlib*/Makefile
/usr/ports/net/courier-authlib-ldap/Makefile:PORTREVISION=      2
/usr/ports/security/courier-authlib/Makefile:PORTREVISION=      1

As for courier-authlib-ldap, the port system recognizes
PORTREVISION as not 2 but 1. The last bump is not effective.

>How-To-Repeat:
>Fix:

Option 1: Respect PORTREVISION in a slave port.

Index: Makefile
===================================================================
RCS file: /home/ncvs/ports/security/courier-authlib/Makefile,v
retrieving revision 1.47
diff -u -r1.47 Makefile
--- Makefile    11 Jun 2010 22:28:28 -0000      1.47
+++ Makefile    8 Sep 2010 05:26:26 -0000
@@ -7,7 +7,7 @@
 
 PORTNAME=      courier-authlib
 PORTVERSION=   0.63.0
-PORTREVISION=  1
+PORTREVISION?= 1
 CATEGORIES?=   security mail
 .if defined(PKGNAMESUFFIX)
 MASTER_SITES=  SF/courier/authlib/${PORTVERSION}



Option 2: Control PORTREVISION in a master port, for its slaves
          are simple skeltons.

Index: Makefile
===================================================================
RCS file: /home/ncvs/ports/security/courier-authlib/Makefile,v
retrieving revision 1.47
diff -u -r1.47 Makefile
--- Makefile	11 Jun 2010 22:28:28 -0000	1.47
+++ Makefile	10 Sep 2010 15:02:51 -0000
@@ -7,7 +7,7 @@
 
 PORTNAME=	courier-authlib
 PORTVERSION=	0.63.0
-PORTREVISION=	1
+PORTREVISION=	2
 CATEGORIES?=	security mail
 .if defined(PKGNAMESUFFIX)
 MASTER_SITES=	SF/courier/authlib/${PORTVERSION}


Index: Makefile
===================================================================
RCS file: /home/ncvs/ports/net/courier-authlib-ldap/Makefile,v
retrieving revision 1.18
diff -u -r1.18 Makefile
--- Makefile	2 Aug 2009 19:34:56 -0000	1.18
+++ Makefile	10 Sep 2010 15:03:56 -0000
@@ -8,7 +8,6 @@
 PKGNAMESUFFIX=	-ldap
 
 COMMENT=	LDAP support for the Courier authentication library
-PORTREVISION=	2
 CATEGORIES=	net mail
 
 MASTERDIR=	${.CURDIR}/../../security/courier-authlib

>Release-Note:
>Audit-Trail:
>Unformatted: