Christoph Egger
2009-Jul-02 10:14 UTC
[Xen-devel] [PATCH] ioemu: portability fixes for qemu-ifup
Hi! Attached patch applies portability fixes to ioemu''s qemu-ifup script: - rename qemu-ifup to qemu-ifup-Linux - add qemu-ifup-NetBSD for NetBSD - add qemu-ifup which calls one of above. Signed-off-by: Christoph Egger <Christoph.Egger@amd.com> -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Karl-Hammerschmidt-Str. 34, 85609 Dornach b. Muenchen Geschaeftsfuehrer: Thomas M. McCoy, Giuliano Meroni Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
Ian Jackson
2009-Jul-08 13:36 UTC
[Xen-devel] Re: [PATCH] ioemu: portability fixes for qemu-ifup
Christoph Egger writes ("[PATCH] ioemu: portability fixes for
qemu-ifup"):> Attached patch applies portability fixes to ioemu''s qemu-ifup
script:
> - rename qemu-ifup to qemu-ifup-Linux
> - add qemu-ifup-NetBSD for NetBSD
> - add qemu-ifup which calls one of above.
This is quite ugly. Why would we install both of them on the same
computer and then switch at runtime ? Surely the build system should
pick the right one and install just that under the conventional name ?
Ian.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel
Christoph Egger
2009-Jul-09 14:19 UTC
[Xen-devel] Re: [PATCH] ioemu: portability fixes for qemu-ifup
On Wednesday 08 July 2009 15:36:43 Ian Jackson wrote:> Christoph Egger writes ("[PATCH] ioemu: portability fixes for qemu-ifup"): > > Attached patch applies portability fixes to ioemu''s qemu-ifup script: > > - rename qemu-ifup to qemu-ifup-Linux > > - add qemu-ifup-NetBSD for NetBSD > > - add qemu-ifup which calls one of above. > > This is quite ugly. Why would we install both of them on the same > computer and then switch at runtime ? Surely the build system should > pick the right one and install just that under the conventional name ?Good idea. Attached patch does this. Signed-off-by: Christoph Egger <Christoph.Egger@amd.com> -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Karl-Hammerschmidt-Str. 34, 85609 Dornach b. Muenchen Geschaeftsfuehrer: Thomas M. McCoy, Giuliano Meroni Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632 _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel