* [Buildroot] [PATCH 0/2] add more dependencies on atomic operations @ 2014-09-08 6:32 Alexey Brodkin 2014-09-08 6:32 ` [Buildroot] [PATCH 1/2] libtirpc: add dependency " Alexey Brodkin 2014-09-08 6:32 ` [Buildroot] [PATCH 2/2] zeromq: " Alexey Brodkin 0 siblings, 2 replies; 12+ messages in thread From: Alexey Brodkin @ 2014-09-08 6:32 UTC (permalink / raw) To: buildroot More packages were discovered to use atomic operatoins. With this series dependencies are applied to libraries that use atomic operations and packages that explicitly select mentioned libraries. Alexey Brodkin (2): libtirpc: add dependency on atomic operations zeromq: add dependency on atomic operations package/argus/Config.in | 1 + package/conntrack-tools/Config.in | 1 + package/cppzmq/Config.in | 1 + package/czmq/Config.in | 1 + package/filemq/Config.in | 1 + package/libtirpc/Config.in | 1 + package/lmbench/Config.in | 1 + package/mongrel2/Config.in | 1 + package/nfs-utils/Config.in | 1 + package/php-zmq/Config.in | 1 + package/python-pyzmq/Config.in | 1 + package/quota/Config.in | 1 + package/rpcbind/Config.in | 1 + package/zeromq/Config.in | 1 + package/zmqpp/Config.in | 1 + package/zyre/Config.in | 1 + 16 files changed, 16 insertions(+) -- 1.9.3 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations 2014-09-08 6:32 [Buildroot] [PATCH 0/2] add more dependencies on atomic operations Alexey Brodkin @ 2014-09-08 6:32 ` Alexey Brodkin 2014-09-08 7:23 ` Thomas Petazzoni 2014-09-08 6:32 ` [Buildroot] [PATCH 2/2] zeromq: " Alexey Brodkin 1 sibling, 1 reply; 12+ messages in thread From: Alexey Brodkin @ 2014-09-08 6:32 UTC (permalink / raw) To: buildroot Fixes: http://autobuild.buildroot.net/results/6bb90d3f2b840a1faacf152a1d93a909a2dc98d3 http://autobuild.buildroot.net/results/994a84d192848b90a70e28e5ebaa769701bcaec9 Following packages select "libtirpc", so adding dependency as well: * argus * conntrack-tools * lmbench * nfs-utils * quota * rpcbind Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> Cc: Anton Kolesov <akolesov@synopsys.com> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar> Cc: Peter Korsgaard <peter@korsgaard.com> CC: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> --- package/argus/Config.in | 1 + package/conntrack-tools/Config.in | 1 + package/libtirpc/Config.in | 1 + package/lmbench/Config.in | 1 + package/nfs-utils/Config.in | 1 + package/quota/Config.in | 1 + package/rpcbind/Config.in | 1 + 7 files changed, 7 insertions(+) diff --git a/package/argus/Config.in b/package/argus/Config.in index 6d47734..2fab07a 100644 --- a/package/argus/Config.in +++ b/package/argus/Config.in @@ -1,6 +1,7 @@ config BR2_PACKAGE_ARGUS bool "argus" depends on BR2_INET_IPV6 + depends on BR2_ARCH_HAS_ATOMICS # libtirpc depends on BR2_TOOLCHAIN_HAS_THREADS depends on BR2_USE_MMU # fork() select BR2_PACKAGE_LIBPCAP diff --git a/package/conntrack-tools/Config.in b/package/conntrack-tools/Config.in index ec19d8e..36b12b7 100644 --- a/package/conntrack-tools/Config.in +++ b/package/conntrack-tools/Config.in @@ -3,6 +3,7 @@ config BR2_PACKAGE_CONNTRACK_TOOLS depends on BR2_INET_IPV6 depends on BR2_LARGEFILE depends on BR2_USE_MMU # fork() + depends on BR2_ARCH_HAS_ATOMICS # libtirpc depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC || BR2_TOOLCHAIN_HAS_THREADS # libtirpc select BR2_PACKAGE_LIBNETFILTER_CONNTRACK select BR2_PACKAGE_LIBNETFILTER_CTHELPER diff --git a/package/libtirpc/Config.in b/package/libtirpc/Config.in index a8e6dd3..a5fe175 100644 --- a/package/libtirpc/Config.in +++ b/package/libtirpc/Config.in @@ -7,6 +7,7 @@ config BR2_PACKAGE_LIBTIRPC # used instead of libtirpc. depends on !BR2_TOOLCHAIN_EXTERNAL_BLACKFIN_UCLINUX_2012R2 depends on BR2_TOOLCHAIN_HAS_THREADS + depends on BR2_ARCH_HAS_ATOMICS help Libtirpc is a port of Sun's Transport-Independent RPC library to Linux. diff --git a/package/lmbench/Config.in b/package/lmbench/Config.in index c4ea522..5b88d62 100644 --- a/package/lmbench/Config.in +++ b/package/lmbench/Config.in @@ -1,5 +1,6 @@ config BR2_PACKAGE_LMBENCH bool "lmbench" + depends on BR2_ARCH_HAS_ATOMICS # libtirpc depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC || BR2_TOOLCHAIN_HAS_THREADS # libtirpc select BR2_PACKAGE_LIBTIRPC if !BR2_TOOLCHAIN_HAS_NATIVE_RPC depends on BR2_USE_MMU # fork() diff --git a/package/nfs-utils/Config.in b/package/nfs-utils/Config.in index dff5150..3168333 100644 --- a/package/nfs-utils/Config.in +++ b/package/nfs-utils/Config.in @@ -5,6 +5,7 @@ comment "nfs-utils needs a toolchain w/ largefile, threads" config BR2_PACKAGE_NFS_UTILS bool "nfs-utils" + depends on BR2_ARCH_HAS_ATOMICS # libtirpc depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC || BR2_TOOLCHAIN_HAS_THREADS # libtirpc select BR2_PACKAGE_LIBTIRPC if !BR2_TOOLCHAIN_HAS_NATIVE_RPC depends on BR2_LARGEFILE diff --git a/package/quota/Config.in b/package/quota/Config.in index 69f03af..cceb2ab 100644 --- a/package/quota/Config.in +++ b/package/quota/Config.in @@ -4,6 +4,7 @@ config BR2_PACKAGE_QUOTA depends on BR2_USE_WCHAR depends on BR2_USE_MMU # fork() depends on !(BR2_microblazeel || BR2_microblazebe) # util-linux + depends on BR2_ARCH_HAS_ATOMICS # libtirpc depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC || BR2_TOOLCHAIN_HAS_THREADS # libtirpc select BR2_PACKAGE_UTIL_LINUX select BR2_PACKAGE_UTIL_LINUX_BINARIES diff --git a/package/rpcbind/Config.in b/package/rpcbind/Config.in index 2a79bdc..bc658d6 100644 --- a/package/rpcbind/Config.in +++ b/package/rpcbind/Config.in @@ -2,6 +2,7 @@ config BR2_PACKAGE_RPCBIND bool "rpcbind" # We really need libtirpc and can't work with the native RPC # implementation of toolchains. + depends on BR2_ARCH_HAS_ATOMICS # libtirpc depends on BR2_TOOLCHAIN_HAS_THREADS # libtirpc select BR2_PACKAGE_LIBTIRPC # uClibc toolchains provided by ADI don't have AI_ADDRCONFIG -- 1.9.3 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations 2014-09-08 6:32 ` [Buildroot] [PATCH 1/2] libtirpc: add dependency " Alexey Brodkin @ 2014-09-08 7:23 ` Thomas Petazzoni 2014-09-08 9:23 ` Alexey Brodkin 0 siblings, 1 reply; 12+ messages in thread From: Thomas Petazzoni @ 2014-09-08 7:23 UTC (permalink / raw) To: buildroot Dear Alexey Brodkin, On Mon, 8 Sep 2014 10:32:43 +0400, Alexey Brodkin wrote: > Fixes: > http://autobuild.buildroot.net/results/6bb90d3f2b840a1faacf152a1d93a909a2dc98d3 > http://autobuild.buildroot.net/results/994a84d192848b90a70e28e5ebaa769701bcaec9 > > Following packages select "libtirpc", so adding dependency as well: > * argus > * conntrack-tools > * lmbench > * nfs-utils > * quota > * rpcbind > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> That's not that simple because you forget to update the comments: they should not be displayed if the architecture doesn't have the atomic intrinsics. And this is where things start to be really, really weird. > diff --git a/package/conntrack-tools/Config.in b/package/conntrack-tools/Config.in > index ec19d8e..36b12b7 100644 > --- a/package/conntrack-tools/Config.in > +++ b/package/conntrack-tools/Config.in > @@ -3,6 +3,7 @@ config BR2_PACKAGE_CONNTRACK_TOOLS > depends on BR2_INET_IPV6 > depends on BR2_LARGEFILE > depends on BR2_USE_MMU # fork() > + depends on BR2_ARCH_HAS_ATOMICS # libtirpc That's not correct: if the toolchain has native RPC support, we do not need libtirpc, so there is no need to depend on BR2_ARCH_HAS_ATOMICS in this case. I already tried to solve this thing, but it's much more complicated than just adding a bunch of "depends on". Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations 2014-09-08 7:23 ` Thomas Petazzoni @ 2014-09-08 9:23 ` Alexey Brodkin 2014-09-08 9:27 ` Thomas Petazzoni 0 siblings, 1 reply; 12+ messages in thread From: Alexey Brodkin @ 2014-09-08 9:23 UTC (permalink / raw) To: buildroot Hi Thomas, On Mon, 2014-09-08 at 09:23 +0200, Thomas Petazzoni wrote: > Dear Alexey Brodkin, > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > > That's not that simple because you forget to update the comments: they > should not be displayed if the architecture doesn't have the atomic > intrinsics. And this is where things start to be really, really weird. > > diff --git a/package/conntrack-tools/Config.in b/package/conntrack-tools/Config.in > > index ec19d8e..36b12b7 100644 > > --- a/package/conntrack-tools/Config.in > > +++ b/package/conntrack-tools/Config.in > > @@ -3,6 +3,7 @@ config BR2_PACKAGE_CONNTRACK_TOOLS > > depends on BR2_INET_IPV6 > > depends on BR2_LARGEFILE > > depends on BR2_USE_MMU # fork() > > + depends on BR2_ARCH_HAS_ATOMICS # libtirpc > > That's not correct: if the toolchain has native RPC support, we do not > need libtirpc, so there is no need to depend on BR2_ARCH_HAS_ATOMICS in > this case. > > I already tried to solve this thing, but it's much more complicated > than just adding a bunch of "depends on". So I assume there's no sense for me to try fixing comments, right? I mean if this won't work either or introduce new problems as you meant. Then what could be our next steps to accommodate possibility of existence/absence of atomic ops? This is especially important for ARC where these ops are optional. The simplest solution could be to enable BR2_ARCH_HAS_ATOMICS for ARC by default, but this will just attempt to escape from real problem. And as I saw from other autobuild results other architectures are affected like Blackfin for example. If this is that complicated we may try to discuss it in person on Buildroot summit in few weeks. -Alexey ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations 2014-09-08 9:23 ` Alexey Brodkin @ 2014-09-08 9:27 ` Thomas Petazzoni 2014-09-12 13:12 ` Alexey Brodkin 0 siblings, 1 reply; 12+ messages in thread From: Thomas Petazzoni @ 2014-09-08 9:27 UTC (permalink / raw) To: buildroot Dear Alexey Brodkin, On Mon, 8 Sep 2014 09:23:02 +0000, Alexey Brodkin wrote: > > I already tried to solve this thing, but it's much more complicated > > than just adding a bunch of "depends on". > > So I assume there's no sense for me to try fixing comments, right? > I mean if this won't work either or introduce new problems as you meant. There is definitely some sense in fixing this stuff. I'm not sure yet on how to do this. I had some discussion with Thomas DS a week ago or so on IRC about this. > This is especially important for ARC where these ops are optional. > The simplest solution could be to enable BR2_ARCH_HAS_ATOMICS for ARC by > default, but this will just attempt to escape from real problem. > > And as I saw from other autobuild results other architectures are > affected like Blackfin for example. Yes, sure, we need to fix the problem. It's just not as simple as adding more "depends on", unfortunately :-( > If this is that complicated we may try to discuss it in person on > Buildroot summit in few weeks. Ah, you're planning on coming to the Buildroot meeting? If so, could you register to http://elinux.org/Buildroot:DeveloperDaysELCE2014, or confirm me that you are coming so that I can add your name to the list of participants? Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations 2014-09-08 9:27 ` Thomas Petazzoni @ 2014-09-12 13:12 ` Alexey Brodkin 2014-09-12 13:25 ` Thomas Petazzoni 0 siblings, 1 reply; 12+ messages in thread From: Alexey Brodkin @ 2014-09-12 13:12 UTC (permalink / raw) To: buildroot Hi Thomas, On Mon, 2014-09-08 at 11:27 +0200, Thomas Petazzoni wrote: > Dear Alexey Brodkin, > > On Mon, 8 Sep 2014 09:23:02 +0000, Alexey Brodkin wrote: > Ah, you're planning on coming to the Buildroot meeting? If so, could > you register to http://elinux.org/Buildroot:DeveloperDaysELCE2014, or > confirm me that you are coming so that I can add your name to the list > of participants? Do you mind to add another atomic ops discussion as another item to "Topics to discuss" during the event? -Alexey ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations 2014-09-12 13:12 ` Alexey Brodkin @ 2014-09-12 13:25 ` Thomas Petazzoni 2014-09-12 13:31 ` Alexey Brodkin 0 siblings, 1 reply; 12+ messages in thread From: Thomas Petazzoni @ 2014-09-12 13:25 UTC (permalink / raw) To: buildroot Dear Alexey Brodkin, On Fri, 12 Sep 2014 13:12:03 +0000, Alexey Brodkin wrote: > > Ah, you're planning on coming to the Buildroot meeting? If so, could > > you register to http://elinux.org/Buildroot:DeveloperDaysELCE2014, or > > confirm me that you are coming so that I can add your name to the list > > of participants? > > Do you mind to add another atomic ops discussion as another item to > "Topics to discuss" during the event? I've added it. But to be honest, I hope to be able to make a proposal to solve the problem before the meeting. Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 1/2] libtirpc: add dependency on atomic operations 2014-09-12 13:25 ` Thomas Petazzoni @ 2014-09-12 13:31 ` Alexey Brodkin 0 siblings, 0 replies; 12+ messages in thread From: Alexey Brodkin @ 2014-09-12 13:31 UTC (permalink / raw) To: buildroot Hi Thomas, On Fri, 2014-09-12 at 15:25 +0200, Thomas Petazzoni wrote: > Dear Alexey Brodkin, > > On Fri, 12 Sep 2014 13:12:03 +0000, Alexey Brodkin wrote: > > > > Ah, you're planning on coming to the Buildroot meeting? If so, could > > > you register to http://elinux.org/Buildroot:DeveloperDaysELCE2014, or > > > confirm me that you are coming so that I can add your name to the list > > > of participants? > > > > Do you mind to add another atomic ops discussion as another item to > > "Topics to discuss" during the event? > > I've added it. But to be honest, I hope to be able to make a proposal > to solve the problem before the meeting. Good to know it! Please let me know if I may somehow help you with it. -Alexey ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 2/2] zeromq: add dependency on atomic operations 2014-09-08 6:32 [Buildroot] [PATCH 0/2] add more dependencies on atomic operations Alexey Brodkin 2014-09-08 6:32 ` [Buildroot] [PATCH 1/2] libtirpc: add dependency " Alexey Brodkin @ 2014-09-08 6:32 ` Alexey Brodkin 2014-09-08 7:23 ` Thomas Petazzoni 1 sibling, 1 reply; 12+ messages in thread From: Alexey Brodkin @ 2014-09-08 6:32 UTC (permalink / raw) To: buildroot As it is seen from http://autobuild.buildroot.org/results/3bd/3bda860a269dd9a4e38533cc9350892c68102618 "zeromq" uses "___sync_add_and_fetch_2" & "___sync_fetch_and_add_2" in "libpgm" so make it dependent on atomic operations. Following packages select "zeromq", so adding dependency as well: * cppzmq * czmq * filemq * mongrel2 * php-zmq * python-pyzmq * zmqpp * zyre Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> Cc: Anton Kolesov <akolesov@synopsys.com> Cc: Gustavo Zacarias <gustavo@zacarias.com.ar> Cc: Peter Korsgaard <peter@korsgaard.com> CC: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> --- package/cppzmq/Config.in | 1 + package/czmq/Config.in | 1 + package/filemq/Config.in | 1 + package/mongrel2/Config.in | 1 + package/php-zmq/Config.in | 1 + package/python-pyzmq/Config.in | 1 + package/zeromq/Config.in | 1 + package/zmqpp/Config.in | 1 + package/zyre/Config.in | 1 + 9 files changed, 9 insertions(+) diff --git a/package/cppzmq/Config.in b/package/cppzmq/Config.in index e17963e..9f32c51 100644 --- a/package/cppzmq/Config.in +++ b/package/cppzmq/Config.in @@ -5,6 +5,7 @@ config BR2_PACKAGE_CPPZMQ depends on BR2_LARGEFILE # util-linux depends on BR2_USE_WCHAR # util-linux depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq + depends on BR2_ARCH_HAS_ATOMICS # zeromq select BR2_PACKAGE_ZEROMQ help C++ binding for zeromq (ZeroMQ, 0MQ, zmq). diff --git a/package/czmq/Config.in b/package/czmq/Config.in index fbdedd1..b005a0e 100644 --- a/package/czmq/Config.in +++ b/package/czmq/Config.in @@ -7,6 +7,7 @@ config BR2_PACKAGE_CZMQ depends on BR2_LARGEFILE # util-linux depends on BR2_USE_WCHAR # util-linux depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq + depends on BR2_ARCH_HAS_ATOMICS # zeromq help High-level C Binding for 0MQ diff --git a/package/filemq/Config.in b/package/filemq/Config.in index 1604846..0fea057 100644 --- a/package/filemq/Config.in +++ b/package/filemq/Config.in @@ -8,6 +8,7 @@ config BR2_PACKAGE_FILEMQ depends on BR2_LARGEFILE # util-linux depends on BR2_USE_WCHAR # util-linux depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq + depends on BR2_ARCH_HAS_ATOMICS # zeromq depends on BR2_USE_MMU # czmq help FileMQ is a publish-subscribe file service based on 0MQ. diff --git a/package/mongrel2/Config.in b/package/mongrel2/Config.in index 680e0ce..21e0706 100644 --- a/package/mongrel2/Config.in +++ b/package/mongrel2/Config.in @@ -14,6 +14,7 @@ config BR2_PACKAGE_MONGREL2 depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq depends on BR2_LARGEFILE # zeromq -> util-linux depends on BR2_USE_WCHAR # zeromq -> util-linux + depends on BR2_ARCH_HAS_ATOMICS # zeromq # {get,make,swap}context functions present in # {e,}glibc and in uClibc's master branch. Source has arm workaround depends on BR2_UCLIBC_VERSION_SNAPSHOT || \ diff --git a/package/php-zmq/Config.in b/package/php-zmq/Config.in index f32b5a3..ff34082 100644 --- a/package/php-zmq/Config.in +++ b/package/php-zmq/Config.in @@ -10,6 +10,7 @@ config BR2_PACKAGE_PHP_ZMQ depends on BR2_LARGEFILE # util-linux depends on BR2_USE_WCHAR # util-linux depends on BR2_TOOLCHAIN_HAS_THREADS + depends on BR2_ARCH_HAS_ATOMICS # zeromq select BR2_PACKAGE_ZEROMQ help ZeroMQ messaging bindings for PHP diff --git a/package/python-pyzmq/Config.in b/package/python-pyzmq/Config.in index 0f868d0..b2a60c2 100644 --- a/package/python-pyzmq/Config.in +++ b/package/python-pyzmq/Config.in @@ -5,6 +5,7 @@ config BR2_PACKAGE_PYTHON_PYZMQ depends on BR2_USE_WCHAR # zeromq depends on BR2_INSTALL_LIBSTDCPP # zeromq depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq + depends on BR2_ARCH_HAS_ATOMICS # zeromq select BR2_PACKAGE_ZEROMQ help This package contains the python language binding for zeromq. diff --git a/package/zeromq/Config.in b/package/zeromq/Config.in index 778a503..4178014 100644 --- a/package/zeromq/Config.in +++ b/package/zeromq/Config.in @@ -9,6 +9,7 @@ config BR2_PACKAGE_ZEROMQ depends on BR2_LARGEFILE # util-linux depends on BR2_USE_WCHAR # util-linux depends on BR2_TOOLCHAIN_HAS_THREADS + depends on BR2_ARCH_HAS_ATOMICS select BR2_PACKAGE_UTIL_LINUX select BR2_PACKAGE_UTIL_LINUX_LIBUUID help diff --git a/package/zmqpp/Config.in b/package/zmqpp/Config.in index 2ccaf7e..7740679 100644 --- a/package/zmqpp/Config.in +++ b/package/zmqpp/Config.in @@ -8,6 +8,7 @@ config BR2_PACKAGE_ZMQPP depends on BR2_LARGEFILE # util-linux depends on BR2_USE_WCHAR # util-linux depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq + depends on BR2_ARCH_HAS_ATOMICS # zeromq select BR2_PACKAGE_ZEROMQ help C++ binding for zeromq (ZeroMQ, 0MQ, zmq). diff --git a/package/zyre/Config.in b/package/zyre/Config.in index 86fff35..2c32068 100644 --- a/package/zyre/Config.in +++ b/package/zyre/Config.in @@ -5,6 +5,7 @@ config BR2_PACKAGE_ZYRE depends on BR2_LARGEFILE # zeromq depends on BR2_USE_WCHAR # zeromq depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq + depends on BR2_ARCH_HAS_ATOMICS # zeromq depends on BR2_USE_MMU # czmq # uClibc toolchains provided by ADI don't have AI_ADDRCONFIG # support, needed by czmq -- 1.9.3 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 2/2] zeromq: add dependency on atomic operations 2014-09-08 6:32 ` [Buildroot] [PATCH 2/2] zeromq: " Alexey Brodkin @ 2014-09-08 7:23 ` Thomas Petazzoni 2014-09-08 9:27 ` Alexey Brodkin 0 siblings, 1 reply; 12+ messages in thread From: Thomas Petazzoni @ 2014-09-08 7:23 UTC (permalink / raw) To: buildroot Dear Alexey Brodkin, On Mon, 8 Sep 2014 10:32:44 +0400, Alexey Brodkin wrote: > diff --git a/package/cppzmq/Config.in b/package/cppzmq/Config.in > index e17963e..9f32c51 100644 > --- a/package/cppzmq/Config.in > +++ b/package/cppzmq/Config.in > @@ -5,6 +5,7 @@ config BR2_PACKAGE_CPPZMQ > depends on BR2_LARGEFILE # util-linux > depends on BR2_USE_WCHAR # util-linux > depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq > + depends on BR2_ARCH_HAS_ATOMICS # zeromq > select BR2_PACKAGE_ZEROMQ > help > C++ binding for zeromq (ZeroMQ, 0MQ, zmq). Same thing here: you should change the comment to *not* display it if the architecture doesn't have atomic intrinsics. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 2/2] zeromq: add dependency on atomic operations 2014-09-08 7:23 ` Thomas Petazzoni @ 2014-09-08 9:27 ` Alexey Brodkin 2014-09-08 9:33 ` Thomas Petazzoni 0 siblings, 1 reply; 12+ messages in thread From: Alexey Brodkin @ 2014-09-08 9:27 UTC (permalink / raw) To: buildroot Hi Thomas, On Mon, 2014-09-08 at 09:23 +0200, Thomas Petazzoni wrote: > Dear Alexey Brodkin, > > On Mon, 8 Sep 2014 10:32:44 +0400, Alexey Brodkin wrote: > > > diff --git a/package/cppzmq/Config.in b/package/cppzmq/Config.in > > index e17963e..9f32c51 100644 > > --- a/package/cppzmq/Config.in > > +++ b/package/cppzmq/Config.in > > @@ -5,6 +5,7 @@ config BR2_PACKAGE_CPPZMQ > > depends on BR2_LARGEFILE # util-linux > > depends on BR2_USE_WCHAR # util-linux > > depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq > > + depends on BR2_ARCH_HAS_ATOMICS # zeromq > > select BR2_PACKAGE_ZEROMQ > > help > > C++ binding for zeromq (ZeroMQ, 0MQ, zmq). > > Same thing here: you should change the comment to *not* display it if > the architecture doesn't have atomic intrinsics. I assume comments from the previous patch apply here as well, right? Especially since atomic ops are only used in "libpgm", i.e. if only BR2_PACKAGE_ZEROMQ_PGM is enabled. -Alexey ^ permalink raw reply [flat|nested] 12+ messages in thread
* [Buildroot] [PATCH 2/2] zeromq: add dependency on atomic operations 2014-09-08 9:27 ` Alexey Brodkin @ 2014-09-08 9:33 ` Thomas Petazzoni 0 siblings, 0 replies; 12+ messages in thread From: Thomas Petazzoni @ 2014-09-08 9:33 UTC (permalink / raw) To: buildroot Dear Alexey Brodkin, On Mon, 8 Sep 2014 09:27:21 +0000, Alexey Brodkin wrote: > Hi Thomas, > > On Mon, 2014-09-08 at 09:23 +0200, Thomas Petazzoni wrote: > > Dear Alexey Brodkin, > > > > On Mon, 8 Sep 2014 10:32:44 +0400, Alexey Brodkin wrote: > > > > > diff --git a/package/cppzmq/Config.in b/package/cppzmq/Config.in > > > index e17963e..9f32c51 100644 > > > --- a/package/cppzmq/Config.in > > > +++ b/package/cppzmq/Config.in > > > @@ -5,6 +5,7 @@ config BR2_PACKAGE_CPPZMQ > > > depends on BR2_LARGEFILE # util-linux > > > depends on BR2_USE_WCHAR # util-linux > > > depends on BR2_TOOLCHAIN_HAS_THREADS # zeromq > > > + depends on BR2_ARCH_HAS_ATOMICS # zeromq > > > select BR2_PACKAGE_ZEROMQ > > > help > > > C++ binding for zeromq (ZeroMQ, 0MQ, zmq). > > > > Same thing here: you should change the comment to *not* display it if > > the architecture doesn't have atomic intrinsics. > > I assume comments from the previous patch apply here as well, right? > Especially since atomic ops are only used in "libpgm", i.e. if only > BR2_PACKAGE_ZEROMQ_PGM is enabled. If atomic operations are only used when BR2_PACKAGE_ZEROMQ_PGM is enabled, why do we need to propagate all those dependencies then? If a package selects BR2_PACKAGE_ZEROMQ and not BR2_PACKAGE_ZEROMQ_PGM, there's no reason to propagate the dependency. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2014-09-12 13:31 UTC | newest] Thread overview: 12+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-09-08 6:32 [Buildroot] [PATCH 0/2] add more dependencies on atomic operations Alexey Brodkin 2014-09-08 6:32 ` [Buildroot] [PATCH 1/2] libtirpc: add dependency " Alexey Brodkin 2014-09-08 7:23 ` Thomas Petazzoni 2014-09-08 9:23 ` Alexey Brodkin 2014-09-08 9:27 ` Thomas Petazzoni 2014-09-12 13:12 ` Alexey Brodkin 2014-09-12 13:25 ` Thomas Petazzoni 2014-09-12 13:31 ` Alexey Brodkin 2014-09-08 6:32 ` [Buildroot] [PATCH 2/2] zeromq: " Alexey Brodkin 2014-09-08 7:23 ` Thomas Petazzoni 2014-09-08 9:27 ` Alexey Brodkin 2014-09-08 9:33 ` Thomas Petazzoni
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox