Merge pull request #517 from karmix/tickets/master/MODULES-1976_unicode-rule-name...

(MODULES-1976) Revise rule name validation for ruby 1.9

diff --cc README.markdown
index 92e3228178bc2fda342fe45be5ae4b36968d15e9,874ebefbf28fdec5eee93d125578567df624269c..5132d258210f85ed76c9d9af860f2a03d278551e
--- 1/README.markdown
--- 2/README.markdown
+++ b/README.markdown
@@@ -609,9 -544,9 +609,9 @@@ firewall { '000 this runs first'
  firewall { '999 this runs last':
    # this rule will run last
  }
 - ```
 + ~~~
  
-   Depending on the provider, the name of the rule can be stored using the comment feature of the underlying firewall subsystem. Values must match '/^\d+[[:alpha:][:digit:][:punct:][:space:]]+$/'.
+   Depending on the provider, the name of the rule can be stored using the comment feature of the underlying firewall subsystem. Values must match '/^\d+[[:graph:][:space:]]+$/'.
  
  * `outiface`: Output interface to filter on. Values must match '/^!?\s?[a-zA-Z0-9\-\._\+\:]+$/'.  Requires the `interface_match` feature.  Supports interface alias (eth0:0) and negation.
  

diff --cc lib/puppet/provider/firewall/iptables.rb
Simple merge

diff --cc lib/puppet/type/firewall.rb
Simple merge

diff --cc spec/fixtures/iptables/conversion_hash.rb
Simple merge