Seth Haines wrote:
> 
> While trying to troubleshoot a SUSE 10 cloning issue I noticed that some
> of the default commands in the autoinstall.master script under the
> System Configuration Section aren't playing out quite right.
> 
> Under the:
> # Save original boot loader information
> 
> There's a for loop that looks at /etc/grub.conf /etc/lilo.conf and
> /boot/grub/menu.lst, but it's looking at the non-chrooted /boot and /etc
> directories. Using /a/ within the loop seems to clear that up and create
> the .pre-systemconfigurator saves.
> 
> Just an observation.

Good catch Seth! your observation seems correct. I wonder who wrote that
crap^H^H^H^H wrong code:

svn blame etc/autoinstallscript.template
...
  4278     arighi # Save original boot-loader configurations before running 
systemconfigurator
  4278     arighi for file in /boot/grub/menu.lst /etc/lilo.conf 
/etc/grub.conf; do
  4278     arighi     if [ -e ${file} ]; then
  4278     arighi         if [ ! -e ${file}.pre-systemimager ]; then
  4278     arighi             cp -f ${file} ${file}.pre-systemimager
  4278     arighi         fi
  4278     arighi     fi
  4278     arighi done
...

yep! that's me... :-P

Fixed in trunk. It'll be available in the next SI release.

Thanks for reporting,
-Andrea

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
sisuite-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sisuite-users

Reply via email to