All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-java][PATCH] jsch: switch to inclusive language
@ 2022-11-25 12:35 Ulrich Ölmann
  2022-11-25 13:57 ` [oe] " Jacob Kroon
  0 siblings, 1 reply; 3+ messages in thread
From: Ulrich Ölmann @ 2022-11-25 12:35 UTC (permalink / raw)
  To: openembedded-devel
  Cc: Henning Heinold, Otavio Salvador, Richard Leitner,
	Ulrich Ölmann

Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
---
 recipes-core/jcraft/jsch_0.1.40.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/recipes-core/jcraft/jsch_0.1.40.bb b/recipes-core/jcraft/jsch_0.1.40.bb
index 07d5b9e3108e..259712661f4d 100644
--- a/recipes-core/jcraft/jsch_0.1.40.bb
+++ b/recipes-core/jcraft/jsch_0.1.40.bb
@@ -25,8 +25,8 @@ do_compile() {
 SRC_URI[md5sum] = "b59cec19a487e95aed68378976b4b566"
 SRC_URI[sha256sum] = "ca9d2ae08fd7a8983fb00d04f0f0c216a985218a5eb364ff9bee73870f28e097"
 
-# Whitelist the CVE because it only affects Windows platforms
-CVE_CHECK_WHITELIST += "CVE-2016-5725"
+# Ignore the CVE because it only affects Windows platforms
+CVE_CHECK_IGNORE += "CVE-2016-5725"
 
 BBCLASSEXTEND = "native"
 
-- 
2.30.2



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

* Re: [oe] [meta-java][PATCH] jsch: switch to inclusive language
  2022-11-25 12:35 [meta-java][PATCH] jsch: switch to inclusive language Ulrich Ölmann
@ 2022-11-25 13:57 ` Jacob Kroon
  2022-11-28 13:11   ` Ulrich Ölmann
  0 siblings, 1 reply; 3+ messages in thread
From: Jacob Kroon @ 2022-11-25 13:57 UTC (permalink / raw)
  To: Ulrich Ölmann, openembedded-devel
  Cc: Henning Heinold, Otavio Salvador, Richard Leitner

On 11/25/22 13:35, Ulrich Ölmann wrote:
> Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
> ---
>   recipes-core/jcraft/jsch_0.1.40.bb | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/recipes-core/jcraft/jsch_0.1.40.bb b/recipes-core/jcraft/jsch_0.1.40.bb
> index 07d5b9e3108e..259712661f4d 100644
> --- a/recipes-core/jcraft/jsch_0.1.40.bb
> +++ b/recipes-core/jcraft/jsch_0.1.40.bb
> @@ -25,8 +25,8 @@ do_compile() {
>   SRC_URI[md5sum] = "b59cec19a487e95aed68378976b4b566"
>   SRC_URI[sha256sum] = "ca9d2ae08fd7a8983fb00d04f0f0c216a985218a5eb364ff9bee73870f28e097"
>   
> -# Whitelist the CVE because it only affects Windows platforms
> -CVE_CHECK_WHITELIST += "CVE-2016-5725"
> +# Ignore the CVE because it only affects Windows platforms
> +CVE_CHECK_IGNORE += "CVE-2016-5725"
>   
>   BBCLASSEXTEND = "native"
>   

Also your commit:

3161106f0879a6be0a016df3a8bb8fa800bd6044
openjdk-8: refresh patches

seems to have *introduced* fuzz warnings, rather than fix any on master 
branch ...

Jacob


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

* Re: [oe] [meta-java][PATCH] jsch: switch to inclusive language
  2022-11-25 13:57 ` [oe] " Jacob Kroon
@ 2022-11-28 13:11   ` Ulrich Ölmann
  0 siblings, 0 replies; 3+ messages in thread
From: Ulrich Ölmann @ 2022-11-28 13:11 UTC (permalink / raw)
  To: Jacob Kroon
  Cc: Henning Heinold, Otavio Salvador, Richard Leitner,
	openembedded-devel

Hi Jacob,

On Fri, Nov 25 2022 at 14:57 +0100, "Jacob Kroon" <jacob.kroon@gmail.com> wrote:
> On 11/25/22 13:35, Ulrich Ölmann wrote:
>> Signed-off-by: Ulrich Ölmann <u.oelmann@pengutronix.de>
>> ---
>>   recipes-core/jcraft/jsch_0.1.40.bb | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>> diff --git a/recipes-core/jcraft/jsch_0.1.40.bb
>> b/recipes-core/jcraft/jsch_0.1.40.bb
>> index 07d5b9e3108e..259712661f4d 100644
>> --- a/recipes-core/jcraft/jsch_0.1.40.bb
>> +++ b/recipes-core/jcraft/jsch_0.1.40.bb
>> @@ -25,8 +25,8 @@ do_compile() {
>>   SRC_URI[md5sum] = "b59cec19a487e95aed68378976b4b566"
>>   SRC_URI[sha256sum] = "ca9d2ae08fd7a8983fb00d04f0f0c216a985218a5eb364ff9bee73870f28e097"
>>   -# Whitelist the CVE because it only affects Windows platforms
>> -CVE_CHECK_WHITELIST += "CVE-2016-5725"
>> +# Ignore the CVE because it only affects Windows platforms
>> +CVE_CHECK_IGNORE += "CVE-2016-5725"
>>     BBCLASSEXTEND = "native"
>>
>
> Also your commit:
>
> 3161106f0879a6be0a016df3a8bb8fa800bd6044
> openjdk-8: refresh patches
>
> seems to have *introduced* fuzz warnings, rather than fix any on master branch
> ...

after reading your hint, at first sight I felt ashamed seeing that my
refreshed patches introduce fuzz warnings instead of fixing them as was
intended by me. Looking more closely I then realized that before my
commit 3161106f0879 ("openjdk-8: refresh patches") it was openjdk-8
where the patches did not apply cleanly (but openjdk-8-native where
everything was okay) while after that commit it is exactly the other way
round. Unfortunately I did not realize that when I stumbled across it
originally, sigh. At the bottom line sadly my commit neither improved
the overall situation nor did it worsen it. So: sorry for the noise!

Best regards
Ulrich


> Jacob
-- 
Pengutronix e.K.                           | Ulrich Ölmann               |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |


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

end of thread, other threads:[~2022-11-28 13:13 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-11-25 12:35 [meta-java][PATCH] jsch: switch to inclusive language Ulrich Ölmann
2022-11-25 13:57 ` [oe] " Jacob Kroon
2022-11-28 13:11   ` Ulrich Ölmann

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.