From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Greylist: delayed 450 seconds by postgrey-1.34 at layers.openembedded.org; Fri, 12 Feb 2016 08:37:08 UTC Received: from mail.analogue-micro.com (mail.analogue-micro.com [217.144.149.242]) by mail.openembedded.org (Postfix) with ESMTP id 135126FF9F for ; Fri, 12 Feb 2016 08:37:08 +0000 (UTC) Received: by mail.analogue-micro.com (Postfix, from userid 999) id 2A26868A01C; Fri, 12 Feb 2016 08:37:09 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on loki.analogue-micro-ltd.com X-Spam-Level: ** X-Spam-Status: No, score=2.5 required=5.0 tests=ALL_TRUSTED,BAYES_50, DNS_FROM_AHBL_RHSBL autolearn=no version=3.3.2 Received: from zeus.mlbassoc.com (unknown [10.8.0.2]) by mail.analogue-micro.com (Postfix) with ESMTP id 9621468A019; Fri, 12 Feb 2016 08:37:08 +0000 (GMT) Received: from [127.0.0.1] (localhost.localdomain [127.0.0.1]) by zeus.mlbassoc.com (Postfix) with ESMTP id D8A2E674009E; Fri, 12 Feb 2016 09:37:07 +0100 (CET) To: Richard Purdie , openembedded-core@lists.openembedded.org References: <1455249507-30989-1-git-send-email-gary@mlbassoc.com> <1455265043.16142.267.camel@linuxfoundation.org> <56BD97ED.1040802@mlbassoc.com> <1455265928.16142.269.camel@linuxfoundation.org> From: Gary Thomas Message-ID: <56BD99B2.2010004@mlbassoc.com> Date: Fri, 12 Feb 2016 09:37:06 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <1455265928.16142.269.camel@linuxfoundation.org> Subject: Re: [PATCH] ffmpeg: Add -dbg packages for all libraries X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 12 Feb 2016 08:37:09 -0000 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit On 2016-02-12 09:32, Richard Purdie wrote: > On Fri, 2016-02-12 at 09:29 +0100, Gary Thomas wrote: >> On 2016-02-12 09:17, Richard Purdie wrote: >>> On Fri, 2016-02-12 at 04:58 +0100, Gary Thomas wrote: >>>> Improve the packaging of the libraries built by this recipe. >>>> These >>>> are created using special code in the recipe and the debug (-dbg) >>>> packages were not being created. Adding these packages allow the >>>> libraries in question to be debugged using GDB. >>> >>> This isn't really policy, the policy is one -dbg package per recipe >>> and >>> that is how the dependency chains and dbg-pkgs in IMAGE_FEATURES >>> work >>> and so on. >>> >>> I'm not arguing this is perfect, its not and I would like to see it >>> change. It is how it all works today though. Is there a pressing >>> reason >>> we need to do something different here? >> >> Without this change, none of the [renamed] libraries generated by >> the ffmpeg recipe have debug symbols available. As is, the recipe >> is generating separate -dev packages for each library - how is that >> different [policy-wise]? >> >> Should the -dev and -dbg info for the libraries be bundled into >> ffmpeg-dbg and ffmpeg-dev? Or perhaps the machinations generating >> the -dev packages in that recipe are just wrong? > > There should only be one -dev package too. > > As you saying the debug symbols are getting placed into the -dev > packages? They must be getting placed and hence packaged somewhere? I'm not sure where they were going before this change. It does look like this recipe is packaging things incorrectly, at least against policy. I think the biggest thing they were attempting to achieve was packaging of static development libraries (*.a) in their own packages. Should all of this just be in ffmpeg-dev? I could try just disabling their special packaging and see how well it works. -- ------------------------------------------------------------ Gary Thomas | Consulting for the MLB Associates | Embedded world ------------------------------------------------------------