all messages for Guix-related lists mirrored at yhetil.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; 3+ 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] 3+ 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; 3+ 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] 3+ 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; 3+ 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] 3+ messages in thread

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

Thread overview: 3+ 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

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.