rtyler/puppet-jenkins — pull request #6 by tbroyer (2012-02-24 10:30:32+00:00)
Title: Fix jenkins::firewall in case you don't have puppetlabs-firewall.

In case you don't have puppetlabs-firewall, the defined('firewall') still evaluates to true because of scoping: there's a firewall class in the jenkins namespace (indeed, we're inside that jenkins::firewall class). Puppet therefore complains that the type Firewall doesn't exist on the next line.
Using defined('::firewall') fixes the issue, and for consistency let's use ::firewall on the next line too.