unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#29359] More java packages
@ 2017-11-19 17:55 Julien Lepiller
  2017-11-24 15:31 ` [bug#29359] [PATCH 08/31] gnu: Add java-jansi Tobias Geerinckx-Rice
  2018-02-10 18:06 ` bug#29359: More java packages Julien Lepiller
  0 siblings, 2 replies; 6+ messages in thread
From: Julien Lepiller @ 2017-11-19 17:55 UTC (permalink / raw)
  To: 29359

Hi,

this patch series adds 30 more java packages and fixes one that now
fails. With this, we have about 60 packages left before maven.

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

* [bug#29359] [PATCH 08/31] gnu: Add java-jansi.
  2017-11-19 17:57 ` [bug#29359] [PATCH 01/31] gnu: Add java-gson Julien Lepiller
@ 2017-11-19 17:57   ` Julien Lepiller
  2017-11-22  9:33     ` julien lepiller
  0 siblings, 1 reply; 6+ messages in thread
From: Julien Lepiller @ 2017-11-19 17:57 UTC (permalink / raw)
  To: 29359

* gnu/packages/java.scm (java-jansi): New variable.
---
 gnu/packages/java.scm | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/gnu/packages/java.scm b/gnu/packages/java.scm
index 3811dcdc8..5d45a9949 100644
--- a/gnu/packages/java.scm
+++ b/gnu/packages/java.scm
@@ -7714,3 +7714,30 @@ that is part of the SWT Tools project.")
     (synopsis "Native library for jansi")
     (description "Java-jansi-native contains the native library for jansi.")
     (license license:asl2.0)))
+
+(define-public java-jansi
+  (package
+    (name "java-jansi")
+    (version "1.16")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "https://github.com/fusesource/jansi/archive/"
+                                  "jansi-project-" version ".tar.gz"))
+              (sha256
+               (base32
+                "11kh3144i3fzp21dpy8zg52mjmsr214k7km9p8ly0rqk2px0qq2z"))))
+    (build-system ant-build-system)
+    (arguments
+     `(#:jar-name "jansi.jar"
+       #:source-dir "jansi/src/main/java"
+       #:test-dir "jansi/src/test"))
+    (inputs
+     `(("java-jansi-native" ,java-jansi-native)))
+    (native-inputs
+     `(("java-junit" ,java-junit)
+       ("java-hamcrest-core" ,java-hamcrest-core)))
+    (home-page "https://fusesource.github.io/jansi/")
+    (synopsis "Portable ANSI escape sequences")
+    (description "Jansi is a Java library that allows you to use ANSI escape
+sequences to format your console output which works on every platform.")
+    (license license:asl2.0)))
-- 
2.15.0

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

* [bug#29359] [PATCH 08/31] gnu: Add java-jansi.
  2017-11-19 17:57   ` [bug#29359] [PATCH 08/31] gnu: Add java-jansi Julien Lepiller
@ 2017-11-22  9:33     ` julien lepiller
  2017-11-24 14:41       ` Leo Famulari
  0 siblings, 1 reply; 6+ messages in thread
From: julien lepiller @ 2017-11-22  9:33 UTC (permalink / raw)
  To: 29359

Le 2017-11-19 18:57, Julien Lepiller a écrit :
> * gnu/packages/java.scm (java-jansi): New variable.
> ---
>  gnu/packages/java.scm | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
> 
> diff --git a/gnu/packages/java.scm b/gnu/packages/java.scm
> index 3811dcdc8..5d45a9949 100644
> --- a/gnu/packages/java.scm
> +++ b/gnu/packages/java.scm
> @@ -7714,3 +7714,30 @@ that is part of the SWT Tools project.")
>      (synopsis "Native library for jansi")
>      (description "Java-jansi-native contains the native library for 
> jansi.")
>      (license license:asl2.0)))
> +
> +(define-public java-jansi
> +  (package
> +    (name "java-jansi")
> +    (version "1.16")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> "https://github.com/fusesource/jansi/archive/"
> +                                  "jansi-project-" version ".tar.gz"))
> +              (sha256
> +               (base32
> +                
> "11kh3144i3fzp21dpy8zg52mjmsr214k7km9p8ly0rqk2px0qq2z"))))
> +    (build-system ant-build-system)
> +    (arguments
> +     `(#:jar-name "jansi.jar"
> +       #:source-dir "jansi/src/main/java"
> +       #:test-dir "jansi/src/test"))
> +    (inputs
> +     `(("java-jansi-native" ,java-jansi-native)))
> +    (native-inputs
> +     `(("java-junit" ,java-junit)
> +       ("java-hamcrest-core" ,java-hamcrest-core)))
> +    (home-page "https://fusesource.github.io/jansi/")
> +    (synopsis "Portable ANSI escape sequences")
> +    (description "Jansi is a Java library that allows you to use ANSI 
> escape
> +sequences to format your console output which works on every 
> platform.")
> +    (license license:asl2.0)))

There's a small issue with this package: when it runs its tests, it will 
output
formatted text with colors but forgets to clean the console afterwards, 
which
leaves the console with a bold red font. Any idea on how to clean that? 
Shouldn't
the environment block this kind of thing?

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

* [bug#29359] [PATCH 08/31] gnu: Add java-jansi.
  2017-11-22  9:33     ` julien lepiller
@ 2017-11-24 14:41       ` Leo Famulari
  0 siblings, 0 replies; 6+ messages in thread
From: Leo Famulari @ 2017-11-24 14:41 UTC (permalink / raw)
  To: julien lepiller; +Cc: 29359

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

On Wed, Nov 22, 2017 at 10:33:56AM +0100, julien lepiller wrote:
> There's a small issue with this package: when it runs its tests, it will
> output
> formatted text with colors but forgets to clean the console afterwards,
> which
> leaves the console with a bold red font. Any idea on how to clean that?
> Shouldn't
> the environment block this kind of thing?

The Vim package's test suite does something similar, resizing the
terminal to (I think) 80x24.

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

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

* [bug#29359] [PATCH 08/31] gnu: Add java-jansi.
  2017-11-19 17:55 [bug#29359] More java packages Julien Lepiller
@ 2017-11-24 15:31 ` Tobias Geerinckx-Rice
  2018-02-10 18:06 ` bug#29359: More java packages Julien Lepiller
  1 sibling, 0 replies; 6+ messages in thread
From: Tobias Geerinckx-Rice @ 2017-11-24 15:31 UTC (permalink / raw)
  To: julien, 29359

Julien,

On Wed, Nov 22, 2017 at 10:33:56AM +0100, julien lepiller wrote:
> There's a small issue with this package: when it runs its tests, it
> will output formatted text with colors but forgets to clean the
> console afterwards, which leaves the console with a bold red font.> Any idea on how to clean that?

You could add a reset-terminal phase after the test phase that sends an
ANSI reset sequence. Either by invoking the ‘reset’ command, which would
add a dependency on ncurses and is probably overkill, or by sending
(from memory, so check) \e[0m which resets only colour and weight and is
probably all you need here.

> Shouldn't the environment block this kind of thing?

The problem is that the state modified by these escape sequences is in
the terminal itself.

We could filter build output through a sanitiser that strips or mangles
those sequences so we only print ‘plain text’ to the terminal (while
keeping them in logs — I'm sure some source-based package managers do
just that). I don't know if that would break some tests, though.

Kind regards,

T G-R

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

* bug#29359: More java packages
  2017-11-19 17:55 [bug#29359] More java packages Julien Lepiller
  2017-11-24 15:31 ` [bug#29359] [PATCH 08/31] gnu: Add java-jansi Tobias Geerinckx-Rice
@ 2018-02-10 18:06 ` Julien Lepiller
  1 sibling, 0 replies; 6+ messages in thread
From: Julien Lepiller @ 2018-02-10 18:06 UTC (permalink / raw)
  To: 29359-done

Le Sun, 19 Nov 2017 18:55:41 +0100,
Julien Lepiller <julien@lepiller.eu> a écrit :

> Hi,
> 
> this patch series adds 30 more java packages and fixes one that now
> fails. With this, we have about 60 packages left before maven.
> 
> 
> 

Finally pushed as 09c44196e10e7f619715fcc2beb7a1e67239638e -
73a1f10a80397597436d08e1992412148dbceea8

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

end of thread, other threads:[~2018-02-10 18:07 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-19 17:55 [bug#29359] More java packages Julien Lepiller
2017-11-24 15:31 ` [bug#29359] [PATCH 08/31] gnu: Add java-jansi Tobias Geerinckx-Rice
2018-02-10 18:06 ` bug#29359: More java packages Julien Lepiller
     [not found] <handler.29359.B.151111429223890.ack@debbugs.gnu.org>
2017-11-19 17:57 ` [bug#29359] [PATCH 01/31] gnu: Add java-gson Julien Lepiller
2017-11-19 17:57   ` [bug#29359] [PATCH 08/31] gnu: Add java-jansi Julien Lepiller
2017-11-22  9:33     ` julien lepiller
2017-11-24 14:41       ` Leo Famulari

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).