New port: mojoshader

[cooljeanius (Eric Gallager)]

Came across this while reading ​Valve's GDC slides. It's a translator for changing HLSL to GLSL. Portfile is attached. It passes port -v lint --nitpick, port test (with the +tests variant; see #38208), and port-depcheck.sh.

[cooljeanius (Eric Gallager)]

Portfile for mojoshader

[ryandesign (Ryan Carsten Schmidt)]

I understand that you added a +tests variant because you need a dependency on perl, and MacPorts doesn't have depends_test yet. But it's weird that selecting the +tests variant ends up installing additional files; usually tests are just run at build time and do not affect what gets installed. Also, is MacPorts perl really required to run the tests, or would OS X perl in /usr/bin/perl work? If the latter, then maybe you can use that and get rid of the variant. Usually MacPorts prefers to avoid OS X dependencies, but for such ubiquitous utilities as perl (and sed and awk and grep) we make exceptions.

Also you don't need a foreach loop to install multiple files (if you're not going to be renaming them), so you can install the documentation files in a single line:

xinstall -W ${worksrcpath} -m 644 README.txt LICENSE.txt ${destroot}${prefix}/share/${name}

Also note the use of -m 644 so that documentation files don't get the execute bit.

[cooljeanius (Eric Gallager)]

Updated portfile

[cooljeanius (Eric Gallager)]

I updated the portfile per your recommendations.