[36303] trunk/dports/www/yaws/Portfile

[Ryan Schmidt]

On Apr 26, 2008, at 2:55 AM, pguyot at kallisys.net wrote:

> Revision: 36303
>           http://trac.macosforge.org/projects/macports/changeset/36303
> Author:   pguyot at kallisys.net
> Date:     2008-04-26 00:55:04 -0700 (Sat, 26 Apr 2008)
>
> Log Message:
> -----------
> www/yaws: added yapp variant for yapp application handler
>
> Modified Paths:
> --------------
>     trunk/dports/www/yaws/Portfile
>
> Modified: trunk/dports/www/yaws/Portfile
> ===================================================================
> --- trunk/dports/www/yaws/Portfile	2008-04-26 07:42:41 UTC (rev 36302)
> +++ trunk/dports/www/yaws/Portfile	2008-04-26 07:55:04 UTC (rev 36303)
> @@ -36,9 +36,20 @@
>  	reinplace "s|__PREFIX|${prefix}|g" ${worksrcpath}/src/ 
> yaws_config.erl
>  }
>
> +default_variants	+yapp
> +
>  configure.args		--prefix=${prefix} \
>  					--sysconfdir=${prefix}/etc \
>  					--localstatedir=${prefix}/var
> +
> +variant yapp description {Yapp application handler} {
> +	post-build {
> +		system "cd ${worksrcpath}/applications/yapp && make && make docs"
> +	}
> +	post-destroot {
> +		system "cd ${worksrcpath}/applications/yapp && make install  
> DESTDIR=${destroot}"
> +	}
> +}


Why make it a variant? If it doesn't have extra dependencies, why not  
just make it all the time?

If there is benefit in turning off, but you want it on by default,  
better to make it a no_yapp variant that turns it off, since there is  
a bug with user overrides of default_variants not being preserved  
during upgrades.

Should increment port revision so everyone who has the port installed  
already gets yapp.