Install PHPUnit via phar just like Composer

[westonruter]

This copies the approach used to install Composer via phar.

Will fix https://github.com/Automattic/vip-quickstart/issues/289

[jippi]

seems legit 👍

[igalic]

proposal: replace exec-wget, with @maestrodev's wget.
(here, and in composer)

[jippi]

won't that make it pain in the ass for people to install dependencies ?

[joshbetz]

An external dependency to make wget easier seems like overkill. @jippi is there anything else that needs to happen before this is merged?

[jippi]

Other than the ensure should be absent = delete the file again I think its good

[westonruter]

I confess I am not a puppet master. Where would "absent = delete" go?

[jippi]

ensure = absent should be in the primary manifest, and just delete the composer file :)

[westonruter]

Like b6e83bd?

[joshbetz]

Probably something like:

if ( 'absent' == $ensure ) {
  file { $destination:
    path   => $destination,
    ensure => absent
  }
} else {
  // Install phpunit
}

[westonruter]

@joshbetz I'm out of my depth here. I'll leave it with you.

[joshbetz]

Thanks @westonruter. It looks like schkovich@ea1ead5 breaks Quickstart too so hopefully we can get that reverted. Otherwise I'll look into a workaround.

[igalic]

i believe this is now covered by #128
thank you @joshbetz and @westonruter!