[MacPorts] #38685: New port: mojoshader

[MacPorts]

#38685: New port: mojoshader
-------------------------+--------------------------------
  Reporter:  egall@…     |      Owner:  macports-tickets@…
      Type:  submission  |     Status:  new
  Priority:  Normal      |  Milestone:
 Component:  ports       |    Version:  2.1.3
Resolution:              |   Keywords:
      Port:  mojoshader  |
-------------------------+--------------------------------

Comment (by ryandesign@…):

 I understand that you added a +tests variant because you need a dependency
 on perl, and [ticket:38208 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 [wiki:FAQ#syslibs
 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.

-- 
Ticket URL: <https://trac.macports.org/ticket/38685#comment:1>
MacPorts <http://www.macports.org/>
Ports system for OS X