Hi,
thank you for your patch! Please see my comments below...
On 07/17/2018 09:20 PM, Matthew McClain wrote:
Renaming the function to anything else allows the recipe to be parsed. The
problem appears to be the word "_remove_". This affects both the sumo and
master branches.
In the subject you say it causes problems with the pyro branch, but here
you say sumo and master... So which branches are really affected? Can
you please adapt the title/commit-message accordingly?
Do you (or somebody else here) have any clue why the "remove" keyword
triggers such a problem (since pyro?).
If I get it right the mentioned parsing error only occurs when we have
no X11 (in PACKAGECONFIG). Is that true? Can you please mention it in
the commit message?
Signed-off-by: Matthew McClain <[email protected]>
---
recipes-core/openjdk/openjdk-8-common.inc | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/recipes-core/openjdk/openjdk-8-common.inc
b/recipes-core/openjdk/openjdk-8-common.inc
index b2020c3..fbb10be 100644
--- a/recipes-core/openjdk/openjdk-8-common.inc
+++ b/recipes-core/openjdk/openjdk-8-common.inc
@@ -39,7 +39,7 @@ do_unpack_extract_submodules () {
tar xjf ${WORKDIR}/${NASHORN_FILE} --transform
"s,-${NASHORN_CHANGESET},,g"
}
-do_unpack_remove_X11_wrappers() {
+do_unpack_delete_X11_wrappers() {
find ${S}/jdk/src/solaris/classes/sun/awt/X11 -maxdepth 1 -name '*.java'
-delete
}
@@ -153,7 +153,7 @@ def jdk_configure_options(d):
return options[3]
do_unpack[postfuncs] += "do_unpack_extract_submodules"
-do_unpack[postfuncs] += "${@bb.utils.contains('PACKAGECONFIG', 'x11', '',
'do_unpack_remove_X11_wrappers', d)}"
+do_unpack[postfuncs] += "${@bb.utils.contains('PACKAGECONFIG', 'x11', '',
'do_unpack_delete_X11_wrappers', d)}"
do_configure_prepend() {
export ${@jdk_environment_options(d)}
Otherwise the patch itself looks fine to me.
Thank you!
regards;Richard.L
--
_______________________________________________
Openembedded-devel mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-devel
