Dustin Kamper wrote:
> In the below, I made a reference to $overrides.  This should have been
> $default_override_dir.  The variable for $default_override_dir is not in
> Server.pm <http://Server.pm> and perhaps should be yanked in with the
> same method that si_getimage uses as per below:
> my $default_override_dir    = $config->default_override_dir();
> 

Dustin,

correct! I think the following patch should fix the problem. This is already in
the trunk and it'll be available from the next release.

Thanks,
-Andrea

Index: lib/SystemImager/Server.pm
===================================================================
--- lib/SystemImager/Server.pm  (revision 4216)
+++ lib/SystemImager/Server.pm  (working copy)
@@ -14,6 +14,7 @@
 use File::Copy;
 use File::Path;
 use XML::Simple;
+use SystemImager::Config qw($config);
 use vars qw($VERSION @mount_points %device_by_mount_point 
%filesystem_type_by_mount_point $disk_no %dev2disk);
 
 $VERSION="SYSTEMIMAGER_VERSION_STRING";
@@ -2051,7 +2052,8 @@
     ### BEGIN overrides stuff ###
     # Create default overrides directory. -BEF-
     #
-    my $dir = "/var/lib/systemimager/overrides/$script_name";
+    my $override_dir = $config->default_override_dir;
+    my $dir = "$override_dir/$script_name";
     if (! -d "$dir")  {
       mkdir("$dir", 0755) or die "FATAL: Can't make directory $dir\n";
     }  

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
sisuite-devel mailing list
sisuite-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sisuite-devel

Reply via email to