Re: [PATCH][V2] perf: split package

[Chong Lu <Chong.Lu@windriver.com>]

[-- Attachment #1: Type: text/plain, Size: 1478 bytes --]


On 07/24/2014 01:34 AM, Jacob Kroon wrote:
> On Mon, Jul 21, 2014 at 4:03 AM, Chong Lu <Chong.Lu@windriver.com 
> <mailto:Chong.Lu@windriver.com>> wrote:
>
>     Currently, perf can't split to perf-archive, perf-tests,
>     perf-python and
>     perf-perl. All files are included in perf package.
>     Change the files paths to make split successfully and add perf package
>     RDEPEND on the other four to keep it consistent.
>
>     Signed-off-by: Chong Lu <Chong.Lu@windriver.com
>     <mailto:Chong.Lu@windriver.com>>
>     ---
>      meta/recipes-kernel/perf/perf.bb <http://perf.bb> | 9 +++++----
>      1 file changed, 5 insertions(+), 4 deletions(-)
>
>     diff --git a/meta/recipes-kernel/perf/perf.bb <http://perf.bb>
>     b/meta/recipes-kernel/perf/perf.bb <http://perf.bb>
>     index 20968f9..9f9741e 100644
>     --- a/meta/recipes-kernel/perf/perf.bb <http://perf.bb>
>     +++ b/meta/recipes-kernel/perf/perf.bb <http://perf.bb>
>     @@ -157,6 +157,7 @@ PACKAGE_ARCH = "${MACHINE_ARCH}"
>      PACKAGES =+ "${PN}-archive ${PN}-tests ${PN}-perl ${PN}-python"
>
>      RDEPENDS_${PN} += "elfutils"
>     +RDEPENDS_${PN} ?= "${PN}-archive ${PN}-tests ${PN}-python ${PN}-perl"
>
>
> Is this correct ? First RDEPENDS is appended "elfutils", won't this 
> make it pointless to add a default value in the line below ?
Please ignore this patch, I just test in kernel 3.14, but maybe it 
depend on kernel version.

Thanks
Chong

[-- Attachment #2: Type: text/html, Size: 3168 bytes --]