On Thu, Jul 10, 2008 at 09:08:33AM +0100, Daniel P. Berrange wrote: > On Thu, Jul 10, 2008 at 12:07:18AM +0100, John Levon wrote: > > > > Why is it NULL instead of the path to pygrub? On Solaris, the path is > > different. > > Loader is refering specifically to the <loader> tag which is a BIOS > firmware blob. virt-install code seems to disagree: elif self.boot_caps.loader == "pygrub" or (self.boot_caps.loader is None and self.boot_caps.type == "xen"): osblob += " </os>\n" osblob += " <bootloader>/usr/bin/pygrub</bootloader>" Now, if that's an error, that's fine. I have a patch on the way that fixes up the pygrub path based upon caps.host.arch when available (as this is always i86pc on Solaris). The other Daniel wrote: > (xend in that case), but the information is not provided there, are > you suggesting to check a few locations on the filesystem ? Hard-coding it in libvirt would be fine (in much the same way you do for emulator and HVM loader today). My patch to virt-install (hopefully) fixes the code to actually query capabilities for these values, pygrub excepted. regards john -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list