All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [meta-java][PATCH] icedtea7-native: set unique download file names
@ 2020-06-23 12:23 Vyacheslav Yurkov
  2020-06-23 18:08 ` [oe] " Richard Leitner
  0 siblings, 1 reply; 3+ messages in thread
From: Vyacheslav Yurkov @ 2020-06-23 12:23 UTC (permalink / raw
  To: openembedded-devel; +Cc: Vyacheslav Yurkov

From: Vyacheslav Yurkov <Vyacheslav.Yurkov@bruker.com>

Signed-off-by: Vyacheslav Yurkov <Vyacheslav.Yurkov@bruker.com>
---
 recipes-core/icedtea/icedtea7-native.inc      | 14 +++----
 .../icedtea/openjdk-7-release-03b147.inc      | 40 +++++++++++--------
 2 files changed, 31 insertions(+), 23 deletions(-)

diff --git a/recipes-core/icedtea/icedtea7-native.inc b/recipes-core/icedtea/icedtea7-native.inc
index b0b9419..70738e7 100644
--- a/recipes-core/icedtea/icedtea7-native.inc
+++ b/recipes-core/icedtea/icedtea7-native.inc
@@ -83,13 +83,13 @@ EXTRA_OECONF = "\
 	--with-javac=${STAGING_BINDIR_NATIVE}/javac \
 	--with-rhino=${STAGING_DATADIR_NATIVE}/java/rhino.jar \
 	\
-	--with-openjdk-src-zip=${DL_DIR}/${OPENJDK_FILE} \
-        --with-hotspot-src-zip=${DL_DIR}/${HOTSPOT_FILE} \
-        --with-corba-src-zip=${DL_DIR}/${CORBA_FILE} \
-        --with-jaxp-src-zip=${DL_DIR}/${JAXP_FILE} \
-        --with-jaxws-src-zip=${DL_DIR}/${JAXWS_FILE} \
-        --with-jdk-src-zip=${DL_DIR}/${JDK_FILE} \
-        --with-langtools-src-zip=${DL_DIR}/${LANGTOOLS_FILE} \
+	--with-openjdk-src-zip=${DL_DIR}/${OPENJDK_FILE_DOWNLOAD} \
+        --with-hotspot-src-zip=${DL_DIR}/${HOTSPOT_FILE_DOWNLOAD} \
+        --with-corba-src-zip=${DL_DIR}/${CORBA_FILE_DOWNLOAD} \
+        --with-jaxp-src-zip=${DL_DIR}/${JAXP_FILE_DOWNLOAD} \
+        --with-jaxws-src-zip=${DL_DIR}/${JAXWS_FILE_DOWNLOAD} \
+        --with-jdk-src-zip=${DL_DIR}/${JDK_FILE_DOWNLOAD} \
+        --with-langtools-src-zip=${DL_DIR}/${LANGTOOLS_FILE_DOWNLOAD} \
         ac_cv_path_UNZIP_BIN="${STAGING_BINDIR_NATIVE}/unzip" \
   "
 
diff --git a/recipes-core/icedtea/openjdk-7-release-03b147.inc b/recipes-core/icedtea/openjdk-7-release-03b147.inc
index 9816c0c..b504667 100644
--- a/recipes-core/icedtea/openjdk-7-release-03b147.inc
+++ b/recipes-core/icedtea/openjdk-7-release-03b147.inc
@@ -13,50 +13,58 @@ ICEDTEA_PREFIX = "icedtea7-forest-2.1"
 ICEDTEA_HG_URL = "http://icedtea.classpath.org/hg/release/${ICEDTEA_PREFIX}"
 
 OPENJDK_CHANGESET = "22cc03983e20"
-OPENJDK_FILE = "${OPENJDK_CHANGESET}.tar.bz2"
-OPENJDK_URI = "${ICEDTEA_HG_URL}/archive/${OPENJDK_FILE};name=openjdk;unpack=false"
+OPENJDK_FILE_UPSTREAM = "${OPENJDK_CHANGESET}.tar.bz2"
+OPENJDK_FILE_DOWNLOAD = "openjdk-${OPENJDK_CHANGESET}.tar.bz2"
+OPENJDK_URI = "${ICEDTEA_HG_URL}/archive/${OPENJDK_FILE_UPSTREAM};name=openjdk;unpack=false;downloadfilename=${OPENJDK_FILE_DOWNLOAD}"
 SRC_URI[openjdk.md5sum] = "f98b0f7a69f9427a5b3775dddddc89cb"
 SRC_URI[openjdk.sha256sum] = "758227833982371e3a94def84226a947f8dbbd176517f313b8cb96840bdea41e"
 
 HOTSPOT_CHANGESET = "8b7c4c5f6ba9"
-HOTSPOT_FILE = "${HOTSPOT_CHANGESET}.tar.gz"
-HOTSPOT_URI = "${ICEDTEA_HG_URL}/hotspot/archive/${HOTSPOT_FILE};name=hotspot;unpack=false"
+HOTSPOT_FILE_UPSTREAM = "${HOTSPOT_CHANGESET}.tar.gz"
+HOTSPOT_FILE_DOWNLOAD = "hotspot-${HOTSPOT_CHANGESET}.tar.gz"
+HOTSPOT_URI = "${ICEDTEA_HG_URL}/hotspot/archive/${HOTSPOT_FILE_UPSTREAM};name=hotspot;unpack=false;downloadfilename=${HOTSPOT_FILE_DOWNLOAD}"
 SRC_URI[hotspot.md5sum] = "cb45a8f19d77acc1f4f48e75e36a27fe"
 SRC_URI[hotspot.sha256sum] = "22866990d143e76ced94b76defa3051e5e5d9a51fd272d63daa0df272d6406a2"
 
 CORBA_CHANGESET = "5617f6c5cc94"
-CORBA_FILE = "${CORBA_CHANGESET}.tar.gz"
-CORBA_URI = "${ICEDTEA_HG_URL}/corba/archive/${CORBA_FILE};name=corba;unpack=false"
+CORBA_FILE_UPSTREAM = "${CORBA_CHANGESET}.tar.gz"
+CORBA_FILE_DOWNLOAD = "corba-${CORBA_CHANGESET}.tar.gz"
+CORBA_URI = "${ICEDTEA_HG_URL}/corba/archive/${CORBA_FILE_UPSTREAM};name=corba;unpack=false;downloadfilename=${CORBA_FILE_DOWNLOAD}"
 SRC_URI[corba.md5sum] = "920f1a788a7fdef29a5cd70892331251"
 SRC_URI[corba.sha256sum] = "963915483530f311ff313635f79ed11ea7ce5a1c0dbee5d1acb1994132857fa3"
 
 JAXP_CHANGESET = "7a8825b15df6"
-JAXP_FILE = "${JAXP_CHANGESET}.tar.gz"
-JAXP_URI = "${ICEDTEA_HG_URL}/jaxp/archive/${JAXP_FILE};name=jaxp;unpack=false"
+JAXP_FILE_UPSTREAM = "${JAXP_CHANGESET}.tar.gz"
+JAXP_FILE_DOWNLOAD = "jaxp-${JAXP_CHANGESET}.tar.gz"
+JAXP_URI = "${ICEDTEA_HG_URL}/jaxp/archive/${JAXP_FILE_UPSTREAM};name=jaxp;unpack=false;downloadfilename=${JAXP_FILE_DOWNLOAD}"
 SRC_URI[jaxp.md5sum] = "a0982b6df79739127871e355a40ca5a6"
 SRC_URI[jaxp.sha256sum] = "297c8dbeed5afa5395b6be06b17282d53f8a888c2909074dfc1605afc0daf1c2"
 
 JAXWS_CHANGESET = "7edfbfe974f2"
-JAXWS_FILE = "${JAXWS_CHANGESET}.tar.gz"
-JAXWS_URI = "${ICEDTEA_HG_URL}/jaxws/archive/${JAXWS_FILE};name=jaxws;unpack=false"
+JAXWS_FILE_UPSTREAM = "${JAXWS_CHANGESET}.tar.gz"
+JAXWS_FILE_DOWNLOAD = "jaxws-${JAXWS_CHANGESET}.tar.gz"
+JAXWS_URI = "${ICEDTEA_HG_URL}/jaxws/archive/${JAXWS_FILE_UPSTREAM};name=jaxws;unpack=false;downloadfilename=${JAXWS_FILE_DOWNLOAD}"
 SRC_URI[jaxws.md5sum] = "2deecec809ea8e38835678889c2bc3ac"
 SRC_URI[jaxws.sha256sum] = "823ca6b8fd780bf1b1565a90cfc8ec6f0ee422a5d25ffb20b1ce0272dc4955aa"
 
 JDK_CHANGESET = "d5ddeffc4651"
-JDK_FILE = "${JDK_CHANGESET}.tar.gz"
-JDK_URI = "${ICEDTEA_HG_URL}/jdk/archive/${JDK_FILE};name=jdk;unpack=false"
+JDK_FILE_UPSTREAM = "${JDK_CHANGESET}.tar.gz"
+JDK_FILE_DOWNLOAD = "jdk-${JDK_CHANGESET}.tar.gz"
+JDK_URI = "${ICEDTEA_HG_URL}/jdk/archive/${JDK_FILE_UPSTREAM};name=jdk;unpack=false;downloadfilename=${JDK_FILE_DOWNLOAD}"
 SRC_URI[jdk.md5sum] = "27c28d6be213df818490e69f12efe475"
 SRC_URI[jdk.sha256sum] = "e83e6234842376147a4c32451f23d629ba107e908ebdb0f367657f9cc6c1be98"
 
 LANGTOOLS_CHANGESET = "b534c4c6cd9b"
-LANGTOOLS_FILE = "${LANGTOOLS_CHANGESET}.tar.gz"
-LANGTOOLS_URI = "${ICEDTEA_HG_URL}/langtools/archive/${LANGTOOLS_FILE};name=langtools;unpack=false"
+LANGTOOLS_FILE_UPSTREAM = "${LANGTOOLS_CHANGESET}.tar.gz"
+LANGTOOLS_FILE_DOWNLOAD = "langtools-${LANGTOOLS_CHANGESET}.tar.gz"
+LANGTOOLS_URI = "${ICEDTEA_HG_URL}/langtools/archive/${LANGTOOLS_FILE_UPSTREAM};name=langtools;unpack=false;downloadfilename=${LANGTOOLS_FILE_DOWNLOAD}"
 SRC_URI[langtools.md5sum] = "16b72f3fea8ac4e5b50de693e87f9c53"
 SRC_URI[langtools.sha256sum] = "c77dd42d83f06d18fa8aa65c93d137608a2e02cf24b68fc10d7b1c0bcc12a93c"
 
 CACAO_VERSION = "a567bcb7f589"
-CACAO_FILE = "${CACAO_VERSION}.tar.gz"
-CACAO_URI = "http://icedtea.wildebeest.org/download/drops/cacao/${CACAO_FILE};name=cacao;unpack=false"
+CACAO_FILE_UPSTREAM = "${CACAO_VERSION}.tar.gz"
+CACAO_FILE_DOWNLOAD = "cacao-${CACAO_VERSION}.tar.gz"
+CACAO_URI = "http://icedtea.wildebeest.org/download/drops/cacao/${CACAO_FILE_UPSTREAM};name=cacao;unpack=false;downloadfilename=${CACAO_FILE_DOWNLOAD}"
 SRC_URI[cacao.md5sum] = "ec1def7f0d3d25e9e7da47c480f26a73"
 SRC_URI[cacao.sha256sum] = "d49f79debc131a5694cae6ab3ba2864e7f3249ee8d9dc09aae8afdd4dc6b09f9"
 
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [oe] [meta-java][PATCH] icedtea7-native: set unique download file names
  2020-06-23 12:23 [meta-java][PATCH] icedtea7-native: set unique download file names Vyacheslav Yurkov
@ 2020-06-23 18:08 ` Richard Leitner
  2020-06-24  5:23   ` Vyacheslav Yurkov
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Leitner @ 2020-06-23 18:08 UTC (permalink / raw
  To: Vyacheslav Yurkov; +Cc: openembedded-devel, Vyacheslav Yurkov

Hi Vyacheslav,
thanks for your patch. Please find my comments inline.

On Tue, Jun 23, 2020 at 02:23:41PM +0200, Vyacheslav Yurkov wrote:
> From: Vyacheslav Yurkov <Vyacheslav.Yurkov@bruker.com>

It would be great if you could describe the change in a few more words
here. Thanks!

> 
> Signed-off-by: Vyacheslav Yurkov <Vyacheslav.Yurkov@bruker.com>
> ---
>  recipes-core/icedtea/icedtea7-native.inc      | 14 +++----
>  .../icedtea/openjdk-7-release-03b147.inc      | 40 +++++++++++--------
>  2 files changed, 31 insertions(+), 23 deletions(-)
> 
> diff --git a/recipes-core/icedtea/icedtea7-native.inc b/recipes-core/icedtea/icedtea7-native.inc
> index b0b9419..70738e7 100644
> --- a/recipes-core/icedtea/icedtea7-native.inc
> +++ b/recipes-core/icedtea/icedtea7-native.inc
> @@ -83,13 +83,13 @@ EXTRA_OECONF = "\
>  	--with-javac=${STAGING_BINDIR_NATIVE}/javac \
>  	--with-rhino=${STAGING_DATADIR_NATIVE}/java/rhino.jar \
>  	\
> -	--with-openjdk-src-zip=${DL_DIR}/${OPENJDK_FILE} \
> -        --with-hotspot-src-zip=${DL_DIR}/${HOTSPOT_FILE} \
> -        --with-corba-src-zip=${DL_DIR}/${CORBA_FILE} \
> -        --with-jaxp-src-zip=${DL_DIR}/${JAXP_FILE} \
> -        --with-jaxws-src-zip=${DL_DIR}/${JAXWS_FILE} \
> -        --with-jdk-src-zip=${DL_DIR}/${JDK_FILE} \
> -        --with-langtools-src-zip=${DL_DIR}/${LANGTOOLS_FILE} \
> +	--with-openjdk-src-zip=${DL_DIR}/${OPENJDK_FILE_DOWNLOAD} \

Looks like you have some whitespace issues here.

Apart from that the patch looks good, therefore I'm looking forward to a
v2. Thanks!

regards;rl

> +        --with-hotspot-src-zip=${DL_DIR}/${HOTSPOT_FILE_DOWNLOAD} \
> +        --with-corba-src-zip=${DL_DIR}/${CORBA_FILE_DOWNLOAD} \
> +        --with-jaxp-src-zip=${DL_DIR}/${JAXP_FILE_DOWNLOAD} \
> +        --with-jaxws-src-zip=${DL_DIR}/${JAXWS_FILE_DOWNLOAD} \
> +        --with-jdk-src-zip=${DL_DIR}/${JDK_FILE_DOWNLOAD} \
> +        --with-langtools-src-zip=${DL_DIR}/${LANGTOOLS_FILE_DOWNLOAD} \
>          ac_cv_path_UNZIP_BIN="${STAGING_BINDIR_NATIVE}/unzip" \
>    "
>  
> diff --git a/recipes-core/icedtea/openjdk-7-release-03b147.inc b/recipes-core/icedtea/openjdk-7-release-03b147.inc
> index 9816c0c..b504667 100644
> --- a/recipes-core/icedtea/openjdk-7-release-03b147.inc
> +++ b/recipes-core/icedtea/openjdk-7-release-03b147.inc
> @@ -13,50 +13,58 @@ ICEDTEA_PREFIX = "icedtea7-forest-2.1"
>  ICEDTEA_HG_URL = "http://icedtea.classpath.org/hg/release/${ICEDTEA_PREFIX}"
>  
>  OPENJDK_CHANGESET = "22cc03983e20"
> -OPENJDK_FILE = "${OPENJDK_CHANGESET}.tar.bz2"
> -OPENJDK_URI = "${ICEDTEA_HG_URL}/archive/${OPENJDK_FILE};name=openjdk;unpack=false"
> +OPENJDK_FILE_UPSTREAM = "${OPENJDK_CHANGESET}.tar.bz2"
> +OPENJDK_FILE_DOWNLOAD = "openjdk-${OPENJDK_CHANGESET}.tar.bz2"
> +OPENJDK_URI = "${ICEDTEA_HG_URL}/archive/${OPENJDK_FILE_UPSTREAM};name=openjdk;unpack=false;downloadfilename=${OPENJDK_FILE_DOWNLOAD}"
>  SRC_URI[openjdk.md5sum] = "f98b0f7a69f9427a5b3775dddddc89cb"
>  SRC_URI[openjdk.sha256sum] = "758227833982371e3a94def84226a947f8dbbd176517f313b8cb96840bdea41e"
>  
>  HOTSPOT_CHANGESET = "8b7c4c5f6ba9"
> -HOTSPOT_FILE = "${HOTSPOT_CHANGESET}.tar.gz"
> -HOTSPOT_URI = "${ICEDTEA_HG_URL}/hotspot/archive/${HOTSPOT_FILE};name=hotspot;unpack=false"
> +HOTSPOT_FILE_UPSTREAM = "${HOTSPOT_CHANGESET}.tar.gz"
> +HOTSPOT_FILE_DOWNLOAD = "hotspot-${HOTSPOT_CHANGESET}.tar.gz"
> +HOTSPOT_URI = "${ICEDTEA_HG_URL}/hotspot/archive/${HOTSPOT_FILE_UPSTREAM};name=hotspot;unpack=false;downloadfilename=${HOTSPOT_FILE_DOWNLOAD}"
>  SRC_URI[hotspot.md5sum] = "cb45a8f19d77acc1f4f48e75e36a27fe"
>  SRC_URI[hotspot.sha256sum] = "22866990d143e76ced94b76defa3051e5e5d9a51fd272d63daa0df272d6406a2"
>  
>  CORBA_CHANGESET = "5617f6c5cc94"
> -CORBA_FILE = "${CORBA_CHANGESET}.tar.gz"
> -CORBA_URI = "${ICEDTEA_HG_URL}/corba/archive/${CORBA_FILE};name=corba;unpack=false"
> +CORBA_FILE_UPSTREAM = "${CORBA_CHANGESET}.tar.gz"
> +CORBA_FILE_DOWNLOAD = "corba-${CORBA_CHANGESET}.tar.gz"
> +CORBA_URI = "${ICEDTEA_HG_URL}/corba/archive/${CORBA_FILE_UPSTREAM};name=corba;unpack=false;downloadfilename=${CORBA_FILE_DOWNLOAD}"
>  SRC_URI[corba.md5sum] = "920f1a788a7fdef29a5cd70892331251"
>  SRC_URI[corba.sha256sum] = "963915483530f311ff313635f79ed11ea7ce5a1c0dbee5d1acb1994132857fa3"
>  
>  JAXP_CHANGESET = "7a8825b15df6"
> -JAXP_FILE = "${JAXP_CHANGESET}.tar.gz"
> -JAXP_URI = "${ICEDTEA_HG_URL}/jaxp/archive/${JAXP_FILE};name=jaxp;unpack=false"
> +JAXP_FILE_UPSTREAM = "${JAXP_CHANGESET}.tar.gz"
> +JAXP_FILE_DOWNLOAD = "jaxp-${JAXP_CHANGESET}.tar.gz"
> +JAXP_URI = "${ICEDTEA_HG_URL}/jaxp/archive/${JAXP_FILE_UPSTREAM};name=jaxp;unpack=false;downloadfilename=${JAXP_FILE_DOWNLOAD}"
>  SRC_URI[jaxp.md5sum] = "a0982b6df79739127871e355a40ca5a6"
>  SRC_URI[jaxp.sha256sum] = "297c8dbeed5afa5395b6be06b17282d53f8a888c2909074dfc1605afc0daf1c2"
>  
>  JAXWS_CHANGESET = "7edfbfe974f2"
> -JAXWS_FILE = "${JAXWS_CHANGESET}.tar.gz"
> -JAXWS_URI = "${ICEDTEA_HG_URL}/jaxws/archive/${JAXWS_FILE};name=jaxws;unpack=false"
> +JAXWS_FILE_UPSTREAM = "${JAXWS_CHANGESET}.tar.gz"
> +JAXWS_FILE_DOWNLOAD = "jaxws-${JAXWS_CHANGESET}.tar.gz"
> +JAXWS_URI = "${ICEDTEA_HG_URL}/jaxws/archive/${JAXWS_FILE_UPSTREAM};name=jaxws;unpack=false;downloadfilename=${JAXWS_FILE_DOWNLOAD}"
>  SRC_URI[jaxws.md5sum] = "2deecec809ea8e38835678889c2bc3ac"
>  SRC_URI[jaxws.sha256sum] = "823ca6b8fd780bf1b1565a90cfc8ec6f0ee422a5d25ffb20b1ce0272dc4955aa"
>  
>  JDK_CHANGESET = "d5ddeffc4651"
> -JDK_FILE = "${JDK_CHANGESET}.tar.gz"
> -JDK_URI = "${ICEDTEA_HG_URL}/jdk/archive/${JDK_FILE};name=jdk;unpack=false"
> +JDK_FILE_UPSTREAM = "${JDK_CHANGESET}.tar.gz"
> +JDK_FILE_DOWNLOAD = "jdk-${JDK_CHANGESET}.tar.gz"
> +JDK_URI = "${ICEDTEA_HG_URL}/jdk/archive/${JDK_FILE_UPSTREAM};name=jdk;unpack=false;downloadfilename=${JDK_FILE_DOWNLOAD}"
>  SRC_URI[jdk.md5sum] = "27c28d6be213df818490e69f12efe475"
>  SRC_URI[jdk.sha256sum] = "e83e6234842376147a4c32451f23d629ba107e908ebdb0f367657f9cc6c1be98"
>  
>  LANGTOOLS_CHANGESET = "b534c4c6cd9b"
> -LANGTOOLS_FILE = "${LANGTOOLS_CHANGESET}.tar.gz"
> -LANGTOOLS_URI = "${ICEDTEA_HG_URL}/langtools/archive/${LANGTOOLS_FILE};name=langtools;unpack=false"
> +LANGTOOLS_FILE_UPSTREAM = "${LANGTOOLS_CHANGESET}.tar.gz"
> +LANGTOOLS_FILE_DOWNLOAD = "langtools-${LANGTOOLS_CHANGESET}.tar.gz"
> +LANGTOOLS_URI = "${ICEDTEA_HG_URL}/langtools/archive/${LANGTOOLS_FILE_UPSTREAM};name=langtools;unpack=false;downloadfilename=${LANGTOOLS_FILE_DOWNLOAD}"
>  SRC_URI[langtools.md5sum] = "16b72f3fea8ac4e5b50de693e87f9c53"
>  SRC_URI[langtools.sha256sum] = "c77dd42d83f06d18fa8aa65c93d137608a2e02cf24b68fc10d7b1c0bcc12a93c"
>  
>  CACAO_VERSION = "a567bcb7f589"
> -CACAO_FILE = "${CACAO_VERSION}.tar.gz"
> -CACAO_URI = "http://icedtea.wildebeest.org/download/drops/cacao/${CACAO_FILE};name=cacao;unpack=false"
> +CACAO_FILE_UPSTREAM = "${CACAO_VERSION}.tar.gz"
> +CACAO_FILE_DOWNLOAD = "cacao-${CACAO_VERSION}.tar.gz"
> +CACAO_URI = "http://icedtea.wildebeest.org/download/drops/cacao/${CACAO_FILE_UPSTREAM};name=cacao;unpack=false;downloadfilename=${CACAO_FILE_DOWNLOAD}"
>  SRC_URI[cacao.md5sum] = "ec1def7f0d3d25e9e7da47c480f26a73"
>  SRC_URI[cacao.sha256sum] = "d49f79debc131a5694cae6ab3ba2864e7f3249ee8d9dc09aae8afdd4dc6b09f9"
>  
> -- 
> 2.25.1
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [oe] [meta-java][PATCH] icedtea7-native: set unique download file names
  2020-06-23 18:08 ` [oe] " Richard Leitner
@ 2020-06-24  5:23   ` Vyacheslav Yurkov
  0 siblings, 0 replies; 3+ messages in thread
From: Vyacheslav Yurkov @ 2020-06-24  5:23 UTC (permalink / raw
  To: Richard Leitner; +Cc: openembedded-devel, Vyacheslav Yurkov

Hi Richards,
Thanks for the quick review.

The problem is that the whole file in general has tabs/spaces mixed, 
even just withing EXTRA_OECONF declaration. It's also not clear to me 
why two empty lines needed in it. In my patch I simply left indentation 
as it was in original file. Do you think it would be better just to 
change the whole EXTRA_OECONF to spaces?

Thanks,
Vyacheslav

On 23.06.2020 20:08, Richard Leitner wrote:
> Hi Vyacheslav,
> thanks for your patch. Please find my comments inline.
>
> On Tue, Jun 23, 2020 at 02:23:41PM +0200, Vyacheslav Yurkov wrote:
>> From: Vyacheslav Yurkov <Vyacheslav.Yurkov@bruker.com>
> It would be great if you could describe the change in a few more words
> here. Thanks!
>
>> Signed-off-by: Vyacheslav Yurkov <Vyacheslav.Yurkov@bruker.com>
>> ---
>>   recipes-core/icedtea/icedtea7-native.inc      | 14 +++----
>>   .../icedtea/openjdk-7-release-03b147.inc      | 40 +++++++++++--------
>>   2 files changed, 31 insertions(+), 23 deletions(-)
>>
>> diff --git a/recipes-core/icedtea/icedtea7-native.inc b/recipes-core/icedtea/icedtea7-native.inc
>> index b0b9419..70738e7 100644
>> --- a/recipes-core/icedtea/icedtea7-native.inc
>> +++ b/recipes-core/icedtea/icedtea7-native.inc
>> @@ -83,13 +83,13 @@ EXTRA_OECONF = "\
>>   	--with-javac=${STAGING_BINDIR_NATIVE}/javac \
>>   	--with-rhino=${STAGING_DATADIR_NATIVE}/java/rhino.jar \
>>   	\
>> -	--with-openjdk-src-zip=${DL_DIR}/${OPENJDK_FILE} \
>> -        --with-hotspot-src-zip=${DL_DIR}/${HOTSPOT_FILE} \
>> -        --with-corba-src-zip=${DL_DIR}/${CORBA_FILE} \
>> -        --with-jaxp-src-zip=${DL_DIR}/${JAXP_FILE} \
>> -        --with-jaxws-src-zip=${DL_DIR}/${JAXWS_FILE} \
>> -        --with-jdk-src-zip=${DL_DIR}/${JDK_FILE} \
>> -        --with-langtools-src-zip=${DL_DIR}/${LANGTOOLS_FILE} \
>> +	--with-openjdk-src-zip=${DL_DIR}/${OPENJDK_FILE_DOWNLOAD} \
> Looks like you have some whitespace issues here.
>
> Apart from that the patch looks good, therefore I'm looking forward to a
> v2. Thanks!
>
> regards;rl
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-06-24  5:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-06-23 12:23 [meta-java][PATCH] icedtea7-native: set unique download file names Vyacheslav Yurkov
2020-06-23 18:08 ` [oe] " Richard Leitner
2020-06-24  5:23   ` Vyacheslav Yurkov

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.