guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#31333] [PATCH 22/22] gnu: Add groovy.


From: Danny Milosavljevic
Subject: [bug#31333] [PATCH 22/22] gnu: Add groovy.
Date: Mon, 7 May 2018 21:23:00 +0200

> +                 ;; Groovy uses class loading. It's not enough to put the 
> class
> +                 ;; in the loader's classpath, as it causes breakages:
> +                 ;; the compiler would give this error:
> +                 ;; "Prohibited package name: java.lang"

It might make sense to ask upstream about that.

> +                 ;; antlr.jar is present twice in antlr2.  Symlink doesn't 
> like
> +                 ;; it, so we symlink it here.

Why is it present twice?  Is it useful?

So this package uses an union in order to avoid classloader problems.  Fine, 
but I wonder whether the problem is now merely hidden.

Anyway, LGTM for now (for this version) !

Attachment: pgpoDhkHQLU5R.pgp
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]