Fix undefined line variable error in LVM::External

[wheatevo]

Description

This is a regression that was introduced in commit
9e73c28. The change is a simple fix
to ensure the line variable is defined at the time it is accessed.

Signed-off-by: Matthew Newell [email protected]

Issues Resolved

Fixes #10

Current Behavior

require 'lvm'
LVM::VERSION
 => "0.4.1"

lvm = LVM::LVM.new
/usr/share/rvm/gems/ruby-3.0.2/gems/chef-ruby-lvm-0.4.1/lib/lvm/external.rb:12:in `block in cmd': undefined local variable or method `line' for LVM::External:Module (NameError)
        from /usr/share/rvm/gems/ruby-3.0.2/gems/open4-0.9.6/lib/open4.rb:68:in `popen4'
        from /usr/share/rvm/gems/ruby-3.0.2/gems/chef-ruby-lvm-0.4.1/lib/lvm/external.rb:11:in `cmd'
        from /usr/share/rvm/gems/ruby-3.0.2/gems/chef-ruby-lvm-0.4.1/lib/lvm.rb:51:in `raw'
        from /usr/share/rvm/gems/ruby-3.0.2/gems/chef-ruby-lvm-0.4.1/lib/lvm.rb:68:in `userland'
        from /usr/share/rvm/gems/ruby-3.0.2/gems/chef-ruby-lvm-0.4.1/lib/lvm.rb:62:in `version'
        from /usr/share/rvm/gems/ruby-3.0.2/gems/chef-ruby-lvm-0.4.1/lib/lvm.rb:35:in `initialize'
        from (irb):3:in `new'
        from (irb):3:in `<main>'
        from /usr/share/rvm/rubies/ruby-3.0.2/lib/ruby/gems/3.0.0/gems/irb-1.3.5/exe/irb:11:in `<top (required)>'
        from /usr/share/rvm/rubies/ruby-3.0.2/bin/irb:23:in `load'
        from /usr/share/rvm/rubies/ruby-3.0.2/bin/irb:23:in `<main>'

New Behavior

require 'lvm'
LVM::VERSION
 => "0.4.2"

lvm = LVM::LVM.new
 =>
#<LVM::LVM:0x000055f368c42190
...

Check List

• New functionality includes tests (N/A, no tests defined)
• All tests pass (N/A, no tests defined)
• All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco
• PR title is a worthy inclusion in the CHANGELOG

[wheatevo]

@damacus

This change includes the version bump to 0.4.2, let me know if you would prefer that to be left out of this PR.

[Soulou]

We're also impacting by this issue, so I'm sending this comment as a "up". Can't wait to see it released. Thanks a lot.

[damacus]

Sorry this got buried by all the other 100+ GitHub notifications.