[prev in list] [next in list] [prev in thread] [next in thread] 

List:       webmin-devel
Subject:    Re: [webmin-devel] Backup Configuration Module
From:       Jamie Cameron <jcameron () webmin ! com>
Date:       2004-10-13 10:06:01
Message-ID: 1097652178.14245.123.camel () temasek ! home
[Download RAW message or body]

On Wed, 2004-10-13 at 16:23, Martin Mewes wrote:
> Dear Jamie,
> 
> this is what happens when I try to invoke the Backup Module for Configuration Data.
> 
> > Fehler
> > -----
> > require bind8/backup_config.pl failed : <pre>syntax error at \
> > /usr/libexec/webmin/bind8/backup_config.pl line 61, near ") {"
> > syntax error at /usr/libexec/webmin/bind8/backup_config.pl line 65, near
> > "}"
> > </pre>
> > -----
> 
> Looking at the code
> 
> # post_restore(&files)
> # Called after the files are restored from a backup
> sub post_restore
> {
> local $pidfile = &get_pid_file();
> if (&check_pid_file(&make_chroot($pidfile)) {
> return &restart_bind();
> }
> return undef;
> }
> 
> I think it should be
> 
> if (&check_pid_file(&make_chroot($pidfile))) {
> 
> because you open three '(' but only closing two ')'
> 
> I manually changed the code and after a test at my place everything works ok now.

Thanks for pointing that out .. I'll fix it for the next development
release.

 - Jamie




-------------------------------------------------------
This SF.net email is sponsored by: IT Product Guide on ITManagersJournal
Use IT products in your business? Tell us what you think of them. Give us
Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more
http://productguide.itmanagersjournal.com/guidepromo.tmpl
-
Forwarded by the Webmin development list at webmin-devel@webmin.com
To remove yourself from this list, go to
http://lists.sourceforge.net/lists/listinfo/webadmin-devel


[prev in list] [next in list] [prev in thread] [next in thread]