* tools: build error @ 2010-09-22 12:59 Christoph Egger 2010-09-22 17:27 ` Gianni Tedesco 0 siblings, 1 reply; 6+ messages in thread From: Christoph Egger @ 2010-09-22 12:59 UTC (permalink / raw) To: xen-devel Hi! Changeset 22167:330c4d9010da breaks the build. It adds blktapctl to link against in setup.py which isn't build on NetBSD. Christoph -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach b. Muenchen Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: tools: build error 2010-09-22 12:59 tools: build error Christoph Egger @ 2010-09-22 17:27 ` Gianni Tedesco 2010-09-23 11:51 ` Christoph Egger 0 siblings, 1 reply; 6+ messages in thread From: Gianni Tedesco @ 2010-09-22 17:27 UTC (permalink / raw) To: Christoph Egger; +Cc: Ian, xen-devel@lists.xensource.com, Jackson On Wed, 2010-09-22 at 13:59 +0100, Christoph Egger wrote: > Hi! > > Changeset 22167:330c4d9010da breaks the build. > > It adds blktapctl to link against in setup.py which isn't build > on NetBSD. Oops, Following ought to fix it: ----8<------------------------------------------------------ Link to libblktapctl in python setup only for platforms which build it Signed-off-by: Gianni Tedesco <gianni.tedesco@citrix.com> diff -r 36420e35c65a tools/python/setup.py --- a/tools/python/setup.py Wed Sep 22 16:57:12 2010 +0100 +++ b/tools/python/setup.py Wed Sep 22 18:25:22 2010 +0100 @@ -14,8 +14,7 @@ include_dirs = [ XEN_ROOT + "/tools/libx library_dirs = [ XEN_ROOT + "/tools/libxc", XEN_ROOT + "/tools/xenstore", - XEN_ROOT + "/tools/libxl", - XEN_ROOT + "/tools/blktap2/control", + XEN_ROOT + "/tools/libxl" ] libraries = [ "xenctrl", "xenguest", "xenstore" ] @@ -23,8 +22,11 @@ libraries = [ "xenctrl", "xenguest", "xe plat = os.uname()[0] if plat == 'Linux': uuid_libs = ["uuid"] + blktap_ctl_libs = ["blktapctl"] + library_dirs.append(XEN_ROOT + "/tools/blktap2/control") else: uuid_libs = [] + blktap_ctl_libs = [] xc = Extension("xc", extra_compile_args = extra_compile_args, @@ -96,7 +98,7 @@ xl = Extension("xl", extra_compile_args = extra_compile_args, include_dirs = include_dirs + [ "xen/lowlevel/xl" ], library_dirs = library_dirs, - libraries = libraries + ["xenlight", "blktapctl" ] + uuid_libs, + libraries = libraries + ["xenlight" ] + blktap_ctl_libs + uuid_libs, sources = [ "xen/lowlevel/xl/xl.c", "xen/lowlevel/xl/_pyxl_types.c" ]) modules = [ xc, xs, ptsname, acm, flask, xl ] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: tools: build error 2010-09-22 17:27 ` Gianni Tedesco @ 2010-09-23 11:51 ` Christoph Egger 2010-09-23 15:30 ` Gianni Tedesco 0 siblings, 1 reply; 6+ messages in thread From: Christoph Egger @ 2010-09-23 11:51 UTC (permalink / raw) To: Gianni Tedesco; +Cc: xen-devel@lists.xensource.com, Ian Jackson On Wednesday 22 September 2010 19:27:22 Gianni Tedesco wrote: > On Wed, 2010-09-22 at 13:59 +0100, Christoph Egger wrote: > > Hi! > > > > Changeset 22167:330c4d9010da breaks the build. > > > > It adds blktapctl to link against in setup.py which isn't build > > on NetBSD. > > Oops, Following ought to fix it: Confirmed, this fixes the build error. I am just wondering why do you want to link against blktapctl at all when it isn't used ? Doesn't it build/work on Linux when you don't link against blktapctl ? Christoph > > ----8<------------------------------------------------------ > Link to libblktapctl in python setup only for platforms which build it > > Signed-off-by: Gianni Tedesco <gianni.tedesco@citrix.com> > > diff -r 36420e35c65a tools/python/setup.py > --- a/tools/python/setup.py Wed Sep 22 16:57:12 2010 +0100 > +++ b/tools/python/setup.py Wed Sep 22 18:25:22 2010 +0100 > @@ -14,8 +14,7 @@ include_dirs = [ XEN_ROOT + "/tools/libx > > library_dirs = [ XEN_ROOT + "/tools/libxc", > XEN_ROOT + "/tools/xenstore", > - XEN_ROOT + "/tools/libxl", > - XEN_ROOT + "/tools/blktap2/control", > + XEN_ROOT + "/tools/libxl" > ] > > libraries = [ "xenctrl", "xenguest", "xenstore" ] > @@ -23,8 +22,11 @@ libraries = [ "xenctrl", "xenguest", "xe > plat = os.uname()[0] > if plat == 'Linux': > uuid_libs = ["uuid"] > + blktap_ctl_libs = ["blktapctl"] > + library_dirs.append(XEN_ROOT + "/tools/blktap2/control") > else: > uuid_libs = [] > + blktap_ctl_libs = [] > > xc = Extension("xc", > extra_compile_args = extra_compile_args, > @@ -96,7 +98,7 @@ xl = Extension("xl", > extra_compile_args = extra_compile_args, > include_dirs = include_dirs + [ "xen/lowlevel/xl" ], > library_dirs = library_dirs, > - libraries = libraries + ["xenlight", "blktapctl" ] > + uuid_libs, + libraries = libraries + ["xenlight" ] > + blktap_ctl_libs + uuid_libs, sources = [ > "xen/lowlevel/xl/xl.c", "xen/lowlevel/xl/_pyxl_types.c" ]) > > modules = [ xc, xs, ptsname, acm, flask, xl ] -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach b. Muenchen Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: tools: build error 2010-09-23 11:51 ` Christoph Egger @ 2010-09-23 15:30 ` Gianni Tedesco 0 siblings, 0 replies; 6+ messages in thread From: Gianni Tedesco @ 2010-09-23 15:30 UTC (permalink / raw) To: Christoph Egger; +Cc: Ian, xen-devel@lists.xensource.com, Jackson On Thu, 2010-09-23 at 12:51 +0100, Christoph Egger wrote: > On Wednesday 22 September 2010 19:27:22 Gianni Tedesco wrote: > > On Wed, 2010-09-22 at 13:59 +0100, Christoph Egger wrote: > > > Hi! > > > > > > Changeset 22167:330c4d9010da breaks the build. > > > > > > It adds blktapctl to link against in setup.py which isn't build > > > on NetBSD. > > > > Oops, Following ought to fix it: > > Confirmed, this fixes the build error. I am just wondering why do you want to > link against blktapctl at all when it isn't used ? > > Doesn't it build/work on Linux when you don't link against blktapctl ? Actually you're probably right since libxenlight.so is linking against blktapctl for it's own uses however the extra link in xl.so is harmless (modulo the portability error I just fixed). ^ permalink raw reply [flat|nested] 6+ messages in thread
* tools: build error @ 2010-02-22 18:15 Christoph Egger 2010-02-22 18:43 ` Keir Fraser 0 siblings, 1 reply; 6+ messages in thread From: Christoph Egger @ 2010-02-22 18:15 UTC (permalink / raw) To: xen-devel Hi! Compiling tools/firmware/hvmloader/acpi fails with: head -n -1 dsdt.asl >dsdt_15cpu.asl head: illegal line count -- -1 Christoph -- ---to satisfy European Law for business letters: Advanced Micro Devices GmbH Karl-Hammerschmidt-Str. 34, 85609 Dornach b. Muenchen Geschaeftsfuehrer: Andrew Bowd, Thomas M. McCoy, Giuliano Meroni Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen Registergericht Muenchen, HRB Nr. 43632 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: tools: build error 2010-02-22 18:15 Christoph Egger @ 2010-02-22 18:43 ` Keir Fraser 0 siblings, 0 replies; 6+ messages in thread From: Keir Fraser @ 2010-02-22 18:43 UTC (permalink / raw) To: Christoph Egger, xen-devel@lists.xensource.com On 22/02/2010 18:15, "Christoph Egger" <Christoph.Egger@amd.com> wrote: > Compiling tools/firmware/hvmloader/acpi fails with: > > head -n -1 dsdt.asl >dsdt_15cpu.asl > head: illegal line count -- -1 Fixed by c/s 20965. Thanks, Keir ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2010-09-23 15:30 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2010-09-22 12:59 tools: build error Christoph Egger 2010-09-22 17:27 ` Gianni Tedesco 2010-09-23 11:51 ` Christoph Egger 2010-09-23 15:30 ` Gianni Tedesco -- strict thread matches above, loose matches on Subject: below -- 2010-02-22 18:15 Christoph Egger 2010-02-22 18:43 ` Keir Fraser
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).