cpe_firewall is not attribute driven on Windows

[1dustindavis]

On Windows, the cpe_firewall cookbook will enable the firewall if it is included in the run list. No attributes need to be set.

At minimum, the windows recipe should only run if node['cpe_firewall']['EnableFirewall'] is true.

[erikng]

Ugh. So this is hugely problematic for others using your cookbook. If you have a single run list for all machines, now you will have two behaviors for two different operating systems.

Curious why you didn't make it API driven prior to pushing it.

[ChefAustin]

Blame me, @erikng . I'm the one who added the wonky support for Windows. 😛

[erikng]

I'll blame Nate. Damnit @natewalck!

[natewalck]

I didn't make this or even commit code to this. Don't blame me! :D

[ChefAustin]

I’ll try to fix this tomorrow if the world doesn’t come to and end. Thanks for merging the other PR!