GNU bug report logs - #50375
[PATCH] gnu: Add mlterm.

Previous Next

Full log

View this message in rfc822 format

From: Xinglu Chen <public <at> yoctocell.xyz>
To: iKaSeRo <ahmed <at> ikasero.com>, 50375 <at> debbugs.gnu.org
Cc: iKaSeRo <ahmed <at> ikasero.com>
Subject: [bug#50375] [PATCH v2] gnu: Add mlterm.
Date: Sun, 05 Sep 2021 20:43:08 +0200

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

On Sun, Sep 05 2021, iKaSeRo wrote:

> * gnu/packages/terminals.scm (mlterm): New variable.
> ---
> Xinglu Chen <public <at> yoctocell.xyz> writes:
>
>>> * gnu/packages/terminals.scm (mlterm): New variable.
>>> ---
>>>  gnu/packages/terminals.scm | 40 ++++++++++++++++++++++++++++++++++++++
>>>  1 file changed, 40 insertions(+)
>>>
>>> diff --git a/gnu/packages/terminals.scm b/gnu/packages/terminals.scm
>>> index 1cdf0e1831..037504ee4f 100644
>>> --- a/gnu/packages/terminals.scm
>>> +++ b/gnu/packages/terminals.scm
>>> @@ -25,6 +25,7 @@
>>>  ;;; Copyright © 2020 Leo Famulari <leo <at> famulari.name>
>>>  ;;; Copyright © 2021 Ekaitz Zarraga <ekaitz <at> elenq.tech>
>>>  ;;; Copyright © 2021 Raphaël Mélotte <raphael.melotte <at> mind.be>
>>> +;;; Copyright © 2021 ikasero <ahmed <at> ikasero.com>
>>>  ;;;
>>>  ;;; This file is part of GNU Guix.
>>>  ;;;
>>> @@ -66,6 +67,7 @@
>>>    #:use-module (gnu packages docbook)
>>>    #:use-module (gnu packages fontutils)
>>>    #:use-module (gnu packages freedesktop)
>>> +  #:use-module (gnu packages fribidi)
>>>    #:use-module (gnu packages gettext)
>>>    #:use-module (gnu packages ghostscript)
>>>    #:use-module (gnu packages gl)
>>> @@ -1458,3 +1460,41 @@ ephemeral serial ports.  It features automatic port detection, port enumeration,
>>>  support for non-standard baud rates, the ability to wait for ports to appear,
>>>  and the ability to read and write via stdin and stdout.")
>>>      (license license:expat)))
>>> +
>>> +(define-public mlterm
>>> +  (package
>>> +    (name "mlterm")
>>> +    (version "3.9.1")
>>> +    (source
>>> +     (origin
>>> +       (method url-fetch)
>>> +       (uri (string-append "mirror://sourceforge/mlterm/01release/mlterm-"
>>> +                           version "/mlterm-" version ".tar.gz"))
>>> +       (sha256
>>> +        (base32
>>> +         "03fnynwv7d1aicwk2rp31sgncv5m65agvygqvsgn59v9di40gnnb"))))
>>> +    (build-system gnu-build-system)
>>> +    (arguments `(#:tests? #f
>>
>> What’s the reason for disabling tests; there should be a comment with a
>> short explanation.
>>
> Because there are no tests, Now I added a comment thanks!
>> Nit: This is usually on its own line
>>
>>   (arguments
>>    `(#:tests? #f
>>      #:configure-flags
>>      ...))
>>
> Didn't know that, fixed
>>> +                 #:configure-flags
>>> +                 (list (string-append "--prefix=" (assoc-ref %outputs "out"))
>>> +                       "--disable-static"
>>> +                       "--enable-optimize-redrawing"
>>> +                       "--with-imagelib=gdk-pixbuf")))
>>> +    (native-inputs
>>> +     `(("gettext" ,gettext-minimal)
>>> +       ("pkg-config" ,pkg-config)))
>>> +    (inputs
>>> +     `(("cairo" ,cairo)
>>> +       ("fontconfig" ,fontconfig)
>>> +       ("freetype" ,freetype)
>>> +       ("fribidi" ,fribidi)
>>> +       ("gdk-pixbuf" ,gdk-pixbuf)
>>> +       ("gtk+" ,gtk+)
>>> +       ("libx11" ,libx11)
>>> +       ("libxext" ,libxext)
>>> +       ("libxft" ,libxft)))
>>> +    (home-page "http://mlterm.sourceforge.net/")
>>> +    (synopsis "Multi Lingual TERMinal emulator")
>>> +    (description "mlterm is a multi-lingual terminal emulator which supports
>>> +various character sets and encodings in the world.")
>>> +    (license license:bsd-3)))
>>> -- 
>>> 2.33.0
>>
>> The package builds fine, and the rest LGTM!
> Thanks for the review!

You are welcome!  The updated patch looks good, but I don’t have commit
access.

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