unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#49521] [PATCH] gnu: java-xz: Update to 1.9.
@ 2021-07-11 11:48 Efraim Flashner
  2021-07-11 16:08 ` Julien Lepiller
  0 siblings, 1 reply; 5+ messages in thread
From: Efraim Flashner @ 2021-07-11 11:48 UTC (permalink / raw)
  To: 49521; +Cc: Efraim Flashner

* gnu/packages/java-compression.scm (java-xz): Update to 1.9.
[arguments]: Build with openjdk9.
---
 gnu/packages/java-compression.scm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/java-compression.scm b/gnu/packages/java-compression.scm
index 19b11954c1..033b55c988 100644
--- a/gnu/packages/java-compression.scm
+++ b/gnu/packages/java-compression.scm
@@ -4,6 +4,7 @@
 ;;; Copyright © 2018 Marius Bakke <mbakke@fastmail.com>
 ;;; Copyright © 2018 Mark H Weaver <mhw@netris.org>
 ;;; Copyright © 2018, 2020 Tobias Geerinckx-Rice <me@tobias.gr>
+;;; Copyright © 2021 Efraim Flashner <efraim@flashner.co.il>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -289,16 +290,17 @@ It can be used as a replacement for the Apache @code{CBZip2InputStream} /
 (define-public java-xz
   (package
     (name "java-xz")
-    (version "1.6")
+    (version "1.9")
     (source (origin
               (method url-fetch/zipbomb)
               (uri (string-append "https://tukaani.org/xz/xz-java-" version ".zip"))
               (sha256
                (base32
-                "1z3p1ri1gvl07inxn0agx44ck8n7wrzfmvkz8nbq3njn8r9wba8x"))))
+                "007d9f83277qn70swz9inqhyf0qxq6ygajpq5rqg0xgsyh1sdndi"))))
     (build-system ant-build-system)
     (arguments
      `(#:tests? #f; no tests
+       #:jdk ,openjdk9
        #:phases
        (modify-phases %standard-phases
          (add-before 'install 'generate-pom

base-commit: 7760d28920a920791645c4485f1345af45ee7787
-- 
2.32.0





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

* [bug#49521] [PATCH] gnu: java-xz: Update to 1.9.
  2021-07-11 11:48 [bug#49521] [PATCH] gnu: java-xz: Update to 1.9 Efraim Flashner
@ 2021-07-11 16:08 ` Julien Lepiller
  2021-07-12 12:38   ` Efraim Flashner
  0 siblings, 1 reply; 5+ messages in thread
From: Julien Lepiller @ 2021-07-11 16:08 UTC (permalink / raw)
  To: Efraim Flashner, 49521

Le 11 juillet 2021 07:48:22 GMT-04:00, Efraim Flashner <efraim@flashner.co.il> a écrit :
>* gnu/packages/java-compression.scm (java-xz): Update to 1.9.
>[arguments]: Build with openjdk9.
>---
> gnu/packages/java-compression.scm | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
>diff --git a/gnu/packages/java-compression.scm
>b/gnu/packages/java-compression.scm
>index 19b11954c1..033b55c988 100644
>--- a/gnu/packages/java-compression.scm
>+++ b/gnu/packages/java-compression.scm
>@@ -4,6 +4,7 @@
> ;;; Copyright © 2018 Marius Bakke <mbakke@fastmail.com>
> ;;; Copyright © 2018 Mark H Weaver <mhw@netris.org>
> ;;; Copyright © 2018, 2020 Tobias Geerinckx-Rice <me@tobias.gr>
>+;;; Copyright © 2021 Efraim Flashner <efraim@flashner.co.il>
> ;;;
> ;;; This file is part of GNU Guix.
> ;;;
>@@ -289,16 +290,17 @@ It can be used as a replacement for the Apache
>@code{CBZip2InputStream} /
> (define-public java-xz
>   (package
>     (name "java-xz")
>-    (version "1.6")
>+    (version "1.9")
>     (source (origin
>               (method url-fetch/zipbomb)
> (uri (string-append "https://tukaani.org/xz/xz-java-" version ".zip"))
>               (sha256
>                (base32
>-               
>"1z3p1ri1gvl07inxn0agx44ck8n7wrzfmvkz8nbq3njn8r9wba8x"))))
>+               
>"007d9f83277qn70swz9inqhyf0qxq6ygajpq5rqg0xgsyh1sdndi"))))
>     (build-system ant-build-system)
>     (arguments
>      `(#:tests? #f; no tests
>+       #:jdk ,openjdk9
>        #:phases
>        (modify-phases %standard-phases
>          (add-before 'install 'generate-pom
>
>base-commit: 7760d28920a920791645c4485f1345af45ee7787

Looks good, but building dependents will likely cause issues if they still use openjdk8. At the very least, we'll see a warning.




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

* [bug#49521] [PATCH] gnu: java-xz: Update to 1.9.
  2021-07-11 16:08 ` Julien Lepiller
@ 2021-07-12 12:38   ` Efraim Flashner
  2021-07-12 12:42     ` Julien Lepiller
  0 siblings, 1 reply; 5+ messages in thread
From: Efraim Flashner @ 2021-07-12 12:38 UTC (permalink / raw)
  To: Julien Lepiller; +Cc: 49521

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

On Sun, Jul 11, 2021 at 12:08:35PM -0400, Julien Lepiller wrote:
> Le 11 juillet 2021 07:48:22 GMT-04:00, Efraim Flashner <efraim@flashner.co.il> a écrit :
> >* gnu/packages/java-compression.scm (java-xz): Update to 1.9.
> >[arguments]: Build with openjdk9.
> >---
> > gnu/packages/java-compression.scm | 6 ++++--
> > 1 file changed, 4 insertions(+), 2 deletions(-)
> >
> >diff --git a/gnu/packages/java-compression.scm
> >b/gnu/packages/java-compression.scm
> >index 19b11954c1..033b55c988 100644
> >--- a/gnu/packages/java-compression.scm
> >+++ b/gnu/packages/java-compression.scm
> >@@ -4,6 +4,7 @@
> > ;;; Copyright © 2018 Marius Bakke <mbakke@fastmail.com>
> > ;;; Copyright © 2018 Mark H Weaver <mhw@netris.org>
> > ;;; Copyright © 2018, 2020 Tobias Geerinckx-Rice <me@tobias.gr>
> >+;;; Copyright © 2021 Efraim Flashner <efraim@flashner.co.il>
> > ;;;
> > ;;; This file is part of GNU Guix.
> > ;;;
> >@@ -289,16 +290,17 @@ It can be used as a replacement for the Apache
> >@code{CBZip2InputStream} /
> > (define-public java-xz
> >   (package
> >     (name "java-xz")
> >-    (version "1.6")
> >+    (version "1.9")
> >     (source (origin
> >               (method url-fetch/zipbomb)
> > (uri (string-append "https://tukaani.org/xz/xz-java-" version ".zip"))
> >               (sha256
> >                (base32
> >-               
> >"1z3p1ri1gvl07inxn0agx44ck8n7wrzfmvkz8nbq3njn8r9wba8x"))))
> >+               
> >"007d9f83277qn70swz9inqhyf0qxq6ygajpq5rqg0xgsyh1sdndi"))))
> >     (build-system ant-build-system)
> >     (arguments
> >      `(#:tests? #f; no tests
> >+       #:jdk ,openjdk9
> >        #:phases
> >        (modify-phases %standard-phases
> >          (add-before 'install 'generate-pom
> >
> >base-commit: 7760d28920a920791645c4485f1345af45ee7787
> 
> Looks good, but building dependents will likely cause issues if they still use openjdk8. At the very least, we'll see a warning.

Is that a problem? Starting with 1.9 they added some code that needs
java 9 to compile. I test built the 4 packages which directly depend on
java-xz and they built without problems.

¹https://git.tukaani.org/?p=xz-java.git;a=blob;f=NEWS;h=d18a53b23efdae4b7d2911ef7898189462fac689;hb=HEAD

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* [bug#49521] [PATCH] gnu: java-xz: Update to 1.9.
  2021-07-12 12:38   ` Efraim Flashner
@ 2021-07-12 12:42     ` Julien Lepiller
  2021-07-12 13:57       ` bug#49521: " Efraim Flashner
  0 siblings, 1 reply; 5+ messages in thread
From: Julien Lepiller @ 2021-07-12 12:42 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: 49521

Le 12 juillet 2021 08:38:14 GMT-04:00, Efraim Flashner <efraim@flashner.co.il> a écrit :
>On Sun, Jul 11, 2021 at 12:08:35PM -0400, Julien Lepiller wrote:
>> Le 11 juillet 2021 07:48:22 GMT-04:00, Efraim Flashner
><efraim@flashner.co.il> a écrit :
>> >* gnu/packages/java-compression.scm (java-xz): Update to 1.9.
>> >[arguments]: Build with openjdk9.
>> >---
>> > gnu/packages/java-compression.scm | 6 ++++--
>> > 1 file changed, 4 insertions(+), 2 deletions(-)
>> >
>> >diff --git a/gnu/packages/java-compression.scm
>> >b/gnu/packages/java-compression.scm
>> >index 19b11954c1..033b55c988 100644
>> >--- a/gnu/packages/java-compression.scm
>> >+++ b/gnu/packages/java-compression.scm
>> >@@ -4,6 +4,7 @@
>> > ;;; Copyright © 2018 Marius Bakke <mbakke@fastmail.com>
>> > ;;; Copyright © 2018 Mark H Weaver <mhw@netris.org>
>> > ;;; Copyright © 2018, 2020 Tobias Geerinckx-Rice <me@tobias.gr>
>> >+;;; Copyright © 2021 Efraim Flashner <efraim@flashner.co.il>
>> > ;;;
>> > ;;; This file is part of GNU Guix.
>> > ;;;
>> >@@ -289,16 +290,17 @@ It can be used as a replacement for the Apache
>> >@code{CBZip2InputStream} /
>> > (define-public java-xz
>> >   (package
>> >     (name "java-xz")
>> >-    (version "1.6")
>> >+    (version "1.9")
>> >     (source (origin
>> >               (method url-fetch/zipbomb)
>> > (uri (string-append "https://tukaani.org/xz/xz-java-" version
>".zip"))
>> >               (sha256
>> >                (base32
>> >-               
>> >"1z3p1ri1gvl07inxn0agx44ck8n7wrzfmvkz8nbq3njn8r9wba8x"))))
>> >+               
>> >"007d9f83277qn70swz9inqhyf0qxq6ygajpq5rqg0xgsyh1sdndi"))))
>> >     (build-system ant-build-system)
>> >     (arguments
>> >      `(#:tests? #f; no tests
>> >+       #:jdk ,openjdk9
>> >        #:phases
>> >        (modify-phases %standard-phases
>> >          (add-before 'install 'generate-pom
>> >
>> >base-commit: 7760d28920a920791645c4485f1345af45ee7787
>> 
>> Looks good, but building dependents will likely cause issues if they
>still use openjdk8. At the very least, we'll see a warning.
>
>Is that a problem? Starting with 1.9 they added some code that needs
>java 9 to compile. I test built the 4 packages which directly depend on
>java-xz and they built without problems.
>
>¹https://git.tukaani.org/?p=xz-java.git;a=blob;f=NEWS;h=d18a53b23efdae4b7d2911ef7898189462fac689;hb=HEAD

If dependents build fine, it should work.




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

* bug#49521: [PATCH] gnu: java-xz: Update to 1.9.
  2021-07-12 12:42     ` Julien Lepiller
@ 2021-07-12 13:57       ` Efraim Flashner
  0 siblings, 0 replies; 5+ messages in thread
From: Efraim Flashner @ 2021-07-12 13:57 UTC (permalink / raw)
  To: Julien Lepiller; +Cc: 49521-done

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

On Mon, Jul 12, 2021 at 08:42:32AM -0400, Julien Lepiller wrote:
> If dependents build fine, it should work.

Thanks. Patch pushed.

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

end of thread, other threads:[~2021-07-12 14:00 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-07-11 11:48 [bug#49521] [PATCH] gnu: java-xz: Update to 1.9 Efraim Flashner
2021-07-11 16:08 ` Julien Lepiller
2021-07-12 12:38   ` Efraim Flashner
2021-07-12 12:42     ` Julien Lepiller
2021-07-12 13:57       ` bug#49521: " Efraim Flashner

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).