Issue #18756 has been updated by David Kowis.


https://github.com/puppetlabs/facter/pull/390

I've submitted a pull request for this one. I'm pretty sure I did all the steps 
right. The fork is off the 1.6.17, since this is only a bugfix.
----------------------------------------
Bug #18756: wront regex in util/ip.rb
https://projects.puppetlabs.com/issues/18756#change-82423

Author: javier wilson
Status: Unreviewed
Priority: Normal
Assignee: 
Category: 
Target version: 
Keywords: 
Branch: 
Affected Facter version: 1.6.17


if you compare 
https://github.com/puppetlabs/facter/blob/master/lib/facter/ipaddress.rb#L32
      regexp = /inet (?:addr:)?([0-9]+\.[0-9]+\.[0-9]+\.[0-9]+)/
to https://github.com/puppetlabs/facter/blob/master/lib/facter/util/ip.rb#L8
      :ipaddress => /inet addr:([0-9]+\.[0-9]+\.[0-9]+\.[0-9]+)/,

you can see why the second one will not work in Fedora 16+ where the "addr:" 
string no longer exists.



-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to