perl cookbook has hard coded directory with version and doesn't work on RHEL6

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Blocker Blocker
  • Resolution: Fixed
  • Component/s: perl
  • Labels:
  • Environment:

    RHEL 6

Description

The perl community opscode cookbook tries to specify the wrong directory on RHEL 6. I worked around it with

when "centos","redhat"
if node[:platform_version] > "6"
path "/usr/share/perl5/Config.pm"

But I've yet to verify RHEL5, centos or SL.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Joseph Anthony Pasquale Holsten added a comment -

Vincent: Could you verify that my patch works for you:
https://github.com/opscode-cookbooks/perl/pull/1

Show
Joseph Anthony Pasquale Holsten added a comment - Vincent: Could you verify that my patch works for you: https://github.com/opscode-cookbooks/perl/pull/1
Hide
Permalink
Vincent Janelle added a comment -

I believe I typo'd this path and it should be perl5/CPAN/Config.pm.

Show
Vincent Janelle added a comment - I believe I typo'd this path and it should be perl5/CPAN/Config.pm.
Hide
Permalink
Lamont Granquist added a comment -

working on a patch here:

https://github.com/lamont-opscode/cookbooks/blob/COOK-1129/perl/recipes/default.rb

or pull from:

https://github.com/lamont-opscode/cookbooks/tree/COOK-1129

also working around all the rest of the platform version issues with this cookbook.

currently got it smoke tested by vincent, need to do some more testing since i changed everything...

Show
Lamont Granquist added a comment - working on a patch here: https://github.com/lamont-opscode/cookbooks/blob/COOK-1129/perl/recipes/default.rb or pull from: https://github.com/lamont-opscode/cookbooks/tree/COOK-1129 also working around all the rest of the platform version issues with this cookbook. currently got it smoke tested by vincent, need to do some more testing since i changed everything...
Hide
Permalink
Lamont Granquist added a comment -

forked the single-cookbook repo version:

https://github.com/lamont-opscode/perl/tree/COOK-1129

updated the version number, and metadata, will update the README and such

still needs moar testing... has only had one smoke test on centos6

Show
Lamont Granquist added a comment - forked the single-cookbook repo version: https://github.com/lamont-opscode/perl/tree/COOK-1129 updated the version number, and metadata, will update the README and such still needs moar testing... has only had one smoke test on centos6
Hide
Permalink
Lamont Granquist added a comment -

re-opening and reassinging to myself

Show
Lamont Granquist added a comment - re-opening and reassinging to myself
Hide
Permalink
Lamont Granquist added a comment -

tested on centos 6.0, centos 5.5, and ubuntu 11.04

Show
Lamont Granquist added a comment - tested on centos 6.0, centos 5.5, and ubuntu 11.04
Hide
Permalink
Bryan McLellan [Opscode] added a comment -

https://github.com/lamont-opscode/perl/compare/83c2f7130cb92b28617fadf92fcbef7331586e49...COOK-1129

Marked to be merged.

We'll probably drop the raises on unsupported operating system. The metadata should enforce this, even though it doesn't currently.

Show
Bryan McLellan [Opscode] added a comment - https://github.com/lamont-opscode/perl/compare/83c2f7130cb92b28617fadf92fcbef7331586e49...COOK-1129 Marked to be merged. We'll probably drop the raises on unsupported operating system. The metadata should enforce this, even though it doesn't currently.
Hide
Permalink
Lamont Granquist added a comment -

Thanks brian, and yes, fixing the metadata would be way better than littering code with raises...

Show
Lamont Granquist added a comment - Thanks brian, and yes, fixing the metadata would be way better than littering code with raises...
Hide
Permalink
jtimberman added a comment -

This is merged. Thanks folks!

Show
jtimberman added a comment - This is merged. Thanks folks!
Hide
Permalink
jtimberman added a comment -

Released in perl v1.0.0

Show
jtimberman added a comment - Released in perl v1.0.0

People

  • Assignee:
    Lamont Granquist
    Reporter:
    Vincent Janelle
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: