GNU bug report logs - #26970
plymouth

Previous Next

Package: guix-patches;

Reported by: ng0 <ng0 <at> pragmatique.xyz>

Date: Wed, 17 May 2017 19:52:02 UTC

Severity: normal

Tags: moreinfo

Done: Ricardo Wurmus <rekado <at> elephly.net>

Bug is archived. No further changes may be made.

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

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

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#26970; Package guix-patches. (Wed, 17 May 2017 19:52:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to ng0 <ng0 <at> pragmatique.xyz>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Wed, 17 May 2017 19:52:02 GMT) Full text and rfc822 format available.

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

From: ng0 <ng0 <at> pragmatique.xyz>
To: guix-patches <at> gnu.org
Subject: plymouth
Date: Wed, 17 May 2017 19:51:12 +0000
[Message part 1 (text/plain, inline)]
This patch adds plymouth, the early bootprocess daemon.
It provides a 'boot splash'.

Users will not install this on their own (or be able to
use it freestanding) which is why we need at least a
system-service for this.

I'd hate to be the bottleneck for this, so here is my work
from back in december (builds okay!).

According to Gentoo, we will need to apply changes to the
GRUB menu and are advised to disable the linux boot logo
display (https://wiki.gentoo.org/wiki/Plymouth).

I also advise like we concluded in a chat in december,
that we write a system test for this.

-- 
https://pragmatique.xyz
PGP: https://people.pragmatique.xyz/ng0/
[0001-gnu-Add-plymouth.patch (text/plain, attachment)]

Information forwarded to guix-patches <at> gnu.org:
bug#26970; Package guix-patches. (Tue, 30 May 2017 21:15:02 GMT) Full text and rfc822 format available.

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

From: Ricardo Wurmus <rekado <at> elephly.net>
To: ng0 <ng0 <at> pragmatique.xyz>
Cc: 26970 <at> debbugs.gnu.org
Subject: Re: bug#26970: plymouth
Date: Tue, 30 May 2017 23:14:11 +0200
> Subject: [PATCH] gnu: Add plymouth.
>
> * gnu/packages/freedesktop.scm (plymouth): New variable.
> ---

Thanks!

[…]

> +    (arguments
> +     `(#:configure-flags (list (string-append "--sbindir="
> +                                              (assoc-ref %outputs "out")
> +                                              "/sbin")
> +                               (string-append "--bindir="
> +                                              (assoc-ref %outputs "out")
> +                                              "/bin")
> +                               (string-append "--prefix="
> +                                              (assoc-ref %outputs "out"))
> +                               (string-append "--exec-prefix="
> +                                              (assoc-ref %outputs "out"))
> +                               (string-append "--libexecdir="
> +                                              (assoc-ref %outputs "out")
> +                                              "/lib")

Is it really necessary to set *all* of these?  Usually it’s sufficient
to just set “--prefix” and that’s done by default.

> +                               (string-append "--sysconfdir="
> +                                              "/etc")

You don’t need string-append here.  Is it *necessary* to use the global
/etc here, or can it be told (at runtime) to read its configuration
files from elsewhere?

> +                               (string-append "--localstatedir="
> +                                              "/var")
> +                               (string-append "--with-logo="
> +                                              "/etc/plymouth/logo.png")

I would prefer if we didn’t have to use the global /etc here.  You don’t
need string-append here.

> +                               (string-append "--with-background-color="
> +                                              "0x000000")
> +                               (string-append "--with-background-start-color-stop="
> +                                              "0x000000")
> +                               (string-append "--with-background-end-color-stop="
> +                                              "0x000000")

Why are these provided?

> +                               "--without-system-root-install"
> +                               "--without-rhgb-compat-link"
> +                               "--enable-tracing"
> +                               "--disable-systemd-integration"
> +                               "--enable-pango"
> +                               "--enable-gdm-transition"
> +                               "--enable-gtk")

And what about these?  Does “disable-systemd-integration” mean that it
won’t work with elogind?

> +       #:make-flags (list (string-append "plymouthd_defaultsdir="
> +                                         (assoc-ref %outputs "out")
> +                                         "/share/plymouth")
> +                          (string-append "plymouthd_confdir="
> +                                         (assoc-ref %outputs "out")
> +                                         "/etc/plymouth"))

These differ from the configure flags above that say that the global
/etc should be used.  Is this on purpose?

> +       #:phases
> +       (modify-phases %standard-phases
> +         (add-before 'configure 'fix-docbook
> +           (lambda* (#:key inputs #:allow-other-keys)
> +             (substitute* "docs/Makefile.in"
> +               (("http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl")
> +                (string-append (assoc-ref inputs "docbook-xsl")
> +                               "/xml/xsl/docbook-xsl-"
> +                               ,(package-version docbook-xsl)
> +                               "/manpages/docbook.xsl")))
> +             (setenv "XML_CATALOG_FILES"
> +                     (string-append (assoc-ref inputs "docbook-xml")
> +                                    "/xml/dtd/docbook/catalog.xml"))
> +             #t)))))
> +    (inputs
> +     `(("gtk+" ,gtk+)
> +       ("glib" ,glib)
> +       ("cairo" ,cairo)
> +       ("libdrm" ,libdrm)
> +       ("pango" ,pango)
> +       ("libpng" ,libpng)
> +       ("eudev" ,eudev)))
> +    (native-inputs
> +     `(("pkg-config" ,pkg-config)
> +       ("libxslt" ,libxslt)
> +       ("docbook-xsl" ,docbook-xsl)
> +       ("docbook-xml" ,docbook-xml)))
> +    (synopsis "Graphical boot animation (splash) and logger")
> +    (home-page "https://www.freedesktop.org/wiki/Software/Plymouth/")
> +    (description
> +     "Plymouth is an application that runs very early in the boot
> +process (even before the root filesystem is mounted!) that provides
                                      “and” ——————————————^

> +a graphical boot animation while the boot process happens in the
> +background.

I wonder how this will work “before the root filesystem is mounted” when
the store actually sits on the root filesystem.  I would prefer to only
add this once it actually works, so I’d very much like to see a proof of
concept for how to use it in a system.

-- 
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
https://elephly.net





Added tag(s) moreinfo. Request was from Ricardo Wurmus <rekado <at> elephly.net> to control <at> debbugs.gnu.org. (Tue, 30 May 2017 21:16:01 GMT) Full text and rfc822 format available.

Reply sent to Ricardo Wurmus <rekado <at> elephly.net>:
You have taken responsibility. (Thu, 07 May 2020 08:42:01 GMT) Full text and rfc822 format available.

Notification sent to ng0 <ng0 <at> pragmatique.xyz>:
bug acknowledged by developer. (Thu, 07 May 2020 08:42:02 GMT) Full text and rfc822 format available.

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

From: Ricardo Wurmus <rekado <at> elephly.net>
To: 26970-done <at> debbugs.gnu.org
Subject: plymouth
Date: Thu, 07 May 2020 10:41:51 +0200
Commit 5b9bdeb52cc25e35e74bbc10fa102e3fcc4baab9 adds Plymouth.

I updated to 0.9.4, removed all the unnecessary configure and make
flags, removed GTK support because it blew up the closure size from
~200MB to 800MB, which isn’t suitable for use in an initrd.

I tried using it in a VM, but I haven’t yet been able to make the splash
appear.  It seems to be unable to find a graphical display and switches
to the “detailed” view, which merely prints to the console.

I did try loading uvesafb in the initrd, but all I get is a garbled
screen.

On the plus side, plymouth does work as expected on my booted machine,
so I went ahead and pushed the commit.

--
Ricardo




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Thu, 04 Jun 2020 11:24:07 GMT) Full text and rfc822 format available.

This bug report was last modified 3 years and 327 days ago.

Previous Next


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