On Mon, Jun 12, 2017 at 7:55 PM Sam Ruby <ru...@intertwingly.net> wrote:

> On Mon, Jun 12, 2017 at 7:44 PM,  <johndam...@apache.org> wrote:
> > ---
> >  lib/whimsy/asf/svn.rb         | 11 +++++++++++
> >  www/roster/public_podlings.rb |  7 ++++++-
> >  2 files changed, 17 insertions(+), 1 deletion(-)
> >
> > diff --git a/lib/whimsy/asf/svn.rb b/lib/whimsy/asf/svn.rb
> > index 134609c..64a596e 100644
> > --- a/lib/whimsy/asf/svn.rb
> > +++ b/lib/whimsy/asf/svn.rb
> > @@ -141,6 +141,17 @@ module ASF
> >        return revision, content
> >      end
> >
> > +    def self.updateSimple(path)
> > +      cmd = ['svn', 'update', path, '--non-interactive']
>
> This will undoubtedly fail as the $apache::user (www-data) does not
> have write access to those directories.
>

Err so should we run cron as whimsysvn ?


>
>
> https://github.com/apache/infrastructure-puppet/blob/deployment/modules/whimsy_server/manifests/cronjobs.pp#L89
>
>
> https://github.com/apache/infrastructure-puppet/blob/deployment/modules/whimsy_server/manifests/init.pp#L116
>
> - Sam Ruby
>

Reply via email to