GNU bug report logs - #48977
[PATCH 0/4] Update openjdk 13 and later

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Pierre Langlois <pierre.langlois <at> gmx.com>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#48977: closed ([PATCH 0/4] Update openjdk 13 and later)
Date: Sun, 13 Jun 2021 17:04:01 +0000

[Message part 1 (text/plain, inline)]

Your message dated Sun, 13 Jun 2021 18:03:39 +0100
with message-id <87sg1ln1ck.fsf <at> gmx.com>
and subject line Re: [bug#48977] [PATCH 4/4] gnu: Add openjdk16.
has caused the debbugs.gnu.org bug report #48977,
regarding [PATCH 0/4] Update openjdk 13 and later
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
48977: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=48977
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Pierre Langlois <pierre.langlois <at> gmx.com>
To: Guix-patches <guix-patches <at> gnu.org>
Subject: [PATCH 0/4] Update openjdk 13 and later
Date: Sat, 12 Jun 2021 20:45:53 +0100

[Message part 3 (text/plain, inline)]

Hi Guix!

Here are a few patches to include the OpenJDK 15 and 16 releases, and
update 13 and 14. The codebase seems to have now moved to git, using
github as the default place to get the code.

Regarding release versions, it seems the scheme was changed for versions
10 and later to $FEATURE.$INTERIM.$UPDATE, where $INTERIM is always
zero, quoting from [0]:

--8<---------------cut here---------------start------------->8---
Under the six-month release model the elements of version numbers vary
as follows:

    $FEATURE is incremented every six months: The March 2018 release is
    JDK 10, the September 2018 release is JDK 11, and so forth.

    $INTERIM is always zero, since the six-month model does not include
    interim releases. We reserve it here for flexibility, so that a
    future revision to the release model could include such releases and
    say that JDK $N.1 and JDK $N.2 are compatible upgrades of JDK $N. As
    examples, the JDK 1.4.1 and 1.4.2 releases were, in essence, interim
    releases, and would have been numbered 4.1 and 4.2 under this
    scheme.

    $UPDATE is incremented one month after $FEATURE is incremented, and
    every three months thereafter: The April 2018 release is JDK 10.0.1,
    the July release is JDK 10.0.2, and so forth.
--8<---------------cut here---------------end--------------->8---

It seems the easiest thing to do is to refer to "GA" releases, for
"general availability" with git tags that look like "jdk-<version>-ga".

I've done it for OpenJDK 13 and later, however if we change 10, 11 and
12, it looks like a downgrade, is that a problem? 

Thanks,
Pierre

[0]: https://openjdk.java.net/jeps/322

[signature.asc (application/pgp-signature, inline)]

[Message part 5 (message/rfc822, inline)]

From: Pierre Langlois <pierre.langlois <at> gmx.com>
To: Julien Lepiller <julien <at> lepiller.eu>
Cc: 48977-done <at> debbugs.gnu.org, Pierre Langlois <pierre.langlois <at> gmx.com>
Subject: Re: [bug#48977] [PATCH 4/4] gnu: Add openjdk16.
Date: Sun, 13 Jun 2021 18:03:39 +0100

[Message part 6 (text/plain, inline)]

Hi Julien,

Julien Lepiller writes:

> Le Sat, 12 Jun 2021 20:54:49 +0100,
> Pierre Langlois <pierre.langlois <at> gmx.com> a écrit :
>
>> +    (arguments
>> +     (substitute-keyword-arguments (package-arguments openjdk15)
>> +       ((#:phases phases)
>> +        `(modify-phases ,phases
>> +           (add-after 'unpack 'make-templates-writable
>> +             (lambda _
>> +               ;; The build system copies a few .template files from
>> the
>> +               ;; source directory into the build directory and then
>> modifies
>> +               ;; them in-place. So these files have to be writable.
>> +               (for-each
>> +                (lambda (file)
>> +                  (invoke "chmod" "u+w" file))
>> +                (find-files
>> "src/java.base/share/classes/jdk/internal/misc/"
>> +                            "\\.template"))
>
> Should be "\\.template$", otherwise the whole series LGTM!

Thanks for the review! Fixed and pushed with
afd4924d6a64c01a1d5138954b9f22c9bd4f4690.

Pierre

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.