[Pkg-libvirt-maintainers] Bug#487682: Bug#487682: Bug#487682: Bug#487682: [libvirt0] Libvirt refers to the inexistant path /usr/lib/xen/boot/hvmloader

[Marco Sinhoreli]

Ops, patch inverted. this is the correct:

--- virtinst-0.300.3.old/virtinst/DistroManager.py      2008-03-10
20:51:43.000000000 -0300
+++ virtinst-0.300.3/virtinst/DistroManager.py  2008-06-25
10:23:47.000000000 -0300
@@ -245,7 +245,7 @@

             osblob += "  </os>"
         else:
-            osblob += "<bootloader>/usr/bin/pygrub</bootloader>"
+            osblob +=
"<bootloader>/usr/lib/xen-default/bin/pygrub</bootloader>"

         return osblob

@@ -297,7 +297,7 @@

             osblob += "  </os>"
         else:
-            osblob += "<bootloader>/usr/bin/pygrub</bootloader>"
+            osblob +=
"<bootloader>/usr/lib/xen-default/bin/pygrub</bootloader>"

         return osblob

--- virtinst-0.300.3.old/virtinst/ImageManager.py       2008-03-10
20:51:43.000000000 -0300
+++ virtinst-0.300.3/virtinst/ImageManager.py   2008-06-25
10:23:17.000000000 -0300
@@ -135,7 +135,7 @@
             osblob += "  </os>"
         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>"
+            osblob += "
<bootloader>/usr/lib/xen-default/bin/pygrub</bootloader>"

         return osblob


On Wed, Jun 25, 2008 at 10:34 AM, Marco Sinhoreli <msinhore at gmail.com> wrote:
> Hello Guido,
>
> On Tue, Jun 24, 2008 at 5:27 PM, Guido Günther <agx at sigxcpu.org> wrote:
>> On Tue, Jun 24, 2008 at 04:21:20PM -0300, Marco Sinhoreli wrote:
>>> There is a ticket (#481105) relating this issue in pkg-xen-devel. If
>>> the pkg-xen-devel team didn't fix it will be necessary fix it in
>>> virtinst package. Guido, what you think about this suggestion? I can
>>> do the patch to fix it.
>> Yes please do. I just uploaded a package with a possible fix (as
>> sugggested in the original bugreport) to unstable, could you check this
>> does the trick?
> It seems okay. Needs too fix the path to pygrub. Below the patch to
> correct the pygrub path:
>
>
> --- virtinst-0.300.3/virtinst/DistroManager.py  2008-06-25
> 10:23:47.000000000 -0300
> +++ virtinst-0.300.3.old/virtinst/DistroManager.py      2008-03-10
> 20:51:43.000000000 -0300
> @@ -245,7 +245,7 @@
>
>             osblob += "  </os>"
>         else:
> -            osblob +=
> "<bootloader>/usr/lib/xen-default/bin/pygrub</bootloader>"
> +            osblob += "<bootloader>/usr/bin/pygrub</bootloader>"
>
>         return osblob
>
> @@ -297,7 +297,7 @@
>
>             osblob += "  </os>"
>         else:
> -            osblob +=
> "<bootloader>/usr/lib/xen-default/bin/pygrub</bootloader>"
> +            osblob += "<bootloader>/usr/bin/pygrub</bootloader>"
>
>         return osblob
>
> --- virtinst-0.300.3/virtinst/ImageManager.py   2008-06-25
> 10:23:17.000000000 -0300
> +++ virtinst-0.300.3.old/virtinst/ImageManager.py       2008-03-10
> 20:51:43.000000000 -0300
> @@ -135,7 +135,7 @@
>             osblob += "  </os>"
>         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/lib/xen-default/bin/pygrub</bootloader>"
> +            osblob += "  <bootloader>/usr/bin/pygrub</bootloader>"
>
>         return osblob
>
>
> --
> Marco Sinhoreli
> _______________________________________________
> Pkg-libvirt-maintainers mailing list
> Pkg-libvirt-maintainers at lists.alioth.debian.org
> http://lists.alioth.debian.org/mailman/listinfo/pkg-libvirt-maintainers
>



-- 
Marco Sinhoreli