#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