diff --git a/manifests/setup.pp b/manifests/setup.pp
index fbc6e51..1a6af64 100644
--- a/manifests/setup.pp
+++ b/manifests/setup.pp
@@ -1,64 +1,64 @@
 # === Class: concat::setup
 #
 # Sets up the concat system. This is a private class.
 #
 # [$concatdir]
 #   is where the fragments live and is set on the fact concat_basedir.
 #   Since puppet should always manage files in $concatdir and they should
 #   not be deleted ever, /tmp is not an option.
 #
 # It also copies out the concatfragments.{sh,rb} file to ${concatdir}/bin
 #
 class concat::setup {
   if $caller_module_name != $module_name {
     warning("${name} is deprecated as a public API of the ${module_name} module and should no longer be directly included in the manifest.")
   }
 
   if $::concat_basedir {
     $concatdir = $::concat_basedir
   } else {
     fail ('$concat_basedir not defined. Try running again with pluginsync=true on the [master] and/or [main] section of your node\'s \'/etc/puppet/puppet.conf\'.')
   }
 
   # owner and mode of fragment files (on windows owner and access rights should
   # be inherited from concatdir and not explicitly set to avoid problems)
   $fragment_owner = $::osfamily ? { 'windows' => undef, default => $::id }
   $fragment_mode  = $::osfamily ? { 'windows' => undef, default => '0640' }
 
   # PR #174 introduced changes to the concatfragments.sh script that are
   # incompatible with Solaris 10 but reportedly OK on Solaris 11.  As a work
   # around we are enable the .rb concat script on all Solaris versions.  If
   # this goes smoothly, we should move towards completely eliminating the .sh
   # version.
   $script_name = $::osfamily? {
-    /(Windows|Solaris)/ => 'concatfragments.rb',
-    default             => 'concatfragments.sh'
+    /(?i:(Windows|Solaris))/ => 'concatfragments.rb',
+    default                  => 'concatfragments.sh'
   }
 
   $script_path = "${concatdir}/bin/${script_name}"
 
   $script_owner = $::osfamily ? { 'windows' => undef, default => $::id }
 
   $script_mode = $::osfamily ? { 'windows' => undef, default => '0755' }
 
   $script_command = $::osfamily? {
     'windows' => "ruby.exe '${script_path}'",
     default   => $script_path
   }
 
   File {
     backup => false,
   }
 
   file { $script_path:
     ensure => file,
     owner  => $script_owner,
     mode   => $script_mode,
     source => "puppet:///modules/concat/${script_name}",
   }
 
   file { [ $concatdir, "${concatdir}/bin" ]:
     ensure => directory,
     mode   => '0755',
   }
 }
diff --git a/spec/unit/classes/concat_setup_spec.rb b/spec/unit/classes/concat_setup_spec.rb
index 4e83cb2..3096e73 100644
--- a/spec/unit/classes/concat_setup_spec.rb
+++ b/spec/unit/classes/concat_setup_spec.rb
@@ -1,84 +1,84 @@
 require 'spec_helper'
 
 describe 'concat::setup', :type => :class do
 
   shared_examples 'setup' do |concatdir|
     concatdir = '/foo' if concatdir.nil?
 
     let(:facts) {{ :concat_basedir => concatdir }}
 
     it do
       should contain_file("#{concatdir}/bin/concatfragments.sh").with({
         :mode   => '0755',
         :source => 'puppet:///modules/concat/concatfragments.sh',
         :backup => false,
       })
     end
 
     [concatdir, "#{concatdir}/bin"].each do |file|
       it do
         should contain_file(file).with({
           :ensure => 'directory',
           :mode   => '0755',
           :backup => false,
         })
       end
     end
   end
 
   context 'facts' do
     context 'concat_basedir =>' do
       context '/foo' do
         it_behaves_like 'setup', '/foo'
       end
     end
   end # facts
 
   context 'deprecated as a public class' do
     it 'should create a warning' do
       pending('rspec-puppet support for testing warning()')
     end
   end
 
   context "on osfamily Solaris" do
     concatdir = '/foo'
     let(:facts) do
       {
         :concat_basedir => concatdir,
         :osfamily       => 'Solaris',
         :id             => 'root',
       }
     end
 
     it do
       should contain_file("#{concatdir}/bin/concatfragments.rb").with({
         :ensure => 'file',
         :owner  => 'root',
         :mode   => '0755',
         :source => 'puppet:///modules/concat/concatfragments.rb',
         :backup => false,
       })
     end
   end # on osfamily Solaris
 
-  context "on osfamily Windows" do
+  context "on osfamily windows" do
     concatdir = '/foo'
     let(:facts) do
       {
         :concat_basedir => concatdir,
-        :osfamily       => 'Windows',
+        :osfamily       => 'windows',
         :id             => 'batman',
       }
     end
 
     it do
       should contain_file("#{concatdir}/bin/concatfragments.rb").with({
         :ensure => 'file',
         :owner  => nil,
         :mode   => nil,
         :source => 'puppet:///modules/concat/concatfragments.rb',
         :backup => false,
       })
     end
-  end # on osfamily Windows
+  end # on osfamily windows
 end