Browse files

make foodcritic happy again; move the cron include in the cron recipe…

…; use platform_family instead of platform to catch more platforms here
  • Loading branch information...
mdxp committed Feb 12, 2013
1 parent ede30db commit 50d11cfb5ee3c947945e198c953c8c4540b9bda5
Showing with 4 additions and 3 deletions.
  1. +2 −0 recipes/cron.rb
  2. +2 −3 recipes/default.rb
@@ -18,6 +18,8 @@
# limitations under the License.
+include_recipe "cron"
cron "drupal hourly cron" do
command "cd #{node['drupal']['dir']}; /usr/bin/php cron.php"
minute "0"
@@ -21,12 +21,11 @@
include_recipe %w{apache2 apache2::mod_php5 apache2::mod_rewrite apache2::mod_expires}
include_recipe %w{php php::module_mysql php::module_gd}
include_recipe "postfix"
-include_recipe "cron"
include_recipe "drupal::drush"
# Centos does not include the php-dom extension in it's minimal php install.
-case node[:platform]
-when "centos"
+case node['platform_family']
+when 'rhel', 'fedora'
package 'php-dom' do
action :install

0 comments on commit 50d11cf

Please sign in to comment.