systemd: remove SYSTEMD_PAGER= profile settings
There was support to inject an empty "SYSTEMD_PAGER=" variable
into the profile. This was to workaround ANSI control characters
being injected into the default systemd tool output, but has a
side-effect of disabling the pager for useful commands like
`journalctl`.
The underlying cause of these control characters was that busybox less
was missing features that systemd relies on. Many of these busybox
features say "this is essential for less applet to work with ...
systemd": https://git.busybox.net/busybox/tree/miscutils/less.c#n103
These are now enabled so this empty environment variable can be removed.
(From meta-phosphor rev: d8f8d70f7e4771eb295fff3ca80182b6636411b6)
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
Change-Id: Idc2fef9ccfb28e7956caf4efc141698940879f65
Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
diff --git a/meta-phosphor/classes/obmc-phosphor-image.bbclass b/meta-phosphor/classes/obmc-phosphor-image.bbclass
index b2d3b5e..da8c337 100644
--- a/meta-phosphor/classes/obmc-phosphor-image.bbclass
+++ b/meta-phosphor/classes/obmc-phosphor-image.bbclass
@@ -86,11 +86,6 @@
rm ${IMAGE_ROOTFS}${sysconfdir}/version
}
-disable_systemd_pager() {
- echo "SYSTEMD_PAGER=" >> ${IMAGE_ROOTFS}${sysconfdir}/profile
- echo "export SYSTEMD_PAGER" >> ${IMAGE_ROOTFS}${sysconfdir}/profile
-}
-
enable_ldap_nsswitch() {
sed -i 's/\(\(passwd\|group\|shadow\):\s*\).*/\1files ldap/' \
"${IMAGE_ROOTFS}${sysconfdir}/nsswitch.conf"
diff --git a/meta-phosphor/recipes-phosphor/images/obmc-phosphor-image-base.bb b/meta-phosphor/recipes-phosphor/images/obmc-phosphor-image-base.bb
index 2d91377..38b1c2d 100644
--- a/meta-phosphor/recipes-phosphor/images/obmc-phosphor-image-base.bb
+++ b/meta-phosphor/recipes-phosphor/images/obmc-phosphor-image-base.bb
@@ -8,7 +8,3 @@
# The /etc/version file is misleading and not useful. Remove it.
# Users should instead rely on /etc/os-release.
ROOTFS_POSTPROCESS_COMMAND += "remove_etc_version ; "
-
-# Disable the pager to prevent systemd injecting control characters into the
-# output stream that are not interpreted by busybox tools.
-ROOTFS_POSTPROCESS_COMMAND += "disable_systemd_pager ; "
diff --git a/meta-phosphor/recipes-phosphor/images/obmc-phosphor-image.bb b/meta-phosphor/recipes-phosphor/images/obmc-phosphor-image.bb
index 63908a1..76515fa 100644
--- a/meta-phosphor/recipes-phosphor/images/obmc-phosphor-image.bb
+++ b/meta-phosphor/recipes-phosphor/images/obmc-phosphor-image.bb
@@ -48,10 +48,6 @@
# Users should instead rely on /etc/os-release.
ROOTFS_POSTPROCESS_COMMAND += "remove_etc_version ; "
-# Disable the pager to prevent systemd injecting control characters into the
-# output stream that are not interpreted by busybox tools.
-ROOTFS_POSTPROCESS_COMMAND += "disable_systemd_pager ; "
-
# The shadow recipe provides the binaries(like useradd, usermod) needed by the
# phosphor-user-manager.
ROOTFS_RO_UNNEEDED_remove = "shadow"