unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Efraim Flashner <efraim@flashner.co.il>
To: Julien Lepiller <julien@lepiller.eu>
Cc: 49521@debbugs.gnu.org
Subject: [bug#49521] [PATCH] gnu: java-xz: Update to 1.9.
Date: Mon, 12 Jul 2021 15:38:14 +0300	[thread overview]
Message-ID: <YOw3tqZeN8dQxv4x@3900XT> (raw)
In-Reply-To: <659CEC20-6F93-4922-988D-439705029C17@lepiller.eu>

[-- 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 --]

  reply	other threads:[~2021-07-12 12:53 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
2021-07-12 12:42     ` Julien Lepiller
2021-07-12 13:57       ` bug#49521: " Efraim Flashner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YOw3tqZeN8dQxv4x@3900XT \
    --to=efraim@flashner.co.il \
    --cc=49521@debbugs.gnu.org \
    --cc=julien@lepiller.eu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).