new port: bowtie2

[cooljeanius (Eric Gallager)]

port -v lint --nitpick reports the following warnings:

Warning: Line 2 is missing RCS tag ($Id$)
Warning: no license set

Also I'd add some manual line wrapping to the long_description there. And also generally openmaintainer goes last in the maintainers list. But those are just nitpicks; otherwise it looks good.

[sean@…]

Updated

[ryandesign (Ryan Carsten Schmidt)]

The destroot can be simplified to just:

destroot {
    xinstall -W ${worksrcpath} bowtie2 bowtie2-align bowtie2-build bowtie2-inspect ${destroot}${prefix}/bin
}

eval is not necessary because you're not evaluating any globs. 0755 is the default mode. xinstall can install multiple files at once.

Using use_configure no means you need to manually ensure you're UsingTheRightCompiler and -arch flags and add a universal variant if possible—unless this port doesn't build architecture-specific software, in which case you need supported_archs noarch.

When writing sourceforge master_sites you have to be sure to avoid redirects.

[larryv (Lawrence Velázquez)]

If all you want to do is specify that the distfile has a different name than the default, don’t set distfiles; set distname instead, and leave off the extension. use_zip sets the correct extension for you. You’ll probably have to set worksrcdir back to ${name}-${version}.

Does the build seriously not support make install?

xinstall supports installing several files at once; you should do this instead of calling xinstall separately for each file. See the portfile(7) man page.

[sean@…]

Updated, thanks for the comments. I'll hold off on the others until this one is correct.

[mf2k (Frank Schima)]

This port was added at some point.