[Buildroot] [autobuild.buildroot.net] Your daily results for 2021-02-04

From
Heiko Thiery <hidden>
Date
2021-02-06 09:37:51
Reply to
Yann E. MORIN, 2021-02-05
Hi Yann and all,

Am Sa., 6. Feb. 2021 um 00:37 Uhr schrieb Yann E. MORIN
[off-list ref]:
Heiko, All,

On 2021-02-05 15:42 +0100, Heiko Thiery spake thusly:
quoted
Am Fr., 5. Feb. 2021 um 09:18 Uhr schrieb Thomas Petazzoni
[off-list ref]:
quoted
   xtensa    |        netopeer2-1.1.53        | http://autobuild.buildroot.net/results/e21834d4d2ee580f00f0fdcbd3728787148c0da9
I checked the reason for the build failure on the netopeer2 package.
It is caused by some files that are created in /dev/shm/sr_* during
the installation process.
Turns out, the 'sr_' prefix can be customised at runtime:

    $ SYSREPO_SHM_PREFIX=GRRR make netopeer2-reinstall
    $ ls /dev/shm
    /dev/shm/GRRR_ext                                     /dev/shm/GRRR_ietf-ssh-server.operational
    /dev/shm/GRRR_ietf-crypto-types.operational           /dev/shm/GRRR_ietf-ssh-server.running
    [--SNIP--]
    /dev/shm/GRRR_ietf-netconf-nmda.running               /dev/shm/GRRR_main
    [--SNIP--]
    /dev/shm/GRRR_ietf-origin.operational                 /dev/shm/GRRR_yang.operational
    /dev/shm/GRRR_ietf-origin.running                     /dev/shm/GRRR_yang.running

Unfortunately, that can not be used be specify a sub-directory, or an
alternate location...

So, by carefully choosing a prefix, we can at least identify what files
we must remove after the fact, and we can ensure that two concurrent
builds will not use the same files.
Great. I was not aware of this possibility. This seems to be the
easiest way to fix the issue from a buildroot point of view.
For example, totally untested:

    diff --git a/package/netopeer2/netopeer2.mk b/package/netopeer2/netopeer2.mk
    index bc02e0dc93..dd10f76cba 100644
    --- a/package/netopeer2/netopeer2.mk
    +++ b/package/netopeer2/netopeer2.mk
    @@ -13,9 +13,20 @@ NETOPEER2_DEPENDENCIES = libnetconf2 libyang sysrepo

     NETOPEER2_CONF_OPTS = -DBUILD_CLI=$(if $(BR2_PACKAGE_NETOPEER2_CLI),ON,OFF)

    +NETOPEER2_SHM_PREFIX = sr_buildroot$(subst /,_,$(CONFIG_DIR))_
    +NETOPEER2_MAKE_ENV = SYSREPO_SHM_PREFIX=$(NETOPEER2_SHM_PREFIX)
    +
     define NETOPEER2_INSTALL_INIT_SYSV
        $(INSTALL) -m 755 -D package/netopeer2/S52netopeer2 \
                $(TARGET_DIR)/etc/init.d/S52netopeer2
     endef

    +# The host sysrepo used to install the netopeer2 modules will leave
    +# its shared memory files lingering about. Clean up in its stead...
    +define NETOPEER2_CLEANUP
    +   rm -f /dev/shm/$(NETOPEER2_SHM_PREFIX)*
    +endef
    +NETOPEER2_PRE_INSTALL_TARGET_HOOKS += NETOPEER2_CLEANUP
    +NETOPEER2_POST_INSTALL_TARGET_HOOKS += NETOPEER2_CLEANUP
    +
     $(eval $(cmake-package))

(note: I hand-edited the patch, so it may be completely unusable as-is
now, but you at least get the idea...)
I used your snipped and (after fixing some small things) it works. So
I will prepare a patch and send it.

By the way ... after that I also do a patch for the host-sysrepo
dependency and add the executable location of sysrepoctl as MAKE_ENV.

Thanks to all for the help!

--
Heiko
Thread (11 messages) 11 messages, 4 authors, 2021-02-06
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help