[26399] trunk/dports/math/nestedsums/Portfile

[Ryan Schmidt]

On Jun 21, 2007, at 00:53, source_changes at macosforge.org wrote:

> Revision: 26399
>           http://trac.macosforge.org/projects/macports/changeset/26399
> Author:   gwright at macports.org
> Date:     2007-06-20 22:53:34 -0700 (Wed, 20 Jun 2007)
>
> Log Message:
> -----------
> Build only on darwin_8 or later.  This eliminates the need for
> a macports g++ compiler, and since cln (a dependency via
> GiNaC) only works on darwin_8 or later, we don't lose anything.
>
> Modified Paths:
> --------------
>     trunk/dports/math/nestedsums/Portfile
>
> Modified: trunk/dports/math/nestedsums/Portfile
> ===================================================================
> --- trunk/dports/math/nestedsums/Portfile	2007-06-21 05:40:45 UTC  
> (rev 26398)
> +++ trunk/dports/math/nestedsums/Portfile	2007-06-21 05:53:34 UTC  
> (rev 26399)
> @@ -5,7 +5,7 @@
>  version		1.4.9
>  categories	math
>  platforms	darwin
> -maintainers	nomaintainer at macports.org
> +maintainers	gwright at macports.org
>  description	nestedsums library
>  long_description	\
>  		A Library for the Symbolic Expansion of a Class of	\
> @@ -21,9 +21,14 @@
>  #user_notes	On Tiger (10.4.x), Xcode 2.1 is required to avoid a bug
>  #		in gcc which causes one of the tests to fail.
>
> +platform darwin 6 {
> +		ui_msg "nesteedsums is not supported on Jaguar (OS X 10.2.x)."
> +		exit 1
> +}
> +
>  platform darwin 7 {
> -		depends_build	port:gcc40
> -		configure.env	CXX=g++-dp-4.0
> +		ui_msg "nesteedsums is not supported on Panther (OS X 10.3.x)."
> +		exit 1
>  }
>
>  default_variants	+test


On Jun 21, 2007, at 00:58, source_changes at macosforge.org wrote:

> Revision: 26400
>           http://trac.macosforge.org/projects/macports/changeset/26400
> Author:   gwright at macports.org
> Date:     2007-06-20 22:58:03 -0700 (Wed, 20 Jun 2007)
>
> Log Message:
> -----------
> Don't build on darwin 6 and 7 anymore.  Since GiNaC depends on cln
> which doesn't build on these versions, we don't lose anything.
>
> Modified Paths:
> --------------
>     trunk/dports/math/GiNaC/Portfile
>
> Modified: trunk/dports/math/GiNaC/Portfile
> ===================================================================
> --- trunk/dports/math/GiNaC/Portfile	2007-06-21 05:53:34 UTC (rev  
> 26399)
> +++ trunk/dports/math/GiNaC/Portfile	2007-06-21 05:58:03 UTC (rev  
> 26400)
> @@ -40,12 +40,14 @@
>  depends_lib	port:readline	\
>  		port:cln
>
> +platform darwin 6 {
> +		ui_msg "GiNaC is not supported on Jaguar (OS X 10.2.x)."
> +		exit 1
> +}
> +
>  platform darwin 7 {
> -		depends_build	port:gcc40
> -
> -		configure.env	CXX=g++-dp-4.0			\
> -				CPPFLAGS=-I${prefix}/include	\
> -				LDFLAGS=-L${prefix}/lib
> +		ui_msg "GiNaC is not supported on Panther (OS X 10.3.x)."
> +		exit 1
>  }
>
>  platform darwin 8 {


But could you please not output messages and exit merely in a  
platform statement? Please do so only within a stage such as pre- 
fetch, e.g.:

platfrom darwin 7 {
	pre-fetch {
		ui_msg "GiNaC is not supported on Panther (OS X 10.3.x)."
		exit 1
	}
}

Otherwise, people running port info, portindex, etc. on darwin 7 will  
be inconvenienced by the exit.