unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
* bug#55449: recutils cross-compilation "fix" breaks bash builtins
@ 2022-05-16 12:52 Liliana Marie Prikler
  2022-05-16 13:37 ` Tobias Geerinckx-Rice via Bug reports for GNU Guix
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Liliana Marie Prikler @ 2022-05-16 12:52 UTC (permalink / raw)
  To: 55449; +Cc: Tobias Geerinckx-Rice

Hi Guix,

The fix in commit 20fbd870938e239c038d8524a56729f123f19f80, which lets
recutils cross-compile unfortunately omits support for the bash
builtins in all build modes, as recutils can't actually detect bash
headers there.

Unfortunately, recutils' configure.ac silently swallows this error in
the following check.
  AM_CONDITIONAL([BASH_BUILTINS],
                 [test "x$bash_headers_available" = "xyes" && 
                  test "x$bash_builtins_enabled" = "xyes"])
I only noticed, because I symlink the builtins to lib/bash, where
they're actually needed, which causes runpath validation to fail
because the symlink points to a file that doesn't exist.  I fixed this
locally, but still wanted y'all to know.

Is there a way we can support bash headers in cross-compilation
contexts?  I don't think having bash:include as a native input is even
helpful here, is it?  WDYT?




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

* bug#55449: recutils cross-compilation "fix" breaks bash builtins
  2022-05-16 12:52 bug#55449: recutils cross-compilation "fix" breaks bash builtins Liliana Marie Prikler
@ 2022-05-16 13:37 ` Tobias Geerinckx-Rice via Bug reports for GNU Guix
  2022-05-16 15:15 ` Maxime Devos
  2022-06-04  8:30 ` bug#55449: [PATCH] gnu: recutils: Use correct bash headers Liliana Marie Prikler
  2 siblings, 0 replies; 5+ messages in thread
From: Tobias Geerinckx-Rice via Bug reports for GNU Guix @ 2022-05-16 13:37 UTC (permalink / raw)
  To: liliana.prikler, 55449

Yeah, I toyed with the idea of a 'assert-readrec-existence phase but it seemed a bit silly at the time.  And look, now it vanished.

Headers being a native input looked and still looks hella sus to me, but presumably done for a reason...?  (They said, optimistically.)

Kind regards,

T G-R

Sent on the go.  Excuse or enjoy my brevity.




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

* bug#55449: recutils cross-compilation "fix" breaks bash builtins
  2022-05-16 12:52 bug#55449: recutils cross-compilation "fix" breaks bash builtins Liliana Marie Prikler
  2022-05-16 13:37 ` Tobias Geerinckx-Rice via Bug reports for GNU Guix
@ 2022-05-16 15:15 ` Maxime Devos
  2022-06-04  8:30 ` bug#55449: [PATCH] gnu: recutils: Use correct bash headers Liliana Marie Prikler
  2 siblings, 0 replies; 5+ messages in thread
From: Maxime Devos @ 2022-05-16 15:15 UTC (permalink / raw)
  To: Liliana Marie Prikler, 55449; +Cc: Tobias Geerinckx-Rice

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

Liliana Marie Prikler schreef op ma 16-05-2022 om 14:52 [+0200]:
> Is there a way we can support bash headers in cross-compilation
> contexts?  I don't think having bash:include as a native input is even
> helpful here, is it?  WDYT?
> 
> 


Maybe you can give <https://issues.guix.gnu.org/55377> a try ... needs
a rebase though.  No idea if it helps here.

Greetings,
Maxime.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

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

* bug#55449: [PATCH] gnu: recutils: Use correct bash headers.
  2022-05-16 12:52 bug#55449: recutils cross-compilation "fix" breaks bash builtins Liliana Marie Prikler
  2022-05-16 13:37 ` Tobias Geerinckx-Rice via Bug reports for GNU Guix
  2022-05-16 15:15 ` Maxime Devos
@ 2022-06-04  8:30 ` Liliana Marie Prikler
  2023-06-18  7:39   ` Liliana Marie Prikler
  2 siblings, 1 reply; 5+ messages in thread
From: Liliana Marie Prikler @ 2022-06-04  8:30 UTC (permalink / raw)
  To: 55449; +Cc: Tobias Geerinckx-Rice, Maxime Devos

* gnu/packages/databases.scm (recutils)[arguments]: Convert to G-Expressions.
Drop gratuitous dirname.
[native-inputs]: Drop labels.  Move bash:include...
[inputs]: ... here.  Also add regular bash.
---
 gnu/packages/databases.scm | 19 ++++++++-----------
 1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index 09d825af3e..5cd8cc2fa3 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -1527,20 +1527,17 @@ (define-public recutils
     (build-system gnu-build-system)
     (arguments
      (list #:configure-flags
-           '(list "--disable-static"
-                  (string-append "--with-bash-headers="
-                                 (dirname (search-input-directory
-                                           %build-inputs
-                                           "include/bash"))))))
+           #~(list "--disable-static"
+                   (string-append "--with-bash-headers="
+                                  (search-input-directory %build-inputs
+                                                          "include/bash")))))
     (native-inputs
-     ;; XXX Without labels, the default 'configure phase picks the wrong "bash".
-     `(("bc" ,bc)
-       ("bash:include" ,bash "include")
-       ("check" ,check-0.14)
-       ("pkg-config" ,pkg-config)))
+     (list bc check-0.14 pkg-config))
     (inputs
      ;; TODO: Add more optional inputs.
-     (list curl
+     (list bash                         ; /bin/bash for native compilation
+           `(,bash "include")
+           curl
            libgcrypt
            `(,util-linux "lib")))
     (synopsis "Manipulate plain text files as databases")
-- 
2.36.1





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

* bug#55449: [PATCH] gnu: recutils: Use correct bash headers.
  2022-06-04  8:30 ` bug#55449: [PATCH] gnu: recutils: Use correct bash headers Liliana Marie Prikler
@ 2023-06-18  7:39   ` Liliana Marie Prikler
  0 siblings, 0 replies; 5+ messages in thread
From: Liliana Marie Prikler @ 2023-06-18  7:39 UTC (permalink / raw)
  To: 55449-done; +Cc: Tobias Geerinckx-Rice, Maxime Devos

Am Samstag, dem 04.06.2022 um 10:30 +0200 schrieb Liliana Marie
Prikler:
> * gnu/packages/databases.scm (recutils)[arguments]: Convert to G-
> Expressions.
> Drop gratuitous dirname.
> [native-inputs]: Drop labels.  Move bash:include...
> [inputs]: ... here.  Also add regular bash.
> ---
Hi Guix, I've pushed this now.

There's still a UX issue in that recutils doesn't put these libraries
into lib/bash, but at the very least it works as described in the
manual right now.

Cheers




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

end of thread, other threads:[~2023-06-18  7:40 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-05-16 12:52 bug#55449: recutils cross-compilation "fix" breaks bash builtins Liliana Marie Prikler
2022-05-16 13:37 ` Tobias Geerinckx-Rice via Bug reports for GNU Guix
2022-05-16 15:15 ` Maxime Devos
2022-06-04  8:30 ` bug#55449: [PATCH] gnu: recutils: Use correct bash headers Liliana Marie Prikler
2023-06-18  7:39   ` Liliana Marie Prikler

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