[Mageia-sysadm] [480] Fix perms

Fri Nov 26 19:54:27 CET 2010

On Fri, Nov 26, 2010 at 04:36:45PM +0100, root at mageia.org wrote:
+> Revision: 480
+> Author:   dmorgan
+> Date:     2010-11-26 16:36:45 +0100 (Fri, 26 Nov 2010)
+> Log Message:
+> -----------
+> Fix perms
+> 
+> Modified Paths:
+> --------------
+>     puppet/modules/bugzilla/manifests/init.pp
+> 
+> Modified: puppet/modules/bugzilla/manifests/init.pp
+> ===================================================================
+> --- puppet/modules/bugzilla/manifests/init.pp	2010-11-26 15:21:37 UTC (rev 479)
+> +++ puppet/modules/bugzilla/manifests/init.pp	2010-11-26 15:36:45 UTC (rev 480)
+> @@ -33,18 +33,17 @@
+>      apache::vhost_other_app { "bugs.$domain":
+>        vhost_file => "bugzilla/vhost_bugs.conf",
+>      }
+> +    subversion::snapshot { $bugzilla_location:
+> +      source => "svn://svn.mageia.org/svn/web/templates/bugzilla/trunk"
+> +    }
+>  
+>      file { "custom":
+>        path => "/usr/share/bugzilla/template/en/custom",
+
+I think you should reuse $bugzilla_location here too.
+
+>        ensure => directory,
+>        owner => root,
+>        group => apache,
+> -      mode => 700,
+> +      mode => 640,
+>        recurse => true,
+>        require => Subversion::Snapshot[$bugzilla_location]
+>      }
+> -
+> -    subversion::snapshot { $bugzilla_location:
+> -      source => "svn://svn.mageia.org/svn/web/templates/bugzilla/trunk"
+> -    }
+>  }
+
+There is also more changes than fixing perms.
+
+-- 
+Michael Scherer
+