[79471] trunk/dports/science

[Thomas Robitaille]

> > +PortSystem 1.0
> > +PortGroup xcodeversion 1.0There'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.

Ok, thanks - I'll fix that! 

> > +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.
> 

The issue is that it's a custom license, so I'm not sure what to do in this case?

> > +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 noUse 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

I will try and add this - it will require patching quite a few files, but should be possible.

Thanks,
Thomas 

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