GNU bug report logs - #57659
[PATCH] gnu: Add blesh.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 57659 in the body.
You can then email your comments to 57659 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: kiasoc5 <kiasoc5 <at> disroot.org>
To: guix-patches <at> gnu.org
Cc: kiasoc5 <kiasoc5 <at> disroot.org>
Subject: [PATCH] gnu: Add blesh.
Date: Wed,  7 Sep 2022 22:11:06 -0400

* gnu/packages/bash.scm (blesh): New variable.
---
 gnu/packages/bash.scm | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/gnu/packages/bash.scm b/gnu/packages/bash.scm
index 72758560cd..2460c7cc00 100644
--- a/gnu/packages/bash.scm
+++ b/gnu/packages/bash.scm
@@ -44,6 +44,7 @@ (define-module (gnu packages bash)
   #:use-module (guix gexp)
   #:use-module (guix monads)
   #:use-module (guix store)
+  #:use-module (guix build-system copy)
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system trivial)
   #:autoload   (guix gnupg) (gnupg-verify*)
@@ -456,3 +457,25 @@ (define-public bash-ctypes
 function interface (FFI) directly in your shell.  In other words, it allows
 you to call routines in shared libraries from within Bash.")
     (license license:expat)))
+
+(define-public blesh
+  (package
+    (name "blesh")
+    (version "0.3.3")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://github.com/akinomyoga/ble.sh/releases/download/v"
+			   version "/ble-" version ".tar.xz"))
+       (sha256
+        (base32
+         "0fpkacw0r5zhby1wpsirlzq4w1yq1fmhivik47l891c2pn1n7rm7"))))
+    (build-system copy-build-system)
+    (inputs (list bash))
+    (arguments
+      (list
+	#:install-plan #~'(("." "share/blesh"))))
+    (home-page "https://github.com/akinomyoga/ble.sh/")
+    (synopsis "A line editor written in pure Bash")
+    (description "Bash Line Editor (ble.sh) is a command line editor written in pure Bash which replaces the default GNU Readline. It adds syntax highlighting, auto suggestions, vim modes, and more to Bash interactive sessions.")
+    (license license:bsd-3)))

base-commit: b45a44eaad890f31d9418dbb8cb14e3ee1d83c19
-- 
2.37.2

Message #8 received at 57659 <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: kiasoc5 <kiasoc5 <at> disroot.org>
Cc: 57659 <at> debbugs.gnu.org
Subject: Re: [bug#57659] [PATCH] gnu: Add blesh.
Date: Sun, 11 Sep 2022 20:59:15 +0100

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

kiasoc5 via Guix-patches via <guix-patches <at> gnu.org> writes:

> * gnu/packages/bash.scm (blesh): New variable.
> ---
>  gnu/packages/bash.scm | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
>
> diff --git a/gnu/packages/bash.scm b/gnu/packages/bash.scm
> index 72758560cd..2460c7cc00 100644
> --- a/gnu/packages/bash.scm
> +++ b/gnu/packages/bash.scm
> @@ -44,6 +44,7 @@ (define-module (gnu packages bash)
>    #:use-module (guix gexp)
>    #:use-module (guix monads)
>    #:use-module (guix store)
> +  #:use-module (guix build-system copy)
>    #:use-module (guix build-system gnu)
>    #:use-module (guix build-system trivial)
>    #:autoload   (guix gnupg) (gnupg-verify*)
> @@ -456,3 +457,25 @@ (define-public bash-ctypes
>  function interface (FFI) directly in your shell.  In other words, it allows
>  you to call routines in shared libraries from within Bash.")
>      (license license:expat)))
> +
> +(define-public blesh
> +  (package
> +    (name "blesh")
> +    (version "0.3.3")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append "https://github.com/akinomyoga/ble.sh/releases/download/v"
> +			   version "/ble-" version ".tar.xz"))
> +       (sha256
> +        (base32
> +         "0fpkacw0r5zhby1wpsirlzq4w1yq1fmhivik47l891c2pn1n7rm7"))))
> +    (build-system copy-build-system)
> +    (inputs (list bash))
> +    (arguments
> +      (list
> +	#:install-plan #~'(("." "share/blesh"))))
> +    (home-page "https://github.com/akinomyoga/ble.sh/")
> +    (synopsis "A line editor written in pure Bash")
> +    (description "Bash Line Editor (ble.sh) is a command line editor written in pure Bash which replaces the default GNU Readline. It adds syntax highlighting, auto suggestions, vim modes, and more to Bash interactive sessions.")
> +    (license license:bsd-3)))

Simple things first, the linter makes a number of complaints. There are
tab characters on a couple of lines and some issues with the formatting
of the description.

Maybe more importantly, this package uses the copy build system, but it
looks like ble.sh should be built with make. I realise that the output
of this build process is a mostly readable shell script, but the Guix
package should still build the source to generate it. Does that make
sense?

Thanks,

Chris

[signature.asc (application/pgp-signature, inline)]

Message #11 received at 57659 <at> debbugs.gnu.org (full text, mbox):

From: kiasoc5 <kiasoc5 <at> disroot.org>
To: mail <at> cbaines.net
Cc: kiasoc5 <kiasoc5 <at> disroot.org>, 57659 <at> debbugs.gnu.org
Subject: [PATCH v2] gnu: Add blesh.
Date: Mon, 12 Sep 2022 22:32:02 -0400

On Sun, Sep 11 2022, 08:59:15 PM +0100
Christopher Baines <mail <at> cbaines.net> wrote:

> Maybe more importantly, this package uses the copy build system, but
> it looks like ble.sh should be built with make. I realise that the
> output of this build process is a mostly readable shell script, but
> the Guix package should still build the source to generate it. Does
> that make sense?

Updated to latest tag and use git reference for makefile. However the contrib submodule fails to install during build phase, and there are 2 test failures:

```
starting phase `build'
mkdir -p out
mkdir -p out/keymap
mkdir -p out/lib
DEPENDENCIES_PHONY=1 DEPENDENCIES_OUTPUT=out/ble.dep DEPENDENCIES_TARGET=out/ble.sh FULLVER=0.4.0-devel2 \
  /gnu/store/690qz3fg334dpwn3pn6k59n4wc943p2b-gawk-5.1.0/bin/gawk -f ext/mwg_pp.awk ble.pp >/dev/null
cp -p keymap/emacs.sh out/keymap/emacs.sh
cp -p keymap/vi.sh out/keymap/vi.sh
cp -p keymap/vi_digraph.sh out/keymap/vi_digraph.sh
cp -p keymap/vi_digraph.txt out/keymap/vi_digraph.txt
cp -p keymap/vi_test.sh out/keymap/vi_test.sh
cp -p keymap/emacs.rlfunc.txt out/keymap/emacs.rlfunc.txt
cp -p keymap/vi_imap.rlfunc.txt out/keymap/vi_imap.rlfunc.txt
cp -p keymap/vi_nmap.rlfunc.txt out/keymap/vi_nmap.rlfunc.txt
cp -p lib/init-term.sh out/lib/init-term.sh
cp -p lib/init-bind.sh out/lib/init-bind.sh
cp -p lib/init-cmap.sh out/lib/init-cmap.sh
cp -p lib/init-msys1.sh out/lib/init-msys1.sh
cp -p lib/core-complete.sh out/lib/core-complete.sh
/gnu/store/690qz3fg334dpwn3pn6k59n4wc943p2b-gawk-5.1.0/bin/gawk -f ext/mwg_pp.awk lib/core-syntax.sh > out/lib/core-syntax.sh
cp -p lib/core-test.sh out/lib/core-test.sh
cp -p lib/core-edit.ignoreeof-messages.txt out/lib/core-edit.ignoreeof-messages.txt
cp -p lib/vim-surround.sh out/lib/vim-surround.sh
cp -p lib/vim-arpeggio.sh out/lib/vim-arpeggio.sh
cp -p lib/test-main.sh out/lib/test-main.sh
cp -p lib/test-util.sh out/lib/test-util.sh
fatal: not a git repository (or any parent up to mount point /tmp)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
fatal: not a git repository (or any parent up to mount point /tmp)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
phase `build' succeeded after 0.2 seconds
starting phase `check'
bash make_command.sh check
ble.sh: Insane environment: $USER is empty.
ble.sh: modified USER=nixbld
ble/term.sh: updating tput cache for TERM=dumb... done
stty: 'standard input': Inappropriate ioctl for device
MACHTYPE: x86_64-unknown-linux-gnu
BLE_VERSION: 0.4.0-devel2+
BASH_VERSION: 5.1.8(1)-release
[section] main: 14/14 (0 fail, 0 crash, 0 skip)
/tmp/guix-build-blesh-0.4.0-devel2.drv-0/source/out/lib/test-util.sh:1671: ble/util/c2s 956; [[ $ret != μ ]]
--- 289.exit.expect	2022-09-13 02:34:16.953615150 +0000
+++ 289.exit.result	2022-09-13 02:34:16.953615150 +0000
@@ -1 +1 @@
-0
+1

/tmp/guix-build-blesh-0.4.0-devel2.drv-0/source/out/lib/test-util.sh:1672: ble/util/c2s 12354; [[ $ret != あ ]]
--- 289.exit.expect	2022-09-13 02:34:16.956948541 +0000
+++ 289.exit.result	2022-09-13 02:34:16.956948541 +0000
@@ -1 +1 @@
-0
+1

[section] util: 1045/1047 (2 fail, 0 crash, 0 skip)
make: *** [GNUmakefile:158: check] Error 1

Test suite failed, dumping logs.
error: in phase 'check': uncaught exception:
%exception #<&invoke-error program: "make" arguments: ("check" "-j" "8") exit-status: 2 term-signal: #f stop-signal: #f> 
phase `check' failed after 1.9 seconds
command "make" "check" "-j" "8" failed with status 2
builder for `/gnu/store/pli4h8yvddlv3mm1bgq47l2kfadm0dfi-blesh-0.4.0-devel2.drv' failed with exit code 1
build of /gnu/store/pli4h8yvddlv3mm1bgq47l2kfadm0dfi-blesh-0.4.0-devel2.drv failed
View build log at '/var/log/guix/drvs/pl/i4h8yvddlv3mm1bgq47l2kfadm0dfi-blesh-0.4.0-devel2.drv.gz'.
guix build: error: build of `/gnu/store/pli4h8yvddlv3mm1bgq47l2kfadm0dfi-blesh-0.4.0-devel2.drv' failed
```

Help appreciated.

* gnu/packages/bash.scm (blesh): New variable.
---
 gnu/packages/bash.scm | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/gnu/packages/bash.scm b/gnu/packages/bash.scm
index 72758560cd..bf6d5fc1cc 100644
--- a/gnu/packages/bash.scm
+++ b/gnu/packages/bash.scm
@@ -37,6 +37,8 @@ (define-module (gnu packages bash)
   #:use-module (gnu packages libffi)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages guile)
+  #:use-module (gnu packages version-control)
+  #:use-module (gnu packages less)
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix git-download)
@@ -456,3 +458,38 @@ (define-public bash-ctypes
 function interface (FFI) directly in your shell.  In other words, it allows
 you to call routines in shared libraries from within Bash.")
     (license license:expat)))
+
+(define-public blesh
+  (package
+    (name "blesh")
+    (version "0.4.0-devel2")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/akinomyoga/ble.sh")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "02fdjyh4x6wr5hg3i86nsxhz8ysgjrvvxdmk6pqr0lm8ngw9p3sh"))))
+    (arguments
+     (list #:phases #~(modify-phases %standard-phases
+                        ;TODO: install contrib submodule
+                        (add-after 'unpack 'pretend-contrib-.git-exists
+                          (lambda _
+                            (mkdir-p "contrib/.git")))
+                        (add-after 'unpack 'make-readlink-work
+                          (lambda _
+                            (substitute* "ble.pp"
+                              (("PATH=/bin:/usr/bin readlink" path-to-readlink)
+                               (string-append #$coreutils "/bin/readlink")))))
+                        (delete 'configure)))) ;no configure
+    (build-system gnu-build-system)
+    (native-inputs (list git-minimal coreutils less))
+    (home-page "https://github.com/akinomyoga/ble.sh/")
+    (synopsis "Bash Line Editor")
+    (description
+     "Bash Line Editor (ble.sh) is a command line editor written in pure Bash
+which replaces the default GNU Readline.  It adds syntax highlighting, auto
+suggestions, vim modes, and more to Bash interactive sessions.")
+    (license license:bsd-3)))
-- 
2.37.3

Message #14 received at 57659 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: kiasoc5 <kiasoc5 <at> disroot.org>, mail <at> cbaines.net
Cc: 57659 <at> debbugs.gnu.org
Subject: Re: [bug#57659] [PATCH v2] gnu: Add blesh.
Date: Tue, 13 Sep 2022 14:02:02 +0200

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

On 13-09-2022 04:32, kiasoc5 via Guix-patches via wrote:
> +                        (add-after 'unpack 'make-readlink-work
> +                          (lambda _
> +                            (substitute* "ble.pp"
> +                              (("PATH=/bin:/usr/bin readlink" path-to-readlink)
> +                               (string-append #$coreutils "/bin/readlink")))))

This does not allow --with-input transformations.

Try (search-input-file inputs "bin/readlink") or (which "readlink") 
instead, depending on whether 'inputs' or 'native-inputs' is appropriate.

Greetings,
Maxime.

[OpenPGP_0x49E3EE22191725EE.asc (application/pgp-keys, attachment)]

[OpenPGP_signature (application/pgp-signature, attachment)]

Message #17 received at 57659 <at> debbugs.gnu.org (full text, mbox):

From: kiasoc5 <kiasoc5 <at> disroot.org>
To: mail <at> cbaines.net,
	maximedevos <at> telenet.be
Cc: kiasoc5 <kiasoc5 <at> disroot.org>, 57659 <at> debbugs.gnu.org
Subject: [PATCH v3] gnu: Add blesh.
Date: Tue, 13 Sep 2022 23:07:08 -0400

Tests pass now.

* gnu/packages/bash.scm (blesh): New variable.
---
 gnu/packages/bash.scm | 45 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 45 insertions(+)

diff --git a/gnu/packages/bash.scm b/gnu/packages/bash.scm
index 72758560cd..8f614e145a 100644
--- a/gnu/packages/bash.scm
+++ b/gnu/packages/bash.scm
@@ -37,6 +37,8 @@ (define-module (gnu packages bash)
   #:use-module (gnu packages libffi)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages guile)
+  #:use-module (gnu packages version-control)
+  #:use-module (gnu packages less)
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix git-download)
@@ -456,3 +458,46 @@ (define-public bash-ctypes
 function interface (FFI) directly in your shell.  In other words, it allows
 you to call routines in shared libraries from within Bash.")
     (license license:expat)))
+
+(define-public blesh
+  (package
+    (name "blesh")
+    (version "0.4.0-devel2")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/akinomyoga/ble.sh")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "02fdjyh4x6wr5hg3i86nsxhz8ysgjrvvxdmk6pqr0lm8ngw9p3sh"))))
+    (arguments
+     (list #:make-flags #~(list (string-append "PREFIX="
+                                               #$output))
+           #:phases #~(modify-phases %standard-phasesg
+                        (add-after 'unpack 'pretend-contrib-.git-exists
+                          (lambda _
+                            (mkdir-p "contrib/.git")))
+                        (add-after 'unpack 'make-readlink-work
+                          (lambda _
+                            (substitute* "ble.pp"
+                              (("PATH=/bin:/usr/bin readlink")
+                               (search-input-file %build-inputs
+                                                  "/bin/readlink")))))
+                        (delete 'configure) ;no configure
+                        (add-before 'check 'use-LC_ALL-for-tests
+                          (lambda _
+                            (setenv "LANG"
+                                    (getenv "LC_ALL"))
+                            (unsetenv "LC_ALL"))))))
+    (build-system gnu-build-system)
+    (inputs (list coreutils))
+    (native-inputs (list git-minimal less))
+    (home-page "https://github.com/akinomyoga/ble.sh")
+    (synopsis "Bash Line Editor")
+    (description
+     "Bash Line Editor (ble.sh) is a command line editor written in pure Bash
+which replaces the default GNU Readline.  It adds syntax highlighting, auto
+suggestions, vim modes, and more to Bash interactive sessions.")
+    (license license:bsd-3)))

base-commit: 491dd62b38e1772f3e50de58118d9b9ac97272ff
-- 
2.37.3

Message #22 received at 57659-done <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: kiasoc5 <kiasoc5 <at> disroot.org>
Cc: 57659-done <at> debbugs.gnu.org
Subject: Re: [PATCH v3] gnu: Add blesh.
Date: Wed, 14 Sep 2022 09:40:49 +0100

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

kiasoc5 <kiasoc5 <at> disroot.org> writes:

> Tests pass now.
>
> * gnu/packages/bash.scm (blesh): New variable.
> ---
>  gnu/packages/bash.scm | 45 +++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 45 insertions(+)

Thanks. I've tweaked the name of one of the phaess (use-LANG-for-tests
rather than use-LC_ALL-for-tests) as that seemed more correct. I also
removed some inputs that were redundant (coreutils is provided by the
gnu-build-system, so is an implicit input), and git-minimal didn't seem
to be required.

I pushed to master as 592ca01d8d25849d077035d54d969d90ed5c2609.

Thanks,

Chris

[signature.asc (application/pgp-signature, inline)]

Message #25 received at 57659 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: 57659 <at> debbugs.gnu.org, mail <at> cbaines.net, kiasoc5 <at> disroot.org
Subject: Re: bug#57659: [PATCH v3] gnu: Add blesh.
Date: Thu, 15 Sep 2022 17:26:30 +0200

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

On 14-09-2022 10:40, Christopher Baines wrote:
> 
> kiasoc5 <kiasoc5 <at> disroot.org> writes:
> 
>> Tests pass now.
>>
>> * gnu/packages/bash.scm (blesh): New variable.
>> ---
>>   gnu/packages/bash.scm | 45 +++++++++++++++++++++++++++++++++++++++++++
>>   1 file changed, 45 insertions(+)
> 
> Thanks. I've tweaked the name of one of the phaess (use-LANG-for-tests
> rather than use-LC_ALL-for-tests) as that seemed more correct. I also
> removed some inputs that were redundant (coreutils is provided by the
> gnu-build-system, so is an implicit input), and git-minimal didn't seem
> to be required.

gnu-build-system only provides a _natively compiled_ coreutils (for 
--system), not a cross-compiled coreutils (for --target).  As such, it 
is not redundant (unless it compile-time only (*), the build system 
isn't clear to me).

>+                        (add-after 'unpack 'make-readlink-work
>+                          (lambda _
>+                            (substitute* "ble.pp"
>+                              (("PATH=/bin:/usr/bin readlink")
>+                               (search-input-file %build-inputs
>+                                                  "/bin/readlink")))))

You're mixing G-exp (new) and %build-inputs (old).  How about
(search-input-file inputs "bin/readlink") or (which "readlink") 
(depending on whether inputs or native-inputs is appropriate) instead?

Greetings,
Maxime.

(*) can be verified by using "guix gc --references".

[OpenPGP_0x49E3EE22191725EE.asc (application/pgp-keys, attachment)]

[OpenPGP_signature (application/pgp-signature, attachment)]

Message #31 received at 57659 <at> debbugs.gnu.org (full text, mbox):

From: kiasoc5 <kiasoc5 <at> disroot.org>
To: Maxime Devos <maximedevos <at> telenet.be>
Cc: 57659 <at> debbugs.gnu.org, mail <at> cbaines.net
Subject: Re: bug#57659: [PATCH v3] gnu: Add blesh.
Date: Fri, 16 Sep 2022 00:05:11 +0000

On Thu, Sep 15 2022, 05:26:30 PM +0200
Maxime Devos <maximedevos <at> telenet.be> wrote:

> On 14-09-2022 10:40, Christopher Baines wrote:
> > 
> > kiasoc5 <kiasoc5 <at> disroot.org> writes:
> >   
> >> Tests pass now.
> >>
> >> * gnu/packages/bash.scm (blesh): New variable.
> >> ---
> >>   gnu/packages/bash.scm | 45
> >> +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 45
> >> insertions(+)  
> > 
> > Thanks. I've tweaked the name of one of the phaess
> > (use-LANG-for-tests rather than use-LC_ALL-for-tests) as that
> > seemed more correct. I also removed some inputs that were redundant
> > (coreutils is provided by the gnu-build-system, so is an implicit
> > input), and git-minimal didn't seem to be required.  
> 
> gnu-build-system only provides a _natively compiled_ coreutils (for 
> --system), not a cross-compiled coreutils (for --target).  As such,
> it is not redundant (unless it compile-time only (*), the build
> system isn't clear to me).

Readlink should be required for runtime, how about substituting
"PATH=/bin:/usr/bin readlink" -> "readlink" instead of the path to
readlink?

>  >+                        (add-after 'unpack 'make-readlink-work
>  >+                          (lambda _
>  >+                            (substitute* "ble.pp"
>  >+                              (("PATH=/bin:/usr/bin readlink")
>  >+                               (search-input-file %build-inputs
>  >+
>  >"/bin/readlink")))))  
> 
> You're mixing G-exp (new) and %build-inputs (old).  How about
> (search-input-file inputs "bin/readlink") or (which "readlink") 
> (depending on whether inputs or native-inputs is appropriate) instead?

If we keep coreutils as an input then it would be search-file-inputs.

-- 

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.