[PATCH 0/1] udev: ship source files to related dbg package

[PATCH 0/1] udev: ship source files to related dbg package

[wenzong.fan]

From: Wenzong Fan <wenzong.fan@windriver.com>

Just ship these sources to their own dbg packages instead of udev-dbg:

    libudev* -> libudev-dbg
    gudev*   -> libgudev-dbg
    others   -> udev-dbg

The following changes since commit d6cc7c8ed76c8b1117cf03c7bd4b0742f98f79b3:

  toolchain-scripts/meta-environment: Further cleanup code duplication (2013-11-06 11:02:27 +0000)

are available in the git repository at:

  git://git.pokylinux.org/poky-contrib wenzong/udev-dbg-fix
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=wenzong/udev-dbg-fix

Wenzong Fan (1):
  udev: ship source files to related dbg package

 meta/recipes-core/udev/udev.inc |    7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

-- 
1.7.9.5

[PATCH 1/1] udev: ship source files to related dbg package

[wenzong.fan]

From: Wenzong Fan <wenzong.fan@windriver.com>

Currently all source files will be shipped to udev-dbg regardless they
should belong libudev-dbg or libgudev-dbg, just update the FILES list
to ship those files to their own packages:

    libudev-dbg: libudev*, libgudev-dbg: gudev*, udev-dbg: others

Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
---
 meta/recipes-core/udev/udev.inc |    7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-core/udev/udev.inc b/meta/recipes-core/udev/udev.inc
index 2ff8f00..b4327bd 100644
--- a/meta/recipes-core/udev/udev.inc
+++ b/meta/recipes-core/udev/udev.inc
@@ -63,11 +63,14 @@ FILES_${PN}-dbg += "${base_libdir}/udev/.debug/*"
 FILES_${PN}-dbg += "${nonarch_base_libdir}/udev/.debug/*"
 FILES_${PN}-dev = "${datadir}/pkgconfig/udev.pc"
 FILES_libudev = "${base_libdir}/libudev.so.*"
-FILES_libudev-dbg = "${base_libdir}/.debug/libudev.so.*"
+FILES_libudev-dbg = "/usr/src/debug/${BPN}/${PV}-${PR}/${BP}/src/libudev* \
+                     ${base_libdir}/.debug/libudev.so.*"
 FILES_libudev-dev = "${includedir}/libudev.h ${libdir}/libudev.so ${libdir}/libudev.la \
                      ${libdir}/libudev.a ${libdir}/pkgconfig/libudev.pc"
 FILES_libgudev = "${base_libdir}/libgudev*.so.* ${libdir}/libgudev*.so.*"
-FILES_libgudev-dbg = "${base_libdir}/.debug/libgudev*.so.* ${libdir}/.debug/libgudev*.so.*"
+FILES_libgudev-dbg = "/usr/src/debug/${BPN}/${PV}-${PR}/${BP}/src/gudev* \
+                      /usr/src/debug/${BPN}/${PV}-${PR}/build/src/gudev* \
+                      ${base_libdir}/.debug/libgudev*.so.* ${libdir}/.debug/libgudev*.so.*"
 FILES_libgudev-dev = "${includedir}/gudev* ${libdir}/libgudev*.so ${libdir}/libgudev*.la \
                      ${libdir}/libgudev*.a ${libdir}/pkgconfig/gudev*.pc"
 FILES_udev-cache = "${sysconfdir}/init.d/udev-cache ${sysconfdir}/default/udev-cache"
-- 
1.7.9.5

Re: [PATCH 0/1] udev: ship source files to related dbg package

[Burton, Ross]

On 7 November 2013 11:03,  <wenzong.fan@windriver.com> wrote:
> From: Wenzong Fan <wenzong.fan@windriver.com>
>
> Just ship these sources to their own dbg packages instead of udev-dbg:
>
>     libudev* -> libudev-dbg
>     gudev*   -> libgudev-dbg
>     others   -> udev-dbg

Why do this?  Multiple -dbg packages could make sense in a recipe
which builds a multi-gigabyte -dbg package (such as webkit) but what's
the rationale for doing this in udev?

Ross

Re: [PATCH 0/1] udev: ship source files to related dbg package

[wenzong fan]

On 11/07/2013 07:12 PM, Burton, Ross wrote:
> On 7 November 2013 11:03,  <wenzong.fan@windriver.com> wrote:
>> From: Wenzong Fan <wenzong.fan@windriver.com>
>>
>> Just ship these sources to their own dbg packages instead of udev-dbg:
>>
>>      libudev* -> libudev-dbg
>>      gudev*   -> libgudev-dbg
>>      others   -> udev-dbg
>
> Why do this?  Multiple -dbg packages could make sense in a recipe
> which builds a multi-gigabyte -dbg package (such as webkit) but what's
> the rationale for doing this in udev?

Actually I don't know clear about why it needs three -dbg packages, 
looks they have been there since very early commits of udev. I suspect 
that udev/libudev/libgudev are independent each other, so they are 
shipped into different packages (base/-dev/-dbg). This patch only ships 
their source code to -dbg packages accordingly.

Thanks
Wenzong

>
> Ross
>
>

Re: [PATCH 0/1] udev: ship source files to related dbg package

[Richard Purdie]

On Fri, 2013-11-08 at 16:10 +0800, wenzong fan wrote:
> On 11/07/2013 07:12 PM, Burton, Ross wrote:
> > On 7 November 2013 11:03,  <wenzong.fan@windriver.com> wrote:
> >> From: Wenzong Fan <wenzong.fan@windriver.com>
> >>
> >> Just ship these sources to their own dbg packages instead of udev-dbg:
> >>
> >>      libudev* -> libudev-dbg
> >>      gudev*   -> libgudev-dbg
> >>      others   -> udev-dbg
> >
> > Why do this?  Multiple -dbg packages could make sense in a recipe
> > which builds a multi-gigabyte -dbg package (such as webkit) but what's
> > the rationale for doing this in udev?
> 
> Actually I don't know clear about why it needs three -dbg packages, 
> looks they have been there since very early commits of udev. I suspect 
> that udev/libudev/libgudev are independent each other, so they are 
> shipped into different packages (base/-dev/-dbg). This patch only ships 
> their source code to -dbg packages accordingly.

We don't support multiple -dbg packages and this looks like an error.

Cheers,

Richard

Re: [PATCH 0/1] udev: ship source files to related dbg package

[wenzong fan]

On 11/08/2013 04:18 PM, Richard Purdie wrote:
> On Fri, 2013-11-08 at 16:10 +0800, wenzong fan wrote:
>> On 11/07/2013 07:12 PM, Burton, Ross wrote:
>>> On 7 November 2013 11:03,  <wenzong.fan@windriver.com> wrote:
>>>> From: Wenzong Fan <wenzong.fan@windriver.com>
>>>>
>>>> Just ship these sources to their own dbg packages instead of udev-dbg:
>>>>
>>>>       libudev* -> libudev-dbg
>>>>       gudev*   -> libgudev-dbg
>>>>       others   -> udev-dbg
>>>
>>> Why do this?  Multiple -dbg packages could make sense in a recipe
>>> which builds a multi-gigabyte -dbg package (such as webkit) but what's
>>> the rationale for doing this in udev?
>>
>> Actually I don't know clear about why it needs three -dbg packages,
>> looks they have been there since very early commits of udev. I suspect
>> that udev/libudev/libgudev are independent each other, so they are
>> shipped into different packages (base/-dev/-dbg). This patch only ships
>> their source code to -dbg packages accordingly.
>
> We don't support multiple -dbg packages and this looks like an error.

What about the -dev packages?

So the correct fix is removing below configs, at least -dbg configs, right?

+ FILES_libudev = "${base_libdir}/libudev.so.*"
+ FILES_libudev-dbg = "${base_libdir}/.debug/libudev.so.*"
+ FILES_libudev-dev = "${includedir}/libudev.h ${libdir}/libudev.so 
${libdir}/libudev.la \
                      ${libdir}/libudev.a ${libdir}/pkgconfig/libudev.pc"

+ FILES_libgudev = "${base_libdir}/libgudev*.so.* ${libdir}/libgudev*.so.*"
+ FILES_libgudev-dbg = "${base_libdir}/.debug/libgudev*.so.* 
${libdir}/.debug/libgudev*.so.*"
+ FILES_libgudev-dev = "${includedir}/gudev* ${libdir}/libgudev*.so 
${libdir}/libgudev*.la \
                      ${libdir}/libgudev*.a ${libdir}/pkgconfig/gudev*.pc"


Thanks
Wenzong

>
> Cheers,
>
> Richard
>
>
>

Re: [PATCH 0/1] udev: ship source files to related dbg package

[wenzong fan]

On 11/08/2013 04:18 PM, Richard Purdie wrote:
> On Fri, 2013-11-08 at 16:10 +0800, wenzong fan wrote:
>> On 11/07/2013 07:12 PM, Burton, Ross wrote:
>>> On 7 November 2013 11:03,  <wenzong.fan@windriver.com> wrote:
>>>> From: Wenzong Fan <wenzong.fan@windriver.com>
>>>>
>>>> Just ship these sources to their own dbg packages instead of udev-dbg:
>>>>
>>>>       libudev* -> libudev-dbg
>>>>       gudev*   -> libgudev-dbg
>>>>       others   -> udev-dbg
>>>
>>> Why do this?  Multiple -dbg packages could make sense in a recipe
>>> which builds a multi-gigabyte -dbg package (such as webkit) but what's
>>> the rationale for doing this in udev?
>>
>> Actually I don't know clear about why it needs three -dbg packages,
>> looks they have been there since very early commits of udev. I suspect
>> that udev/libudev/libgudev are independent each other, so they are
>> shipped into different packages (base/-dev/-dbg). This patch only ships
>> their source code to -dbg packages accordingly.
>
> We don't support multiple -dbg packages and this looks like an error.

Ok, another patch for removing the extra -dbg packages has been sent 
with subject:

     udev: remove libudev-dbg and libgudev-dbg

Thanks
Wenzong

>
> Cheers,
>
> Richard
>
>
>