From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Mon, 07 Feb 2022 14:33:55 +0100 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1nH49T-008HzR-TQ for lore@lore.pengutronix.de; Mon, 07 Feb 2022 14:33:55 +0100 Received: from localhost ([127.0.0.1] helo=metis.ext.pengutronix.de) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1nH49T-0007OH-5L; Mon, 07 Feb 2022 14:33:55 +0100 Received: from mail.pqgruber.com ([52.59.78.55]) by metis.ext.pengutronix.de with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.92) (envelope-from ) id 1nH495-0007No-8X for ptxdist@pengutronix.de; Mon, 07 Feb 2022 14:33:34 +0100 Received: from workstation (213-47-165-233.cable.dynamic.surfer.at [213.47.165.233]) by mail.pqgruber.com (Postfix) with ESMTPSA id 63463C2FB90 for ; Mon, 7 Feb 2022 14:33:30 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pqgruber.com; s=mail; t=1644240810; bh=Y/o31XfcxbOXPINCBN6DLbOIxUXQ5ATMRum3h1jiW3A=; h=Date:From:To:Subject:References:In-Reply-To:From; b=CZ+VNU3XEZAiwzaNdigNH4I/3pvYM1MluUoqYC4QGk0THFtvKMszBmmwvJ5TvHj/g /Pcie4ZzlDSRm+gzFOxyUAOV1Fjvbq6a5rUciN0etuT3y91tFUrqTtlZ8CKWevAS1p V9FzLwzbumkQGDoUcu9fkcyFSBOPjeqLp7ZYpt2k= Date: Mon, 7 Feb 2022 14:33:29 +0100 From: Clemens Gruber To: ptxdist@pengutronix.de Message-ID: References: <20220204162809.830058-1-clemens.gruber@pqgruber.com> <20220207073936.GQ11273@pengutronix.de> <20220207103909.GB538@falbala.internal.home.lespocky.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.ext.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-102.6 required=4.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS, T_SCC_BODY_TEXT_LINE,USER_IN_WELCOMELIST,USER_IN_WHITELIST autolearn=ham autolearn_force=no version=3.4.2 Subject: Re: [ptxdist] [PATCH 1/3] hiredis: new package X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false Hi Alex, On Mon, Feb 07, 2022 at 02:11:13PM +0100, Alexander Dahl wrote: > Hello Clemens, > > Am Mon, Feb 07, 2022 at 01:54:04PM +0100 schrieb Clemens Gruber: > > Hi, > > > > On Mon, Feb 07, 2022 at 11:39:10AM +0100, Alexander Dahl wrote: > > > Hei hei, > > > > > > On Mon, Feb 07, 2022 at 10:03:21AM +0100, Clemens Gruber wrote: > > > > On Mon, Feb 07, 2022 at 08:39:36AM +0100, Michael Olbrich wrote: > > > > > On Fri, Feb 04, 2022 at 05:28:07PM +0100, Clemens Gruber wrote: > > > > > > Adds a new package for hiredis, a minimalistic C client library for > > > > > > Redis. > > > > > > > > > > > > Signed-off-by: Clemens Gruber > > > > > > --- > > > > > > rules/hiredis.in | 8 ++++++ > > > > > > rules/hiredis.make | 61 ++++++++++++++++++++++++++++++++++++++++++++++ > > > > > > 2 files changed, 69 insertions(+) > > > > > > create mode 100644 rules/hiredis.in > > > > > > create mode 100644 rules/hiredis.make > > > > > > > > > > > > diff --git a/rules/hiredis.in b/rules/hiredis.in > > > > > > new file mode 100644 > > > > > > index 000000000..d416d69c4 > > > > > > --- /dev/null > > > > > > +++ b/rules/hiredis.in > > > > > > @@ -0,0 +1,8 @@ > > > > > > +## SECTION=system_libraries > > > > > > + > > > > > > +config HIREDIS > > > > > > + tristate > > > > > > + select HOST_CMAKE > > > > > > + prompt "hiredis " > > > > > > + help > > > > > > + Hiredis is a minimalistic C client library for the Redis database. > > > > > > diff --git a/rules/hiredis.make b/rules/hiredis.make > > > > > > new file mode 100644 > > > > > > index 000000000..10e6d81d2 > > > > > > --- /dev/null > > > > > > +++ b/rules/hiredis.make > > > > > > @@ -0,0 +1,61 @@ > > > > > > +# -*-makefile-*- > > > > > > +# > > > > > > +# Copyright (C) 2022 by Clemens Gruber > > > > > > +# > > > > > > +# For further information about the PTXdist project and license conditions > > > > > > +# see the README file. > > > > > > +# > > > > > > + > > > > > > +# > > > > > > +# We provide this package > > > > > > +# > > > > > > +PACKAGES-$(PTXCONF_HIREDIS) += hiredis > > > > > > + > > > > > > +# > > > > > > +# Paths and names > > > > > > +# > > > > > > +HIREDIS_VERSION := 1.0.2 > > > > > > +HIREDIS_MD5 := 58e8313188f66ed1be1c220d14a7752e > > > > > > +HIREDIS := hiredis-$(HIREDIS_VERSION) > > > > > > +HIREDIS_SUFFIX := tar.gz > > > > > > +HIREDIS_URL := https://github.com/redis/hiredis/archive/v$(HIREDIS_VERSION).$(HIREDIS_SUFFIX) > > > > > > +HIREDIS_SOURCE := $(SRCDIR)/$(HIREDIS).$(HIREDIS_SUFFIX) > > > > > > +HIREDIS_DIR := $(BUILDDIR)/$(HIREDIS) > > > > > > +HIREDIS_LICENSE := BSD-3-Clause > > > > > > > > > > Please add _LICENSE_FILES as well. > > > > > > > > > > > + > > > > > > +# ---------------------------------------------------------------------------- > > > > > > +# Prepare > > > > > > +# ---------------------------------------------------------------------------- > > > > > > + > > > > > > +# > > > > > > +# cmake > > > > > > +# > > > > > > +HIREDIS_CONF_TOOL := cmake > > > > > > + > > > > > > +HIREDIS_CONF_OPT := \ > > > > > > + $(CROSS_CMAKE_USR) \ > > > > > > + -DDISABLE_TESTS=ON \ > > > > > > + -DENABLE_SSL=OFF > > > > > > + > > > > > > + > > > > > > +# ---------------------------------------------------------------------------- > > > > > > +# Target-Install > > > > > > +# ---------------------------------------------------------------------------- > > > > > > + > > > > > > +$(STATEDIR)/hiredis.targetinstall: > > > > > > + @$(call targetinfo) > > > > > > + > > > > > > + @$(call install_init, hiredis) > > > > > > + @$(call install_fixup, hiredis,PRIORITY,optional) > > > > > > + @$(call install_fixup, hiredis,SECTION,base) > > > > > > + @$(call install_fixup, hiredis,AUTHOR,"Clemens Gruber ") > > > > > > + @$(call install_fixup, hiredis,DESCRIPTION,"Minimalistic C client for Redis") > > > > > > + > > > > > > + @$(call install_lib, hiredis, 0, 0, 0644, libhiredis) > > > > > > + @$(call install_link, hiredis, libhiredis.so.1.0.0, /usr/lib/libhiredis.so) > > > > > > > > > > In general, this link should not be needed. If it really is needed then > > > > > please add a comment why. > > > > > > > > Maybe you can help me out here: If I don't call install_link, the > > > > libhiredis.so link is not present in my rootfs image, even though it is > > > > present in platform-xyz/packages/hiredis-1.0.2/usr/lib and in > > > > platform-xyz/root/usr/lib > > > > > > I would expect this. Usually you have three pieces: > > > > > > libfoo.so.1.0.0 (real binary file) > > > libfoo.so.1 (symlink to libfoo.so.1.0.0, used at runtime by users of the lib) > > > libfoo.so (symlink, used at build time) > > > > > > > Still in my rootfs image, there is only /usr/lib/libhiredis.so.1.0.0 > > > > It is a normal root.ext2 image with ext2fs type ext4. > > > > > > > > Any idea why that is? > > > > > > No idea why that is, but without further knowledge of that lib I would > > > expect to see libhiredis.so.1 on the target instead of libhiredis.so > > > (which is probably only needed in sysroot-target). > > > > I forgot to mention that the libhiredis.so.1 link is also created in > > platform-xyz/packages/.. and platform-xyz/root/.. > > I should probably have added the create_link for that one as well but > > did not need it because a binary blob not under my control dynloads > > /usr/lib/libhiredis.so I have to correct myself here: The libhiredis.so.1 is not created and not present in platform-xyz/root or platform-xyz/packages/hiredis-1.0.2 This was a leftover from tests/experiments of mine. > This sound like a flaw in that binary blob?! That ".1" usually marks > the so version and different numbers are not necessarily compatible, > so by loading libhiredis.so you are at risk of loading an incompatible > library? Yes, you are right. This could lead to problems in the future. > > > Maybe Michael can shed some light on why the links are not present in > > the rootfs. > > Maybe libhiredis makes something different than usual. Haven't looked > into it. O:-) It seems the libhiredis cmake config only sets VERSION but not SOVERSION, so SOVERSION is assumed to be 1.0.0 as well, which explains why there is only one symlink, but not why that one is not present in the rootfs. > > > Just for testing I enabled the creation of a root.tgz file > > (image-root-tgz). There, the links are also missing. > > Mkay. > > Greets > Alex Thanks, Clemens _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de