beyondgrep / ack2

**ack 2 is no longer being maintained. ack 3 is the latest version.**
https://github.com/beyondgrep/ack3/
Other
1.48k stars 140 forks source link

dev/timings.pl should exit and warn if cannot find parrot #536

Closed pdl closed 8 years ago

pdl commented 9 years ago

I was confused as to why I was getting x_x in all my timings results - @hoelzro pointed out it was expecting to find a copy of parrot, which I hadn't realised before.

As a developer just starting to use timings, or using it on a new machine, this warning will tell me what I need to get and where to put it.

hoelzro commented 9 years ago

@pdl Thanks for the change; would you mind changing the -d ... or ... to unless(-d ...) { ... }? It's a matter of personal preference, but I'm not a big fan of the $condition or $action idiom.