node.js manifests/params.pp & manifests/init.pp should support Scientific Linux about puppet-nodejs

[stefanlasiewski]

I managed to workaround this with the following changes:

root@sl6:/etc/puppet/modules/nodejs # diff manifests/params.pp manifests/params.pp.orig 
26c26
<     'RedHat', 'CentOS', 'OEL', 'OracleLinux', 'Scientific', 'Scientific Linux': {
---
>     'RedHat', 'CentOS', 'OEL', 'OracleLinux': {
root@sl6:/etc/puppet/modules/nodejs # diff manifests/init.pp manifests/init.pp.orig 
42c42
<     'Fedora', 'RedHat', 'CentOS', 'OEL', 'OracleLinux', 'Amazon', 'Scientific', 'Scientific Linux': {
---
>     'Fedora', 'RedHat', 'CentOS', 'OEL', 'OracleLinux', 'Amazon': {
root@sl6:/etc/puppet/modules/nodejs # puppet apply /etc/puppet/modules/nodejs/tests/init.pp

Notice: Finished catalog run in 1.15 seconds

root@sl6:/etc/puppet/modules/nodejs #

[stefanlasiewski]

And I committed a fix to my fork on Github.

stefanlasiewski@b4ae697

[haus]

It looks like another fix here would be to use osfamily instead of operatingsystem. Then osfamily could be redhat and catch all of those flavors.

[eigenhombre]

+1, I just got bit by this as well.

[juniorsysadmin]

Scientific Linux should now work, but I haven't tested and can only base this opinion on unscientific evidence.