All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] hwloc: Add
@ 2016-12-22 20:02 Haris Okanovic
  2016-12-22 21:34 ` Martin Jansa
                   ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: Haris Okanovic @ 2016-12-22 20:02 UTC (permalink / raw)
  To: openembedded-devel; +Cc: haris.okanovic

https://www.open-mpi.org/projects/hwloc/

"The Portable Hardware Locality (hwloc) software package provides a
portable abstraction (across OS, versions, architectures, ...) of the
hierarchical topology of modern architectures, including NUMA memory
nodes, sockets, shared caches, cores and simultaneous multithreading.
It also gathers various system attributes such as cache and memory
information as well as the locality of I/O devices such as network
interfaces, InfiniBand HCAs or GPUs."

This recipe adds libhwloc and lstopo utility as two packages.

Testing: Built in OE/Krogoth. Installed to x86_64 machine. Successfully
ran `lstopo` and verified it printed correct CPU and cache info.

Signed-off-by: Haris Okanovic <haris.okanovic@ni.com>
---
 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)
 create mode 100644 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb

diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
new file mode 100644
index 0000000..3a88f5b
--- /dev/null
+++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
@@ -0,0 +1,25 @@
+SUMMARY = "The Portable Hardware Locality (hwloc) software package \
+ provides a portable abstraction of the hierarchical topology of modern \
+ architectures."
+HOMEPAGE = "https://www.open-mpi.org/software/hwloc/"
+SECTION = "base"
+LICENSE = "BSD"
+LIC_FILES_CHKSUM = "file://COPYING;md5=3282e20dc3cec311deda3c6d4b1f990b"
+
+SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/downloads/${BP}.tar.bz2"
+SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
+
+inherit autotools
+
+# Split hwloc library into separate subpackage
+PACKAGES_prepend = "lib${PN}"
+FILES_lib${PN} += "${libdir}/lib${PN}.so*"
+${PN}_RDEPENDS += "lib${PN}"
+
+do_install_append() {
+    # Let ldconfig do it's thing, no need for these symlinks
+    test -L "${D}${libdir}/lib${PN}.so.5"
+    test -L "${D}${libdir}/lib${PN}.so"
+    rm "${D}${libdir}/lib${PN}.so.5"
+    rm "${D}${libdir}/lib${PN}.so"
+}
-- 
2.10.1



^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH] hwloc: Add
  2016-12-22 20:02 [PATCH] hwloc: Add Haris Okanovic
@ 2016-12-22 21:34 ` Martin Jansa
  2016-12-22 22:35   ` Christopher Larson
  2016-12-26 10:35   ` Andre McCurdy
  2016-12-27 19:06 ` [meta-oe][PATCH v2] " Haris Okanovic
  2017-05-19 16:17 ` [PATCH v4] " Haris Okanovic
  2 siblings, 2 replies; 12+ messages in thread
From: Martin Jansa @ 2016-12-22 21:34 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Haris Okanovic

SUMMARY is for short summary, put long one into DESCRIPTION and create
short one for SUMMARY.

Also missing sha265sum, so I doubt it was tested properly.

On Thu, Dec 22, 2016 at 9:02 PM, Haris Okanovic <haris.okanovic@ni.com>
wrote:

> https://www.open-mpi.org/projects/hwloc/
>
> "The Portable Hardware Locality (hwloc) software package provides a
> portable abstraction (across OS, versions, architectures, ...) of the
> hierarchical topology of modern architectures, including NUMA memory
> nodes, sockets, shared caches, cores and simultaneous multithreading.
> It also gathers various system attributes such as cache and memory
> information as well as the locality of I/O devices such as network
> interfaces, InfiniBand HCAs or GPUs."
>
> This recipe adds libhwloc and lstopo utility as two packages.
>
> Testing: Built in OE/Krogoth. Installed to x86_64 machine. Successfully
> ran `lstopo` and verified it printed correct CPU and cache info.
>
> Signed-off-by: Haris Okanovic <haris.okanovic@ni.com>
> ---
>  meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb | 25
> +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>  create mode 100644 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
>
> diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> new file mode 100644
> index 0000000..3a88f5b
> --- /dev/null
> +++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> @@ -0,0 +1,25 @@
> +SUMMARY = "The Portable Hardware Locality (hwloc) software package \
> + provides a portable abstraction of the hierarchical topology of modern \
> + architectures."
> +HOMEPAGE = "https://www.open-mpi.org/software/hwloc/"
> +SECTION = "base"
> +LICENSE = "BSD"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=3282e20dc3cec311deda3c6d4b1f990b"
> +
> +SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/
> downloads/${BP}.tar.bz2"
> +SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
> +
> +inherit autotools
> +
> +# Split hwloc library into separate subpackage
> +PACKAGES_prepend = "lib${PN}"
> +FILES_lib${PN} += "${libdir}/lib${PN}.so*"
> +${PN}_RDEPENDS += "lib${PN}"
> +
> +do_install_append() {
> +    # Let ldconfig do it's thing, no need for these symlinks
> +    test -L "${D}${libdir}/lib${PN}.so.5"
> +    test -L "${D}${libdir}/lib${PN}.so"
> +    rm "${D}${libdir}/lib${PN}.so.5"
> +    rm "${D}${libdir}/lib${PN}.so"
> +}
> --
> 2.10.1
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] hwloc: Add
  2016-12-22 21:34 ` Martin Jansa
@ 2016-12-22 22:35   ` Christopher Larson
  2016-12-22 22:56     ` Khem Raj
  2016-12-26 10:35   ` Andre McCurdy
  1 sibling, 1 reply; 12+ messages in thread
From: Christopher Larson @ 2016-12-22 22:35 UTC (permalink / raw)
  To: Openembedded Discussion; +Cc: Haris Okanovic

On Thu, Dec 22, 2016 at 2:34 PM, Martin Jansa <martin.jansa@gmail.com>
wrote:

> SUMMARY is for short summary, put long one into DESCRIPTION and create
> short one for SUMMARY.
>
> Also missing sha265sum, so I doubt it was tested properly.
>
> On Thu, Dec 22, 2016 at 9:02 PM, Haris Okanovic <haris.okanovic@ni.com>
> wrote:
>
> > https://www.open-mpi.org/projects/hwloc/
> >
> > "The Portable Hardware Locality (hwloc) software package provides a
> > portable abstraction (across OS, versions, architectures, ...) of the
> > hierarchical topology of modern architectures, including NUMA memory
> > nodes, sockets, shared caches, cores and simultaneous multithreading.
> > It also gathers various system attributes such as cache and memory
> > information as well as the locality of I/O devices such as network
> > interfaces, InfiniBand HCAs or GPUs."
> >
> > This recipe adds libhwloc and lstopo utility as two packages.
> >
> > Testing: Built in OE/Krogoth. Installed to x86_64 machine. Successfully
> > ran `lstopo` and verified it printed correct CPU and cache info.
> >
> > Signed-off-by: Haris Okanovic <haris.okanovic@ni.com>
> > ---
> >  meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb | 25
> > +++++++++++++++++++++++++
> >  1 file changed, 25 insertions(+)
> >  create mode 100644 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> >
> > diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> > b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> > new file mode 100644
> > index 0000000..3a88f5b
> > --- /dev/null
> > +++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> > @@ -0,0 +1,25 @@
> > +SUMMARY = "The Portable Hardware Locality (hwloc) software package \
> > + provides a portable abstraction of the hierarchical topology of modern
> \
> > + architectures."
> > +HOMEPAGE = "https://www.open-mpi.org/software/hwloc/"
> > +SECTION = "base"
> > +LICENSE = "BSD"
> > +LIC_FILES_CHKSUM = "file://COPYING;md5=3282e20dc3cec311deda3c6d4b1f99
> 0b"
> > +
> > +SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/
> > downloads/${BP}.tar.bz2"
> > +SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
> > +
> > +inherit autotools
> > +
> > +# Split hwloc library into separate subpackage
> > +PACKAGES_prepend = "lib${PN}"
> > +FILES_lib${PN} += "${libdir}/lib${PN}.so*"
> > +${PN}_RDEPENDS += "lib${PN}"
> > +
> > +do_install_append() {
> > +    # Let ldconfig do it's thing, no need for these symlinks
> > +    test -L "${D}${libdir}/lib${PN}.so.5"
> > +    test -L "${D}${libdir}/lib${PN}.so"
> > +    rm "${D}${libdir}/lib${PN}.so.5"
> > +    rm "${D}${libdir}/lib${PN}.so"
>

This link manipulation is also inconsistent with every other recipe. It’s
not appropriate. If your distro wants to rely on ldconfig to handle the
links, then your distro could do this for all recipes rather than just one.
This isn’t how things are done in our existing recipes.
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] hwloc: Add
  2016-12-22 22:35   ` Christopher Larson
@ 2016-12-22 22:56     ` Khem Raj
  2016-12-25 23:55       ` Martin Jansa
  0 siblings, 1 reply; 12+ messages in thread
From: Khem Raj @ 2016-12-22 22:56 UTC (permalink / raw)
  To: openembeded-devel; +Cc: Haris Okanovic

On Thu, Dec 22, 2016 at 2:35 PM, Christopher Larson <clarson@kergoth.com> wrote:
> On Thu, Dec 22, 2016 at 2:34 PM, Martin Jansa <martin.jansa@gmail.com>
> wrote:
>
>> SUMMARY is for short summary, put long one into DESCRIPTION and create
>> short one for SUMMARY.
>>
>> Also missing sha265sum, so I doubt it was tested properly.
>>
>> On Thu, Dec 22, 2016 at 9:02 PM, Haris Okanovic <haris.okanovic@ni.com>
>> wrote:
>>
>> > https://www.open-mpi.org/projects/hwloc/
>> >
>> > "The Portable Hardware Locality (hwloc) software package provides a
>> > portable abstraction (across OS, versions, architectures, ...) of the
>> > hierarchical topology of modern architectures, including NUMA memory
>> > nodes, sockets, shared caches, cores and simultaneous multithreading.
>> > It also gathers various system attributes such as cache and memory
>> > information as well as the locality of I/O devices such as network
>> > interfaces, InfiniBand HCAs or GPUs."
>> >
>> > This recipe adds libhwloc and lstopo utility as two packages.
>> >
>> > Testing: Built in OE/Krogoth. Installed to x86_64 machine. Successfully
>> > ran `lstopo` and verified it printed correct CPU and cache info.
>> >
>> > Signed-off-by: Haris Okanovic <haris.okanovic@ni.com>
>> > ---
>> >  meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb | 25
>> > +++++++++++++++++++++++++
>> >  1 file changed, 25 insertions(+)
>> >  create mode 100644 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
>> >
>> > diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
>> > b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
>> > new file mode 100644
>> > index 0000000..3a88f5b
>> > --- /dev/null
>> > +++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
>> > @@ -0,0 +1,25 @@
>> > +SUMMARY = "The Portable Hardware Locality (hwloc) software package \
>> > + provides a portable abstraction of the hierarchical topology of modern
>> \
>> > + architectures."
>> > +HOMEPAGE = "https://www.open-mpi.org/software/hwloc/"
>> > +SECTION = "base"
>> > +LICENSE = "BSD"
>> > +LIC_FILES_CHKSUM = "file://COPYING;md5=3282e20dc3cec311deda3c6d4b1f99
>> 0b"
>> > +
>> > +SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/
>> > downloads/${BP}.tar.bz2"
>> > +SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
>> > +
>> > +inherit autotools
>> > +
>> > +# Split hwloc library into separate subpackage
>> > +PACKAGES_prepend = "lib${PN}"
>> > +FILES_lib${PN} += "${libdir}/lib${PN}.so*"
>> > +${PN}_RDEPENDS += "lib${PN}"
>> > +
>> > +do_install_append() {
>> > +    # Let ldconfig do it's thing, no need for these symlinks
>> > +    test -L "${D}${libdir}/lib${PN}.so.5"
>> > +    test -L "${D}${libdir}/lib${PN}.so"
>> > +    rm "${D}${libdir}/lib${PN}.so.5"
>> > +    rm "${D}${libdir}/lib${PN}.so"
>>
>
> This link manipulation is also inconsistent with every other recipe. It’s
> not appropriate. If your distro wants to rely on ldconfig to handle the
> links, then your distro could do this for all recipes rather than just one.
> This isn’t how things are done in our existing recipes.

There is global knob USE_LDCONFIG, perhaps thats what could be used
but generally I agree with Chris here.

> --
> Christopher Larson
> clarson at kergoth dot com
> Founder - BitBake, OpenEmbedded, OpenZaurus
> Maintainer - Tslib
> Senior Software Engineer, Mentor Graphics
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] hwloc: Add
  2016-12-22 22:56     ` Khem Raj
@ 2016-12-25 23:55       ` Martin Jansa
  2016-12-27 19:00         ` Haris Okanovic
  0 siblings, 1 reply; 12+ messages in thread
From: Martin Jansa @ 2016-12-25 23:55 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Haris Okanovic

Missing dependencies and PACKAGES_prepend breaks hwloc-dbg, because of
missing space at the end of the prepended value.



hwloc-1.11.5: hwloc: Files/directories were installed but not shipped
in any package:
  /usr/src
  /usr/src/debug
  /usr/src/debug/hwloc
  /usr/src/debug/hwloc/1.11.5-r0
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5
  /usr/src/debug/hwloc/1.11.5-r0/build
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-xml.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-text.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-fig.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-draw.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-color.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-cairo.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-ascii.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/misc.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-ps.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-patch.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-info.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-dump-hwdata.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-dump-hwdata-knl.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-distrib.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-distances.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-diff.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-calc.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-calc.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-bind.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-assembler.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-annotate.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/traversal.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-xml.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-xml-nolibxml.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-xml-libxml.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-x86.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-synthetic.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-pci.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-noos.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-linux.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-hardwired.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-custom.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/pci-common.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/misc.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/distances.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/diff.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/components.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/bitmap.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/bind.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/base64.c
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/xml.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/private.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/misc.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/debug.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/cpuid-x86.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/components.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/plugins.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/linux.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/inlines.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/helper.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/diff.h
  /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/bitmap.h
  /usr/src/debug/hwloc/1.11.5-r0/build/src
  /usr/src/debug/hwloc/1.11.5-r0/build/include
  /usr/src/debug/hwloc/1.11.5-r0/build/src/static-components.h
  /usr/src/debug/hwloc/1.11.5-r0/build/include/hwloc
  /usr/src/debug/hwloc/1.11.5-r0/build/include/hwloc/autogen
  /usr/src/debug/hwloc/1.11.5-r0/build/include/hwloc/autogen/config.h
Please set FILES such that these items are packaged. Alternatively if
they are unneeded, avoid installing them or delete them within
do_install.
hwloc: 75 installed and not shipped files. [installed-vs-shipped]
hwloc-1.11.5: hwloc rdepends on numactl, but it isn't a build
dependency, missing numactl in DEPENDS or PACKAGECONFIG? [build-deps]
hwloc-1.11.5: hwloc rdepends on cairo, but it isn't a build
dependency, missing cairo in DEPENDS or PACKAGECONFIG? [build-deps]
hwloc-1.11.5: hwloc rdepends on libpciaccess, but it isn't a build
dependency, missing libpciaccess in DEPENDS or PACKAGECONFIG?
[build-deps]
hwloc-1.11.5: hwloc rdepends on libx11, but it isn't a build
dependency, missing libx11 in DEPENDS or PACKAGECONFIG? [build-deps]
hwloc-1.11.5: hwloc rdepends on libcap, but it isn't a build
dependency, missing libcap in DEPENDS or PACKAGECONFIG? [build-deps]
hwloc-1.11.5: hwloc rdepends on zlib, but it isn't a build dependency,
missing zlib in DEPENDS or PACKAGECONFIG? [build-deps]
hwloc-1.11.5: hwloc rdepends on ncurses-libtinfo, but it isn't a build
dependency, missing ncurses in DEPENDS or PACKAGECONFIG? [build-deps]
hwloc-1.11.5: hwloc rdepends on libxml2, but it isn't a build
dependency, missing libxml2 in DEPENDS or PACKAGECONFIG? [build-deps]
hwloc-1.11.5: hwloc rdepends on libudev, but it isn't a build
dependency, missing systemd in DEPENDS or PACKAGECONFIG? [build-deps]


On Thu, Dec 22, 2016 at 11:56 PM, Khem Raj <raj.khem@gmail.com> wrote:

> On Thu, Dec 22, 2016 at 2:35 PM, Christopher Larson <clarson@kergoth.com>
> wrote:
> > On Thu, Dec 22, 2016 at 2:34 PM, Martin Jansa <martin.jansa@gmail.com>
> > wrote:
> >
> >> SUMMARY is for short summary, put long one into DESCRIPTION and create
> >> short one for SUMMARY.
> >>
> >> Also missing sha265sum, so I doubt it was tested properly.
> >>
> >> On Thu, Dec 22, 2016 at 9:02 PM, Haris Okanovic <haris.okanovic@ni.com>
> >> wrote:
> >>
> >> > https://www.open-mpi.org/projects/hwloc/
> >> >
> >> > "The Portable Hardware Locality (hwloc) software package provides a
> >> > portable abstraction (across OS, versions, architectures, ...) of the
> >> > hierarchical topology of modern architectures, including NUMA memory
> >> > nodes, sockets, shared caches, cores and simultaneous multithreading.
> >> > It also gathers various system attributes such as cache and memory
> >> > information as well as the locality of I/O devices such as network
> >> > interfaces, InfiniBand HCAs or GPUs."
> >> >
> >> > This recipe adds libhwloc and lstopo utility as two packages.
> >> >
> >> > Testing: Built in OE/Krogoth. Installed to x86_64 machine.
> Successfully
> >> > ran `lstopo` and verified it printed correct CPU and cache info.
> >> >
> >> > Signed-off-by: Haris Okanovic <haris.okanovic@ni.com>
> >> > ---
> >> >  meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb | 25
> >> > +++++++++++++++++++++++++
> >> >  1 file changed, 25 insertions(+)
> >> >  create mode 100644 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> >> >
> >> > diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> >> > b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> >> > new file mode 100644
> >> > index 0000000..3a88f5b
> >> > --- /dev/null
> >> > +++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> >> > @@ -0,0 +1,25 @@
> >> > +SUMMARY = "The Portable Hardware Locality (hwloc) software package \
> >> > + provides a portable abstraction of the hierarchical topology of
> modern
> >> \
> >> > + architectures."
> >> > +HOMEPAGE = "https://www.open-mpi.org/software/hwloc/"
> >> > +SECTION = "base"
> >> > +LICENSE = "BSD"
> >> > +LIC_FILES_CHKSUM = "file://COPYING;md5=
> 3282e20dc3cec311deda3c6d4b1f99
> >> 0b"
> >> > +
> >> > +SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/
> >> > downloads/${BP}.tar.bz2"
> >> > +SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
> >> > +
> >> > +inherit autotools
> >> > +
> >> > +# Split hwloc library into separate subpackage
> >> > +PACKAGES_prepend = "lib${PN}"
> >> > +FILES_lib${PN} += "${libdir}/lib${PN}.so*"
> >> > +${PN}_RDEPENDS += "lib${PN}"
> >> > +
> >> > +do_install_append() {
> >> > +    # Let ldconfig do it's thing, no need for these symlinks
> >> > +    test -L "${D}${libdir}/lib${PN}.so.5"
> >> > +    test -L "${D}${libdir}/lib${PN}.so"
> >> > +    rm "${D}${libdir}/lib${PN}.so.5"
> >> > +    rm "${D}${libdir}/lib${PN}.so"
> >>
> >
> > This link manipulation is also inconsistent with every other recipe. It’s
> > not appropriate. If your distro wants to rely on ldconfig to handle the
> > links, then your distro could do this for all recipes rather than just
> one.
> > This isn’t how things are done in our existing recipes.
>
> There is global knob USE_LDCONFIG, perhaps thats what could be used
> but generally I agree with Chris here.
>
> > --
> > Christopher Larson
> > clarson at kergoth dot com
> > Founder - BitBake, OpenEmbedded, OpenZaurus
> > Maintainer - Tslib
> > Senior Software Engineer, Mentor Graphics
> > --
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] hwloc: Add
  2016-12-22 21:34 ` Martin Jansa
  2016-12-22 22:35   ` Christopher Larson
@ 2016-12-26 10:35   ` Andre McCurdy
  1 sibling, 0 replies; 12+ messages in thread
From: Andre McCurdy @ 2016-12-26 10:35 UTC (permalink / raw)
  To: openembeded-devel; +Cc: Haris Okanovic

On Thu, Dec 22, 2016 at 1:34 PM, Martin Jansa <martin.jansa@gmail.com> wrote:
> SUMMARY is for short summary, put long one into DESCRIPTION and create
> short one for SUMMARY.
>
> Also missing sha265sum, so I doubt it was tested properly.

A single hash doesn't generate a warning since:

  http://git.openembedded.org/bitbake/commit/?id=43358a9b595b2928458a5f463cf1949394160c3a


^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] hwloc: Add
  2016-12-25 23:55       ` Martin Jansa
@ 2016-12-27 19:00         ` Haris Okanovic
  0 siblings, 0 replies; 12+ messages in thread
From: Haris Okanovic @ 2016-12-27 19:00 UTC (permalink / raw)
  To: openembedded-devel; +Cc: clarson

Thanks for looking over the patch. I'll post a V2 shortly to address 
these issues.

On 12/25/2016 05:55 PM, Martin Jansa wrote:
> Missing dependencies and PACKAGES_prepend breaks hwloc-dbg, because of missing space at the end of the prepended value.
>
>
>
> hwloc-1.11.5: hwloc: Files/directories were installed but not shipped in any package:
>   /usr/src
>   /usr/src/debug
>   /usr/src/debug/hwloc
>   /usr/src/debug/hwloc/1.11.5-r0
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5
>   /usr/src/debug/hwloc/1.11.5-r0/build
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-xml.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-text.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-fig.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-draw.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-color.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-cairo.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/lstopo/lstopo-ascii.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/misc.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-ps.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-patch.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-info.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-dump-hwdata.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-dump-hwdata-knl.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-distrib.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-distances.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-diff.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-calc.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-calc.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-bind.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-assembler.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/utils/hwloc/hwloc-annotate.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/traversal.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-xml.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-xml-nolibxml.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-xml-libxml.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-x86.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-synthetic.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-pci.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-noos.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-linux.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-hardwired.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/topology-custom.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/pci-common.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/misc.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/distances.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/diff.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/components.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/bitmap.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/bind.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/src/base64.c
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/xml.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/private.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/misc.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/debug.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/cpuid-x86.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/private/components.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/plugins.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/linux.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/inlines.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/helper.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/diff.h
>   /usr/src/debug/hwloc/1.11.5-r0/hwloc-1.11.5/include/hwloc/bitmap.h
>   /usr/src/debug/hwloc/1.11.5-r0/build/src
>   /usr/src/debug/hwloc/1.11.5-r0/build/include
>   /usr/src/debug/hwloc/1.11.5-r0/build/src/static-components.h
>   /usr/src/debug/hwloc/1.11.5-r0/build/include/hwloc
>   /usr/src/debug/hwloc/1.11.5-r0/build/include/hwloc/autogen
>   /usr/src/debug/hwloc/1.11.5-r0/build/include/hwloc/autogen/config.h
> Please set FILES such that these items are packaged. Alternatively if they are unneeded, avoid installing them or delete them within do_install.
> hwloc: 75 installed and not shipped files. [installed-vs-shipped]
> hwloc-1.11.5: hwloc rdepends on numactl, but it isn't a build dependency, missing numactl in DEPENDS or PACKAGECONFIG? [build-deps]
> hwloc-1.11.5: hwloc rdepends on cairo, but it isn't a build dependency, missing cairo in DEPENDS or PACKAGECONFIG? [build-deps]
> hwloc-1.11.5: hwloc rdepends on libpciaccess, but it isn't a build dependency, missing libpciaccess in DEPENDS or PACKAGECONFIG? [build-deps]
> hwloc-1.11.5: hwloc rdepends on libx11, but it isn't a build dependency, missing libx11 in DEPENDS or PACKAGECONFIG? [build-deps]
> hwloc-1.11.5: hwloc rdepends on libcap, but it isn't a build dependency, missing libcap in DEPENDS or PACKAGECONFIG? [build-deps]
> hwloc-1.11.5: hwloc rdepends on zlib, but it isn't a build dependency, missing zlib in DEPENDS or PACKAGECONFIG? [build-deps]
> hwloc-1.11.5: hwloc rdepends on ncurses-libtinfo, but it isn't a build dependency, missing ncurses in DEPENDS or PACKAGECONFIG? [build-deps]
> hwloc-1.11.5: hwloc rdepends on libxml2, but it isn't a build dependency, missing libxml2 in DEPENDS or PACKAGECONFIG? [build-deps]
> hwloc-1.11.5: hwloc rdepends on libudev, but it isn't a build dependency, missing systemd in DEPENDS or PACKAGECONFIG? [build-deps]
>

I'm not really sure what happened here. `bitbake -e` shows a properly 
delimited PACKAGES when I build:

PACKAGES="libhwloc hwloc-dbg hwloc-staticdev [...]"

However, my distro has some core patches which might affect this 
particular variable. I'll pad with spaces as you suggested.

>
> On Thu, Dec 22, 2016 at 11:56 PM, Khem Raj <raj.khem@gmail.com
> <mailto:raj.khem@gmail.com>> wrote:
>
>     On Thu, Dec 22, 2016 at 2:35 PM, Christopher Larson
>     <clarson@kergoth.com <mailto:clarson@kergoth.com>> wrote:
>     > On Thu, Dec 22, 2016 at 2:34 PM, Martin Jansa
>     <martin.jansa@gmail.com <mailto:martin.jansa@gmail.com>>
>     > wrote:
>     >
>     >> SUMMARY is for short summary, put long one into DESCRIPTION and
>     create
>     >> short one for SUMMARY.
>     >>
>     >> Also missing sha265sum, so I doubt it was tested properly.

Andre pointed out that "a single hash doesn't generate a warning" in a 
separate thread. However, I added a sha256sum anyway since it seems 
pretty common.

>     >>
>     >> On Thu, Dec 22, 2016 at 9:02 PM, Haris Okanovic
>     <haris.okanovic@ni.com <mailto:haris.okanovic@ni.com>>
>     >> wrote:
>     >>
>     >> > https://www.open-mpi.org/projects/hwloc/
>     <https://www.open-mpi.org/projects/hwloc/>
>     >> >
>     >> > "The Portable Hardware Locality (hwloc) software package provides a
>     >> > portable abstraction (across OS, versions, architectures, ...)
>     of the
>     >> > hierarchical topology of modern architectures, including NUMA
>     memory
>     >> > nodes, sockets, shared caches, cores and simultaneous
>     multithreading.
>     >> > It also gathers various system attributes such as cache and memory
>     >> > information as well as the locality of I/O devices such as network
>     >> > interfaces, InfiniBand HCAs or GPUs."
>     >> >
>     >> > This recipe adds libhwloc and lstopo utility as two packages.
>     >> >
>     >> > Testing: Built in OE/Krogoth. Installed to x86_64 machine.
>     Successfully
>     >> > ran `lstopo` and verified it printed correct CPU and cache info.
>     >> >
>     >> > Signed-off-by: Haris Okanovic <haris.okanovic@ni.com
>     <mailto:haris.okanovic@ni.com>>
>     >> > ---
>     >> >  meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
>     <http://hwloc_1.11.5.bb> | 25
>     >> > +++++++++++++++++++++++++
>     >> >  1 file changed, 25 insertions(+)
>     >> >  create mode 100644
>     meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb <http://hwloc_1.11.5.bb>
>     >> >
>     >> > diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
>     <http://hwloc_1.11.5.bb>
>     >> > b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
>     <http://hwloc_1.11.5.bb>
>     >> > new file mode 100644
>     >> > index 0000000..3a88f5b
>     >> > --- /dev/null
>     >> > +++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
>     <http://hwloc_1.11.5.bb>
>     >> > @@ -0,0 +1,25 @@
>     >> > +SUMMARY = "The Portable Hardware Locality (hwloc) software
>     package \
>     >> > + provides a portable abstraction of the hierarchical topology
>     of modern
>     >> \
>     >> > + architectures."
>     >> > +HOMEPAGE = "https://www.open-mpi.org/software/hwloc/
>     <https://www.open-mpi.org/software/hwloc/>"
>     >> > +SECTION = "base"
>     >> > +LICENSE = "BSD"
>     >> > +LIC_FILES_CHKSUM =
>     "file://COPYING;md5=3282e20dc3cec311deda3c6d4b1f99
>     >> 0b"
>     >> > +
>     >> > +SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/
>     <https://www.open-mpi.org/software/${PN}/v1.11/>
>     >> > downloads/${BP}.tar.bz2"
>     >> > +SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
>     >> > +
>     >> > +inherit autotools
>     >> > +
>     >> > +# Split hwloc library into separate subpackage
>     >> > +PACKAGES_prepend = "lib${PN}"
>     >> > +FILES_lib${PN} += "${libdir}/lib${PN}.so*"
>     >> > +${PN}_RDEPENDS += "lib${PN}"
>     >> > +
>     >> > +do_install_append() {
>     >> > +    # Let ldconfig do it's thing, no need for these symlinks
>     >> > +    test -L "${D}${libdir}/lib${PN}.so.5"
>     >> > +    test -L "${D}${libdir}/lib${PN}.so"
>     >> > +    rm "${D}${libdir}/lib${PN}.so.5"
>     >> > +    rm "${D}${libdir}/lib${PN}.so"
>     >>
>     >
>     > This link manipulation is also inconsistent with every other
>     recipe. It’s
>     > not appropriate. If your distro wants to rely on ldconfig to
>     handle the
>     > links, then your distro could do this for all recipes rather than
>     just one.
>     > This isn’t how things are done in our existing recipes.
>
>     There is global knob USE_LDCONFIG, perhaps thats what could be used
>     but generally I agree with Chris here.
>

That's fine, I can remove this. I really did it to avoid a dev-so QA 
warning, which isn't very amenable to lib* package names. It trips on 
symlink in any non *-dev or *-dbg package name. Added INSANE_SKIP instead.

Perhaps we should loosen dev-so to permit symlinks in lib*'s as well, if 
it's common practice to package soname symlinks in OE. Thoughts?

>     > --
>     > Christopher Larson
>     > clarson at kergoth dot com
>     > Founder - BitBake, OpenEmbedded, OpenZaurus
>     > Maintainer - Tslib
>     > Senior Software Engineer, Mentor Graphics
>     > --
>     > _______________________________________________
>     > Openembedded-devel mailing list
>     > Openembedded-devel@lists.openembedded.org
>     <mailto:Openembedded-devel@lists.openembedded.org>
>     > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>     <http://lists.openembedded.org/mailman/listinfo/openembedded-devel>
>     --
>     _______________________________________________
>     Openembedded-devel mailing list
>     Openembedded-devel@lists.openembedded.org
>     <mailto:Openembedded-devel@lists.openembedded.org>
>     http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>     <http://lists.openembedded.org/mailman/listinfo/openembedded-devel>
>
>

Thanks,
Haris


^ permalink raw reply	[flat|nested] 12+ messages in thread

* [meta-oe][PATCH v2] hwloc: Add
  2016-12-22 20:02 [PATCH] hwloc: Add Haris Okanovic
  2016-12-22 21:34 ` Martin Jansa
@ 2016-12-27 19:06 ` Haris Okanovic
  2017-01-02 22:33   ` Martin Jansa
  2017-05-19 16:17 ` [PATCH v4] " Haris Okanovic
  2 siblings, 1 reply; 12+ messages in thread
From: Haris Okanovic @ 2016-12-27 19:06 UTC (permalink / raw)
  To: openembedded-devel; +Cc: haris.okanovic

https://www.open-mpi.org/projects/hwloc/

"The Portable Hardware Locality (hwloc) software package provides a
portable abstraction (across OS, versions, architectures, ...) of the
hierarchical topology of modern architectures, including NUMA memory
nodes, sockets, shared caches, cores and simultaneous multithreading.
It also gathers various system attributes such as cache and memory
information as well as the locality of I/O devices such as network
interfaces, InfiniBand HCAs or GPUs."

This recipe adds libhwloc and lstopo utility as two packages.

Testing: Built in OE/Krogoth. Installed to x86_64 machine. Successfully
ran `lstopo` and verified it printed correct CPU and cache info.

Signed-off-by: Haris Okanovic <haris.okanovic@ni.com>
---
 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)
 create mode 100644 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb

diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
new file mode 100644
index 0000000..812f70d
--- /dev/null
+++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
@@ -0,0 +1,22 @@
+SUMMARY = "Portable Hardware Locality (hwloc) software package"
+DESCRIPTION = "The Portable Hardware Locality (hwloc) software package \
+ provides a portable abstraction of the hierarchical topology of modern \
+ architectures."
+HOMEPAGE = "https://www.open-mpi.org/software/hwloc/"
+SECTION = "base"
+LICENSE = "BSD"
+LIC_FILES_CHKSUM = "file://COPYING;md5=3282e20dc3cec311deda3c6d4b1f990b"
+
+SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/downloads/${BP}.tar.bz2"
+SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
+SRC_URI[sha256sum] = "95d80286dfe658a3f79e2ac90698782bb36e5504f4bac1bba2394ba14dbbad24"
+
+inherit autotools
+
+# Split hwloc library into separate subpackage
+PACKAGES_prepend = " lib${PN} "
+FILES_lib${PN} += "${libdir}/lib${PN}.so*"
+${PN}_RDEPENDS += "lib${PN}"
+
+# XXX dev-so QA check doesn't like soname symlinks in non-dev packages
+INSANE_SKIP_lib${PN} += "dev-so"
-- 
2.10.1



^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [meta-oe][PATCH v2] hwloc: Add
  2016-12-27 19:06 ` [meta-oe][PATCH v2] " Haris Okanovic
@ 2017-01-02 22:33   ` Martin Jansa
  2017-01-04 16:20     ` [meta-oe][PATCH v3] " Haris Okanovic
  0 siblings, 1 reply; 12+ messages in thread
From: Martin Jansa @ 2017-01-02 22:33 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Haris Okanovic

By ${PN}_RDEPENDS you probably meant RDEPENDS_${PN}.

On Tue, Dec 27, 2016 at 8:06 PM, Haris Okanovic <haris.okanovic@ni.com>
wrote:

> https://www.open-mpi.org/projects/hwloc/
>
> "The Portable Hardware Locality (hwloc) software package provides a
> portable abstraction (across OS, versions, architectures, ...) of the
> hierarchical topology of modern architectures, including NUMA memory
> nodes, sockets, shared caches, cores and simultaneous multithreading.
> It also gathers various system attributes such as cache and memory
> information as well as the locality of I/O devices such as network
> interfaces, InfiniBand HCAs or GPUs."
>
> This recipe adds libhwloc and lstopo utility as two packages.
>
> Testing: Built in OE/Krogoth. Installed to x86_64 machine. Successfully
> ran `lstopo` and verified it printed correct CPU and cache info.
>
> Signed-off-by: Haris Okanovic <haris.okanovic@ni.com>
> ---
>  meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb | 22
> ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>  create mode 100644 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
>
> diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> new file mode 100644
> index 0000000..812f70d
> --- /dev/null
> +++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> @@ -0,0 +1,22 @@
> +SUMMARY = "Portable Hardware Locality (hwloc) software package"
> +DESCRIPTION = "The Portable Hardware Locality (hwloc) software package \
> + provides a portable abstraction of the hierarchical topology of modern \
> + architectures."
> +HOMEPAGE = "https://www.open-mpi.org/software/hwloc/"
> +SECTION = "base"
> +LICENSE = "BSD"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=3282e20dc3cec311deda3c6d4b1f990b"
> +
> +SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/
> downloads/${BP}.tar.bz2"
> +SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
> +SRC_URI[sha256sum] = "95d80286dfe658a3f79e2ac9069878
> 2bb36e5504f4bac1bba2394ba14dbbad24"
> +
> +inherit autotools
> +
> +# Split hwloc library into separate subpackage
> +PACKAGES_prepend = " lib${PN} "
> +FILES_lib${PN} += "${libdir}/lib${PN}.so*"
> +${PN}_RDEPENDS += "lib${PN}"
> +
> +# XXX dev-so QA check doesn't like soname symlinks in non-dev packages
> +INSANE_SKIP_lib${PN} += "dev-so"
> --
> 2.10.1
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
>


^ permalink raw reply	[flat|nested] 12+ messages in thread

* [meta-oe][PATCH v3] hwloc: Add
  2017-01-02 22:33   ` Martin Jansa
@ 2017-01-04 16:20     ` Haris Okanovic
  2017-01-05 18:04       ` Martin Jansa
  0 siblings, 1 reply; 12+ messages in thread
From: Haris Okanovic @ 2017-01-04 16:20 UTC (permalink / raw)
  To: openembedded-devel; +Cc: haris.okanovic

https://www.open-mpi.org/projects/hwloc/

"The Portable Hardware Locality (hwloc) software package provides a
portable abstraction (across OS, versions, architectures, ...) of the
hierarchical topology of modern architectures, including NUMA memory
nodes, sockets, shared caches, cores and simultaneous multithreading.
It also gathers various system attributes such as cache and memory
information as well as the locality of I/O devices such as network
interfaces, InfiniBand HCAs or GPUs."

This recipe adds libhwloc and lstopo utility as two packages.

Testing: Built in OE/Krogoth. Installed to x86_64 machine. Successfully
ran `lstopo` and verified it printed correct CPU and cache info.

Signed-off-by: Haris Okanovic <haris.okanovic@ni.com>
---
 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)
 create mode 100644 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb

diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
new file mode 100644
index 0000000..1bf7a55
--- /dev/null
+++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
@@ -0,0 +1,22 @@
+SUMMARY = "Portable Hardware Locality (hwloc) software package"
+DESCRIPTION = "The Portable Hardware Locality (hwloc) software package \
+ provides a portable abstraction of the hierarchical topology of modern \
+ architectures."
+HOMEPAGE = "https://www.open-mpi.org/software/hwloc/"
+SECTION = "base"
+LICENSE = "BSD"
+LIC_FILES_CHKSUM = "file://COPYING;md5=3282e20dc3cec311deda3c6d4b1f990b"
+
+SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/downloads/${BP}.tar.bz2"
+SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
+SRC_URI[sha256sum] = "95d80286dfe658a3f79e2ac90698782bb36e5504f4bac1bba2394ba14dbbad24"
+
+inherit autotools
+
+# Split hwloc library into separate subpackage
+PACKAGES_prepend = " lib${PN} "
+FILES_lib${PN} += "${libdir}/lib${PN}.so*"
+RDEPENDS_${PN} += "lib${PN} (= ${EXTENDPKGV})"
+
+# XXX dev-so QA check doesn't like soname symlinks in non-dev packages
+INSANE_SKIP_lib${PN} += "dev-so"
-- 
2.10.1



^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [meta-oe][PATCH v3] hwloc: Add
  2017-01-04 16:20     ` [meta-oe][PATCH v3] " Haris Okanovic
@ 2017-01-05 18:04       ` Martin Jansa
  0 siblings, 0 replies; 12+ messages in thread
From: Martin Jansa @ 2017-01-05 18:04 UTC (permalink / raw)
  To: openembedded-devel; +Cc: haris.okanovic

[-- Attachment #1: Type: text/plain, Size: 3219 bytes --]

On Wed, Jan 04, 2017 at 10:20:43AM -0600, Haris Okanovic wrote:
> https://www.open-mpi.org/projects/hwloc/
> 
> "The Portable Hardware Locality (hwloc) software package provides a
> portable abstraction (across OS, versions, architectures, ...) of the
> hierarchical topology of modern architectures, including NUMA memory
> nodes, sockets, shared caches, cores and simultaneous multithreading.
> It also gathers various system attributes such as cache and memory
> information as well as the locality of I/O devices such as network
> interfaces, InfiniBand HCAs or GPUs."
> 
> This recipe adds libhwloc and lstopo utility as two packages.
> 
> Testing: Built in OE/Krogoth. Installed to x86_64 machine. Successfully
> ran `lstopo` and verified it printed correct CPU and cache info.
> 
> Signed-off-by: Haris Okanovic <haris.okanovic@ni.com>
> ---
>  meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>  create mode 100644 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> 
> diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> new file mode 100644
> index 0000000..1bf7a55
> --- /dev/null
> +++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
> @@ -0,0 +1,22 @@
> +SUMMARY = "Portable Hardware Locality (hwloc) software package"
> +DESCRIPTION = "The Portable Hardware Locality (hwloc) software package \
> + provides a portable abstraction of the hierarchical topology of modern \
> + architectures."
> +HOMEPAGE = "https://www.open-mpi.org/software/hwloc/"
> +SECTION = "base"
> +LICENSE = "BSD"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=3282e20dc3cec311deda3c6d4b1f990b"
> +
> +SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/downloads/${BP}.tar.bz2"
> +SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
> +SRC_URI[sha256sum] = "95d80286dfe658a3f79e2ac90698782bb36e5504f4bac1bba2394ba14dbbad24"
> +
> +inherit autotools
> +
> +# Split hwloc library into separate subpackage
> +PACKAGES_prepend = " lib${PN} "
> +FILES_lib${PN} += "${libdir}/lib${PN}.so*"
> +RDEPENDS_${PN} += "lib${PN} (= ${EXTENDPKGV})"
> +
> +# XXX dev-so QA check doesn't like soname symlinks in non-dev packages
> +INSANE_SKIP_lib${PN} += "dev-so"

Still a lot of missing/autodetected dependencies I've reported in v1:

WARN: hwloc: hwloc rdepends on cairo, but it isn't a build dependency?
WARN: hwloc: hwloc rdepends on ncurses-libtinfo, but it isn't a build
dependency?
WARN: hwloc: libhwloc rdepends on libpciaccess, but it isn't a build
dependency?
WARN: hwloc: libhwloc rdepends on libudev, but it isn't a build
dependency?
WARN: hwloc: libhwloc rdepends on libxml2, but it isn't a build
dependency?
WARN: hwloc: libhwloc rdepends on numactl, but it isn't a build
dependency?
WARN: hwloc: libhwloc rdepends on zlib, but it isn't a build dependency?
> -- 
> 2.10.1
> 
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 201 bytes --]

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH v4] hwloc: Add
  2016-12-22 20:02 [PATCH] hwloc: Add Haris Okanovic
  2016-12-22 21:34 ` Martin Jansa
  2016-12-27 19:06 ` [meta-oe][PATCH v2] " Haris Okanovic
@ 2017-05-19 16:17 ` Haris Okanovic
  2 siblings, 0 replies; 12+ messages in thread
From: Haris Okanovic @ 2017-05-19 16:17 UTC (permalink / raw)
  To: openembedded-devel; +Cc: haris.okanovic, clarson

https://www.open-mpi.org/projects/hwloc/

"The Portable Hardware Locality (hwloc) software package provides a
portable abstraction (across OS, versions, architectures, ...) of the
hierarchical topology of modern architectures, including NUMA memory
nodes, sockets, shared caches, cores and simultaneous multithreading.
It also gathers various system attributes such as cache and memory
information as well as the locality of I/O devices such as network
interfaces, InfiniBand HCAs or GPUs."

This recipe adds libhwloc and lstopo utility as two packages.

Testing: Built in OE/Krogoth. Installed to x86_64 machine. Successfully
ran `lstopo` and verified it printed correct CPU and cache info.

Signed-off-by: Haris Okanovic <haris.okanovic@ni.com>
---
 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)
 create mode 100644 meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb

diff --git a/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
new file mode 100644
index 000000000..e067f2799
--- /dev/null
+++ b/meta-oe/recipes-extended/hwloc/hwloc_1.11.5.bb
@@ -0,0 +1,26 @@
+SUMMARY = "Portable Hardware Locality (hwloc) software package"
+DESCRIPTION = "The Portable Hardware Locality (hwloc) software package \
+ provides a portable abstraction of the hierarchical topology of modern \
+ architectures."
+HOMEPAGE = "https://www.open-mpi.org/software/hwloc/"
+SECTION = "base"
+LICENSE = "BSD"
+LIC_FILES_CHKSUM = "file://COPYING;md5=3282e20dc3cec311deda3c6d4b1f990b"
+
+SRC_URI = "https://www.open-mpi.org/software/${PN}/v1.11/downloads/${BP}.tar.bz2"
+SRC_URI[md5sum] = "96c34136ff416d2b13a7821c27477bed"
+SRC_URI[sha256sum] = "95d80286dfe658a3f79e2ac90698782bb36e5504f4bac1bba2394ba14dbbad24"
+
+inherit autotools
+
+DEPENDS += "cairo ncurses udev libxml2 zlib libpciaccess"
+DEPENDS += "${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'virtual/libx11', '', d)}"
+DEPENDS += "${@bb.utils.contains('DISTRO_FEATURES', 'selinux', 'libselinux', '', d)}"
+
+# Split hwloc library into separate subpackage
+PACKAGES_prepend = " lib${PN} "
+FILES_lib${PN} += "${libdir}/lib${PN}.so*"
+RDEPENDS_${PN} += "lib${PN} (= ${EXTENDPKGV})"
+
+# XXX dev-so QA check doesn't like soname symlinks in non-dev packages
+INSANE_SKIP_lib${PN} += "dev-so"
-- 
2.12.1



^ permalink raw reply related	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2017-05-19 17:01 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-22 20:02 [PATCH] hwloc: Add Haris Okanovic
2016-12-22 21:34 ` Martin Jansa
2016-12-22 22:35   ` Christopher Larson
2016-12-22 22:56     ` Khem Raj
2016-12-25 23:55       ` Martin Jansa
2016-12-27 19:00         ` Haris Okanovic
2016-12-26 10:35   ` Andre McCurdy
2016-12-27 19:06 ` [meta-oe][PATCH v2] " Haris Okanovic
2017-01-02 22:33   ` Martin Jansa
2017-01-04 16:20     ` [meta-oe][PATCH v3] " Haris Okanovic
2017-01-05 18:04       ` Martin Jansa
2017-05-19 16:17 ` [PATCH v4] " Haris Okanovic

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.