[104586] trunk/dports/devel/lpeg/Portfile

[Ryan Schmidt]

On Mar 29, 2013, at 03:05, and.damore at macports.org wrote:

> Revision: 104586
>          https://trac.macports.org/changeset/104586
> Author:   and.damore at macports.org
> Date:     2013-03-29 01:05:36 -0700 (Fri, 29 Mar 2013)
> Log Message:
> -----------
> port lpeg: moving replaced-by error earlier
> 
> Modified Paths:
> --------------
>    trunk/dports/devel/lpeg/Portfile
> 
> Modified: trunk/dports/devel/lpeg/Portfile
> ===================================================================
> --- trunk/dports/devel/lpeg/Portfile	2013-03-29 08:04:51 UTC (rev 104585)
> +++ trunk/dports/devel/lpeg/Portfile	2013-03-29 08:05:36 UTC (rev 104586)
> @@ -17,7 +17,7 @@
> 
> homepage            http://www.inf.puc-rio.br/~roberto/lpeg/lpeg.html
> distfiles
> -pre-configure {
> +pre-fetch {
>     ui_error "Please do not install this port since it has been replaced by 'lua-lpeg'."
>     return -code error
> }

Our standard recipe is to use pre-configure for this kind of message; see:

https://trac.macports.org/wiki/PortfileRecipes#replaced-by

http://guide.macports.org/#development.practices.rename-replace-port

https://trac.macports.org/browser/trunk/dports/_resources/port1.0/group/obsolete-1.0.tcl?rev=102028

Since we already clear distfiles, nothing's going to be fetched. So the only time it makes a difference is if a user runs something like "sudo port fetch outdated". Using a pre-fetch block prevents that from working right, therefore we use a pre-configure block.