Bug 35711 - introduce new modes to distuingish poweroff and reboot
Summary: introduce new modes to distuingish poweroff and reboot
Status: RESOLVED MOVED
Alias: None
Product: plymouth
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Ray Strode [halfline]
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-26 21:20 UTC by william.jon.mccann
Modified: 2018-08-07 09:25 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
Remove the calls to plymouth message (2.12 KB, patch)
2011-03-26 21:27 UTC, william.jon.mccann
Details | Splinter Review

Description william.jon.mccann 2011-03-26 21:20:28 UTC
Here's a patch to remove the plymouth messages.

 * messages aren't translated
 * console text is ugly
 * they are jargonny
 * they really aren't needed
Comment 1 william.jon.mccann 2011-03-26 21:27:13 UTC
Created attachment 44903 [details] [review]
Remove the calls to plymouth message

* messages aren't translated
 * console text is ugly
 * they are jargonny
 * they really aren't needed
Comment 2 Lennart Poettering 2011-03-28 04:10:43 UTC
Uh, should this really go away on the lowest level? 

if the messages currently aren't translated, then the obvious fix appears to be to make them translatable. 

They aren't shown as "console text", but by Plymouth' freetype support.

I'd much rather see this turned off in the theme somehow.
Comment 3 Ray Strode 2011-03-28 07:37:46 UTC
I'm assuming you added those messages because that's what Fedora did before systemd and you were trying to maintain status quo.  Note:

1) Jon was the person who added those messages to Fedora originally
2) When they were in Fedora they were translated
3) I totally agree with you that this would be better done at the theme level.  So I think you're both right.  The generic plymouth message calls should be removed, and themes that want the message can just add it themselves.

We do have one problem here that plymouth doesn't have any generic way to distinguish between shutdown and reboot.  We could fix that in plymouth by having a new mode passed to plymouthd.

In the mean time, themes that care about those messages can probably hook off the update status coming in.
Comment 4 Ray Strode 2011-03-28 07:38:32 UTC
s/When they were in Fedora/When they were originally added to initscripts in Fedora/
Comment 5 Lennart Poettering 2011-03-28 07:53:32 UTC
(In reply to comment #3)
> I'm assuming you added those messages because that's what Fedora did before
> systemd and you were trying to maintain status quo.  

Yes.

> Note:
> 
> 1) Jon was the person who added those messages to Fedora originally
> 2) When they were in Fedora they were translated
> 3) I totally agree with you that this would be better done at the theme level. 
> So I think you're both right.  The generic plymouth message calls should be
> removed, and themes that want the message can just add it themselves.
> 
> We do have one problem here that plymouth doesn't have any generic way to
> distinguish between shutdown and reboot.  We could fix that in plymouth by
> having a new mode passed to plymouthd.

Sounds good. Note that systemd by default offers four shutdown modes: poweroff, halt, reboot, kexec. 

I figure poweroff+halt and reboot+kexec need not to be distuingished in Plymouth, hence it would be cool if we could do "/sbin/plymouthd --mode=reboot" and "/sbin/plymouthd --mode=halt" from systemd and the themes add it in.
 
> In the mean time, themes that care about those messages can probably hook off
> the update status coming in.

Hmmm? I don't follow?

(anyway, I will now apply Jon's patch, and reassign this to Plymouth, so that we get those two new modes.)
Comment 6 Lennart Poettering 2011-03-28 16:21:57 UTC
The patch is now merged, reassigning to plymouth so that we get those new modes.
Comment 7 GitLab Migration User 2018-08-07 09:25:17 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/plymouth/plymouth/issues/12.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.