[79471] trunk/dports/science

[Ryan Schmidt]

On Jun 14, 2011, at 10:48, robitaille at macports.org wrote:

> Revision: 79471
>          http://trac.macports.org/changeset/79471
> Author:   robitaille at macports.org
> Date:     2011-06-14 08:48:50 -0700 (Tue, 14 Jun 2011)
> Log Message:
> -----------
> montage: new port
> 
> Added Paths:
> -----------
>    trunk/dports/science/montage/
>    trunk/dports/science/montage/Portfile
> 
> Added: trunk/dports/science/montage/Portfile
> ===================================================================
> --- trunk/dports/science/montage/Portfile	                        (rev 0)
> +++ trunk/dports/science/montage/Portfile	2011-06-14 15:48:50 UTC (rev 79471)
> @@ -0,0 +1,28 @@
> +# -*- coding: utf-8; mode: tcl; tab-width: 4; truncate-lines: t; indent-tabs-mode: nil; c-basic-offset: 4 -*- vim:fenc=utf-8:et:sw=4:ts=4:sts=4
> +# $Id$
> +
> +PortSystem          1.0
> +PortGroup           xcodeversion 1.0

There's no point to including the xcodeversion portgroup if you're not going to set the minimum_xcodeversions variable. Unless your port has more specific Xcode version needs than MacPorts itself, you can remove this portgroup line.


> +name                montage
> +version             3.3
> +platforms           darwin
> +categories          science
> +maintainers         robitaille
> +description         Montage - Image Mosaic Software for Astronomers
> +long_description    Montage is a toolkit for assembling Flexible Image \
> +                    Transport System (FITS) images into custom mosaics
> +
> +license             http://montage.ipac.caltech.edu/docs/download.html

The license field is supposed to list the name (and, if known, version) of the license, not a URL.


> +homepage            http://montage.ipac.caltech.edu/
> +master_sites        http://montage.ipac.caltech.edu/download/
> +distname            Montage_v${version}
> +checksums           md5     875a88b4a2396a0eb5d0006a656e9c4a \
> +                    sha1    c8db5b9018c96e19b584d45758819e892b453d6d \
> +                    rmd160  5657d2948d997941b6c3037da87263db91b0e339
> +
> +use_configure       no

Use of "use_configure no" implies that you need to manually ensure the port is using the right compiler and build arch, and if possible add a universal variant:

https://trac.macports.org/wiki/UsingTheRightCompiler


> +destroot {
> +    eval file copy [glob ${worksrcpath}/bin/m*] ${destroot}${prefix}/bin/
> +}