[oe] [meta-java][PATCH 2/2] openjdk-8: combine the -dbg package
Richard Leitner
richard.leitner at skidata.com
Tue Jun 12 08:01:56 UTC 2018
Hi Wenlin,
as we clarified the "one -dbg package per recipe" thing I have some
comments on the patch itself.
On 06/07/2018 11:24 AM, Wenlin Kang wrote:
> One recipe should only have one -dbg package, because OE only picks
> up all .debug file into the last one -dbg package listed in variable
> PACKAGES.
May you please include a reference to that statement from "Ross Burton"?
>
> Signed-off-by: Wenlin Kang <wenlin.kang at windriver.com>
> ---
> recipes-core/openjdk/openjdk-8_16xbyy.bb | 10 +++-------
> 1 file changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/recipes-core/openjdk/openjdk-8_16xbyy.bb b/recipes-core/openjdk/openjdk-8_16xbyy.bb
> index dd0ed1d..1d7d794 100644
> --- a/recipes-core/openjdk/openjdk-8_16xbyy.bb
> +++ b/recipes-core/openjdk/openjdk-8_16xbyy.bb
> @@ -14,7 +14,6 @@ do_install() {
> }
>
> PACKAGES_append = " \
> - ${PN}-demo-dbg \
> ${PN}-demo \
> ${PN}-source \
> "
> @@ -47,12 +46,6 @@ FILES_${PN}-dbg_append = "\
> ${JDK_HOME}/jre/lib/${JDK_ARCH}/headless/.debug/ \
> ${JDK_HOME}/jre/lib/${JDK_ARCH}/xawt/.debug/ \
> ${JDK_HOME}/jre/lib/${JDK_ARCH}/client/.debug/ \
> -"
> -
> -FILES_${PN}-demo = " ${JDK_HOME}/demo ${JDK_HOME}/sample "
> -RDEPENDS_${PN}-demo = " ${PN} "
> -
> -FILES_${PN}-demo-dbg = "\
> ${JDK_HOME}/demo/jvmti/gctest/lib/.debug/ \
> ${JDK_HOME}/demo/jvmti/heapTracker/lib/.debug/ \
> ${JDK_HOME}/demo/jvmti/heapViewer/lib/.debug/ \
> @@ -64,6 +57,9 @@ FILES_${PN}-demo-dbg = "\
> ${JDK_HOME}/demo/jvmti/compiledMethodLoad/lib/.debug/ \
> "
AFAIK as all ".debug" are included in the -dbg packages it should
be fine if you remove the whole FILES_${PN}-dbg_append definition.
>
> +FILES_${PN}-demo = " ${JDK_HOME}/demo ${JDK_HOME}/sample "
> +RDEPENDS_${PN}-demo = " ${PN} "
> +
> FILES_${PN}-doc_append = "\
> ${JDK_HOME}/man \
> ${JDK_HOME}/ASSEMBLY_EXCEPTION \
thanks®ards;Richard.L
More information about the Openembedded-devel
mailing list