[macports-ports] branch master updated: certbot: rename py-certbot to certbot

[Ryan Schmidt]

> On Jan 15, 2017, at 10:30, Marius Schamschula <mps at macports.org> wrote:
> 
> Marius Schamschula (Schamschula) pushed a commit to branch master
> in repository macports-ports.
> 
> 
> https://github.com/macports/macports-ports/commit/cb88f3944b1c4949168ca876d4ca4a55a93f6772
> 
> The following commit(s) were added to refs/heads/master by this push:
> 
>      new cb88f39  certbot: rename py-certbot to certbot
> 
> cb88f39 is described below
> 
> 
> commit cb88f3944b1c4949168ca876d4ca4a55a93f6772
> 
> Author: Marius Schamschula <mps at macports.org>
> AuthorDate: Sun Jan 15 10:30:37 2017 -0600
> 
> 
>     certbot: rename py-certbot to certbot
> 
>     
> 
>     Move from py-certbot to certbot
> 
>     Simplify Portfile to only allow python 2.7, as python 3.x is not yet supported
> 
> ---
>  python/py-certbot/Portfile | 42 ++++--------------------------------------
>  security/certbot/Portfile  | 37 +++++++++++++++++++++++++++++++++++++
>  2 files changed, 41 insertions(+), 38 deletions(-)
> 
> 
> diff --git a/python/py-certbot/Portfile b/python/py-certbot/Portfile
> 
> index 4e76f00..e4b765f 100644
> 
> --- a/python/py-certbot/Portfile
> 
> +++ b/python/py-certbot/Portfile
> 
> @@ -1,46 +1,12 @@
> 
>  # -*- coding: utf-8; mode: tcl; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- vim:fenc=utf-8:ft=tcl:et:sw=4:ts=4:sts=4
>  
>  PortSystem          1.0
> 
> -PortGroup           github 1.0
> 
> -PortGroup           python 1.0
> 
>  
> 
> -github.setup        certbot certbot 0.10.1 v
> 
> +replaced_by         certbot
> 
> +
> 
>  name                py-certbot

replaced_by doesn't work unless you increase the revision (or version).


Consider using the obsolete 1.0 portgroup when replacing a port. It takes care of several issues for you, some of which you're currently not handling:

$ port lint py-certbot
--->  Verifying Portfile for py-certbot
Error: Missing required variable: description
Error: Missing required variable: long_description
Error: Missing required variable: platforms
Error: Missing required variable: master_sites
--->  4 errors and 0 warnings found.


Not only py-certbot should be replaced; py27-certbot should also be replaced.