Centralize Win32-detection

beyondgrep / ack2

**ack 2 is no longer being maintained. ack 3 is the latest version.**

https://github.com/beyondgrep/ack3/

Other

1.48k stars 138 forks source link

Centralize Win32-detection #378

Closed petdance closed 11 years ago

petdance commented 11 years ago

There are eight different places in the project, both in ack and its tests, where we test against $^O to determine what OS we're on. Centralize these in functions as best we can.

petdance commented 11 years ago

It seems to me there should be exactly three places we detect against $^O

Makefile.PL
Ack.pm
t/Util.pm

petdance commented 11 years ago

I've centralized what I'm going to centralize.