Bug 91324 - devel/portlint: warn if PORTREVISION=0
Summary: devel/portlint: warn if PORTREVISION=0
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Joe Marcus Clarke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-01-04 23:20 UTC by edwin
Modified: 2006-02-11 23:11 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description edwin 2006-01-04 23:20:02 UTC
	Portlint doesn't complain when PORTREVISION=0.
	
	Of course this is a theoretical excersize, and a matter of
	taste, but I don't really consider PORTREVISION=0 a nice
	thing to have in the ports Makefile.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2006-01-04 23:25:10 UTC
Responsible Changed
From-To: freebsd-ports-bugs->marcus

Over to maintainer
Comment 2 Mark Linimon 2006-01-05 00:53:44 UTC
On Thu, Jan 05, 2006 at 10:16:45AM +1100, Edwin Groothuis wrote:
> 	Portlint doesn't complain when PORTREVISION=0.
> 	
> 	Of course this is a theoretical excersize, and a matter of
> 	taste, but I don't really consider PORTREVISION=0 a nice
> 	thing to have in the ports Makefile.

This is sometimes needed in a slave port to avoid inheriting the PORTREVISION
from a masterport.  In fact, it could be argued that _all_ slave ports
should always set PORTREVISION.  (This can avoid the 'port going backwards'
problem).

Therefore I don't think we should do this.
Comment 3 edwin 2006-01-05 04:47:47 UTC
On Wed, Jan 04, 2006 at 06:53:44PM -0600, Mark Linimon wrote:
> On Thu, Jan 05, 2006 at 10:16:45AM +1100, Edwin Groothuis wrote:
> > 	Portlint doesn't complain when PORTREVISION=0.
> > 	
> > 	Of course this is a theoretical excersize, and a matter of
> > 	taste, but I don't really consider PORTREVISION=0 a nice
> > 	thing to have in the ports Makefile.
> 
> This is sometimes needed in a slave port to avoid inheriting the PORTREVISION
> from a masterport.  In fact, it could be argued that _all_ slave ports
> should always set PORTREVISION.  (This can avoid the 'port going backwards'
> problem).

A slave port will not include bsd.port.{,pre,post}.mk. So any port
which doesn't have this in the Makefile is a non-slave port, so the
check can still go on for them.

Edwin

-- 
Edwin Groothuis      |            Personal website: http://www.mavetju.org
edwin@mavetju.org    |          Weblog: http://weblog.barnet.com.au/edwin/
Comment 4 Joe Marcus Clarke freebsd_committer freebsd_triage 2006-02-11 23:11:02 UTC
State Changed
From-To: open->closed

Implemented in 2.8.4.