On 13 June 2017 at 00:59, John D. Ament <johndam...@apache.org> wrote: > 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 ? >
If it's important to keep the files uptodate, why not use svnpubsub instead of cron for these files? >> >> >> 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 >>
