From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from gravedo.de ([85.214.79.159]) by linuxtogo.org with esmtp (Exim 4.69) (envelope-from ) id 1Oir7W-0007gt-QG for openembedded-devel@lists.openembedded.org; Tue, 10 Aug 2010 17:52:11 +0200 Received: by gravedo.de (Postfix, from userid 5001) id C1FE380C527; Tue, 10 Aug 2010 17:51:54 +0200 (CEST) Received: from [10.1.1.50] (unknown [82.113.106.4]) by gravedo.de (Postfix) with ESMTPSA id B369F80C4FF; Tue, 10 Aug 2010 17:51:39 +0200 (CEST) Message-ID: <4C61757E.20204@gravedo.de> Date: Tue, 10 Aug 2010 17:51:26 +0200 From: Simon Busch User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.12pre) Gecko/20100729 Shredder/3.0.7pre MIME-Version: 1.0 To: openembedded-devel@lists.openembedded.org References: <1279651497-20918-1-git-send-email-morphis@gravedo.de> <1279651497-20918-2-git-send-email-morphis@gravedo.de> <20100726074601.GA27921@excalibur.local> <4C4DE8FA.7060707@gravedo.de> <20100728134438.GA20716@excalibur.local> <4C5066C9.4060902@gravedo.de> <20100728180735.GC20716@excalibur.local> In-Reply-To: <20100728180735.GC20716@excalibur.local> X-Enigmail-Version: 1.0.1 X-SA-Exim-Connect-IP: 85.214.79.159 X-SA-Exim-Mail-From: morphis@gravedo.de X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on discovery X-Spam-Level: X-Spam-Status: No, score=-2.6 required=5.0 tests=AWL,BAYES_00 autolearn=ham version=3.2.5 X-SA-Exim-Version: 4.2.1 (built Wed, 25 Jun 2008 17:20:07 +0000) X-SA-Exim-Scanned: Yes (on linuxtogo.org) Subject: Re: [PATCH 1/4] lvm2: rebase all recipes on a global lvm2.inc recipe X-BeenThere: openembedded-devel@lists.openembedded.org X-Mailman-Version: 2.1.11 Precedence: list Reply-To: openembedded-devel@lists.openembedded.org List-Id: Using the OpenEmbedded metadata to build Distributions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 10 Aug 2010 15:52:11 -0000 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Am 28.07.2010 20:07, schrieb Stefan Schmidt: > Hello. > > On Wed, 2010-07-28 at 19:20, Simon Busch wrote: >> On 28.07.2010 15:44, Stefan Schmidt wrote: >>> >>> On Mon, 2010-07-26 at 21:58, Simon Busch wrote: >>>> On 26.07.2010 09:46, Stefan Schmidt wrote: >>>>> Hello. >>>>> >>>>> On Tue, 2010-07-20 at 20:44, Simon Busch wrote: >>>>>> This rebases all specific versions of lvm2 on a global recipe lvm2.inc which defines the >>>>>> common parameters for building lvm2. Staging is overwritten as we don't need any of the >>>>>> executables or manpages the build of lvm2 produces for any related builds. >>>>>> >>>>>> Signed-off-by: Simon Busch >>>>> >>>>> Looks good. Two comments below. >>>> >>>> Thank you for commenting this patches! >>>> >>>>> >>>>>> diff --git a/recipes/lvm2/lvm2.inc b/recipes/lvm2/lvm2.inc >>>>>> new file mode 100644 >>>>>> index 0000000..a7e37b5 >>>>>> --- /dev/null >>>>>> +++ b/recipes/lvm2/lvm2.inc >>>>>> @@ -0,0 +1,20 @@ >>>>>> +SECTION = "utils" >>>>>> +DESCRIPTION = "LVM2 is a set of utilities to manage logical volumes in Linux." >>>>>> +LICENSE = "GPL" >>>>>> +DEPENDS = "device-mapper" >>>>>> +INC_PR = "r2" >>>>>> + >>>>>> +S = "${WORKDIR}/LVM2.${PV}" >>>>>> +SRC_URI = "ftp://sources.redhat.com/pub/lvm2/old/LVM2.${PV}.tgz \ >>>>>> + file://crosscompile_fix.patch" >>>>>> + >>>>>> +# Unset user/group to unbreak install. >>>>>> +EXTRA_OECONF = "--with-user= --with-group= --disable-o_direct" >>>>>> +EXTRA_OECONF_arm = "--with-user= --with-group= --disable-o_direct" >>>>> >>>>> I can see that you just merged this in from an already existing recipe, but do >>>>> we know why we have an overrirde for ARM here which changes nothing? Looks bogus >>>>> to me. >>>> >>>> Hm, you are right. I checked it with recompiling without the >>>> EXTRA_OECONF_arm statement. It works fine. Is it ok if I supply an extra >>>> patch removing this statement or should I rework this patch? >>> >>> Both is fine with me. >>> >>>>>> +inherit autotools >>>>>> + >>>>>> +# We don't need to stage anything (the executables are no needed at build time by any >>>>>> +# other recipe) >>>>>> +do_stage() { >>>>>> +} >>>>> >>>>> While we don't need the executables the don't hurd either IMHO. Can we get rid >>>>> of this? >>>>> >>>> >>>> The problem was, that the executables and manpages where installed into >>>> the staging dir and never removed, when the packages was clean ur >>>> purged. So I added the empty do_stage block to avoid stage of anything >>>> from this package as we don't need them anyway. >>> >>> Hm, this smells like a bug somewhere else. Did the staged binaries bring in >>> other problems or could they get staged until we find out why they are not >>> removed? >> >> Staging this binaries let the recompile of the same package fail as it >> tries to install the binaries + man pages again. >> Any hints where the bug could be? Within the staging logic? > > Good question. Maybe only a bug in the clean step that it ignores the files > during cleaning. Anybody else an idea? > > You could also add a do_install_append which removes the binaries again. Hm, is that the best solution? I think it's better to stage nothing as we now we don't really need anything to be staged. So is it ok leaving this patch as it is and commit it? regards, morphis