all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Gábor Boskovits" <boskovits@gmail.com>
To: 30050@debbugs.gnu.org
Cc: "Gábor Boskovits" <boskovits@gmail.com>
Subject: [bug#30050] [PATCH 0/2 core-updates] Fix antlr3 bootstrap toolchain to work on java8
Date: Tue,  9 Jan 2018 21:02:30 +0100	[thread overview]
Message-ID: <20180109200230.5043-1-boskovits@gmail.com> (raw)

This patch series includes the fixes needed to bootstrap antlr3 for java8.
The toolchain includes two older versions of antlr3, which does not compile
on java8, because in java7 it was legal to pass null to removeAll, but this
is no longer so. Original behaviour was not to modify the object when null
is passed. java8 throws null pointer exception. Fix is simply to wrap the
call in a null check for the argument. This is backwards compatible.

Gábor Boskovits (2):
  gnu: antlr3-3.1: Fix java8 issue.
  gnu: antlr3-3.3: Fix java8 issue.

 gnu/local.mk                                       |  4 ++-
 gnu/packages/java.scm                              |  9 ++++--
 .../patches/antlr3-3_1-fix-java8-compilation.patch | 35 ++++++++++++++++++++++
 .../patches/antlr3-3_3-fix-java8-compilation.patch | 35 ++++++++++++++++++++++
 4 files changed, 80 insertions(+), 3 deletions(-)
 create mode 100644 gnu/packages/patches/antlr3-3_1-fix-java8-compilation.patch
 create mode 100644 gnu/packages/patches/antlr3-3_3-fix-java8-compilation.patch

-- 
2.15.1

             reply	other threads:[~2018-01-09 20:04 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-09 20:02 Gábor Boskovits [this message]
2018-01-09 20:10 ` [bug#30050] [PATCH 1/2] gnu: antlr3-3.1: Fix java8 issue Gábor Boskovits
2018-01-09 20:12 ` [bug#30050] [PATCH 2/2] gnu: antlr3-3.3: " Gábor Boskovits
2018-03-04  9:10 ` [bug#30050] Please push this to master Gábor Boskovits
2018-03-04 16:52 ` [bug#30050] [PATCH 0/2 core-updates] Fix antlr3 bootstrap toolchain to work on java8 Danny Milosavljevic

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

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

  git send-email \
    --in-reply-to=20180109200230.5043-1-boskovits@gmail.com \
    --to=boskovits@gmail.com \
    --cc=30050@debbugs.gnu.org \
    /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 external index

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

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.