* [PATCH 0/2] systemd: fix dependency issue related to serial getty
@ 2017-09-05 8:07 Chen Qi
2017-09-05 8:07 ` [PATCH 1/2] systemd: change some RRECOMMENDS to RDEPENDS Chen Qi
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Chen Qi @ 2017-09-05 8:07 UTC (permalink / raw
To: openembedded-core
The following changes since commit 15eac6befb6c4d16bc1b814e040bd7bf006a5bee:
linux-yocto: add linux-yocto 4.12 bbappends (2017-08-31 23:37:10 +0100)
are available in the git repository at:
git://git.pokylinux.org/poky-contrib ChenQi/systemd-serialgetty
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/systemd-serialgetty
Chen Qi (2):
systemd: change some RRECOMMENDS to RDEPENDS
systemd-serialgetty: remove systemd from RDEPENDS
meta/recipes-core/systemd/systemd-serialgetty.bb | 2 --
meta/recipes-core/systemd/systemd_234.bb | 8 ++++----
2 files changed, 4 insertions(+), 6 deletions(-)
--
1.9.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 1/2] systemd: change some RRECOMMENDS to RDEPENDS
2017-09-05 8:07 [PATCH 0/2] systemd: fix dependency issue related to serial getty Chen Qi
@ 2017-09-05 8:07 ` Chen Qi
2017-09-22 9:34 ` Martin Jansa
2017-09-05 8:08 ` [PATCH 2/2] systemd-serialgetty: remove systemd from RDEPENDS Chen Qi
2017-09-11 1:43 ` [PATCH 0/2] systemd: fix dependency issue related to serial getty ChenQi
2 siblings, 1 reply; 6+ messages in thread
From: Chen Qi @ 2017-09-05 8:07 UTC (permalink / raw
To: openembedded-core
Set NO_RECOMMENDATIONS to "1", build and start a systemd image, and we
could not get serial getty spawned, thus causing the user not able to
login via serial port.
E.g.
MACHINE=qemux86-64 bitbake core-image-minimal
runqemu qemux86-64 nographic
And we cannot login onto the system.
Move util-linux-agetty and systemd-serialgetty (determined by PACKAGECONFIG)
from RRECOMMENDS to RDEPENDS to fix the above problem.
Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
meta/recipes-core/systemd/systemd_234.bb | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/meta/recipes-core/systemd/systemd_234.bb b/meta/recipes-core/systemd/systemd_234.bb
index 2736f38..2695e15 100644
--- a/meta/recipes-core/systemd/systemd_234.bb
+++ b/meta/recipes-core/systemd/systemd_234.bb
@@ -492,13 +492,13 @@ FILES_${PN} = " ${base_bindir}/* \
FILES_${PN}-dev += "${base_libdir}/security/*.la ${datadir}/dbus-1/interfaces/ ${sysconfdir}/rpm/macros.systemd"
-RDEPENDS_${PN} += "kmod dbus util-linux-mount udev (= ${EXTENDPKGV})"
+RDEPENDS_${PN} += "kmod dbus util-linux-mount udev (= ${EXTENDPKGV}) util-linux-agetty"
+RDEPENDS_${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'serial-getty-generator', '', 'systemd-serialgetty', d)}"
RDEPENDS_${PN} += "volatile-binds update-rc.d"
-RRECOMMENDS_${PN} += "${@bb.utils.contains('PACKAGECONFIG', 'serial-getty-generator', '', 'systemd-serialgetty', d)} \
- systemd-extra-utils \
+RRECOMMENDS_${PN} += "systemd-extra-utils \
systemd-compat-units udev-hwdb \
- util-linux-agetty util-linux-fsck e2fsprogs-e2fsck \
+ util-linux-fsck e2fsprogs-e2fsck \
kernel-module-autofs4 kernel-module-unix kernel-module-ipv6 \
os-release \
"
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/2] systemd-serialgetty: remove systemd from RDEPENDS
2017-09-05 8:07 [PATCH 0/2] systemd: fix dependency issue related to serial getty Chen Qi
2017-09-05 8:07 ` [PATCH 1/2] systemd: change some RRECOMMENDS to RDEPENDS Chen Qi
@ 2017-09-05 8:08 ` Chen Qi
2017-09-11 1:43 ` [PATCH 0/2] systemd: fix dependency issue related to serial getty ChenQi
2 siblings, 0 replies; 6+ messages in thread
From: Chen Qi @ 2017-09-05 8:08 UTC (permalink / raw
To: openembedded-core
By setting systemd-serialgetty to rdepend on systemd, we are making
configuration files have runtime dependency on the main utility that
uses those configuration files. Applied with the same logic, we should
make any package that provides service files under /etc/init.d/ to have
runtime dependency on sysvinit. And this is not right.
So we should remove systemd from RDEPENDS of systemd-serialgetty.
Besides, as we have changed systemd to have systemd-serialgetty in its
RDEPENDS by default, we should avoid circular dependency issue.
Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
meta/recipes-core/systemd/systemd-serialgetty.bb | 2 --
1 file changed, 2 deletions(-)
diff --git a/meta/recipes-core/systemd/systemd-serialgetty.bb b/meta/recipes-core/systemd/systemd-serialgetty.bb
index 768b130..8d1ec7c 100644
--- a/meta/recipes-core/systemd/systemd-serialgetty.bb
+++ b/meta/recipes-core/systemd/systemd-serialgetty.bb
@@ -38,8 +38,6 @@ do_install() {
fi
}
-RDEPENDS_${PN} = "systemd"
-
# This is a machine specific file
FILES_${PN} = "${systemd_unitdir}/system/*.service ${sysconfdir}"
PACKAGE_ARCH = "${MACHINE_ARCH}"
--
1.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 0/2] systemd: fix dependency issue related to serial getty
2017-09-05 8:07 [PATCH 0/2] systemd: fix dependency issue related to serial getty Chen Qi
2017-09-05 8:07 ` [PATCH 1/2] systemd: change some RRECOMMENDS to RDEPENDS Chen Qi
2017-09-05 8:08 ` [PATCH 2/2] systemd-serialgetty: remove systemd from RDEPENDS Chen Qi
@ 2017-09-11 1:43 ` ChenQi
2 siblings, 0 replies; 6+ messages in thread
From: ChenQi @ 2017-09-11 1:43 UTC (permalink / raw
To: openembedded-core
ping
On 09/05/2017 04:07 PM, Chen Qi wrote:
> The following changes since commit 15eac6befb6c4d16bc1b814e040bd7bf006a5bee:
>
> linux-yocto: add linux-yocto 4.12 bbappends (2017-08-31 23:37:10 +0100)
>
> are available in the git repository at:
>
> git://git.pokylinux.org/poky-contrib ChenQi/systemd-serialgetty
> http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/systemd-serialgetty
>
> Chen Qi (2):
> systemd: change some RRECOMMENDS to RDEPENDS
> systemd-serialgetty: remove systemd from RDEPENDS
>
> meta/recipes-core/systemd/systemd-serialgetty.bb | 2 --
> meta/recipes-core/systemd/systemd_234.bb | 8 ++++----
> 2 files changed, 4 insertions(+), 6 deletions(-)
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] systemd: change some RRECOMMENDS to RDEPENDS
2017-09-05 8:07 ` [PATCH 1/2] systemd: change some RRECOMMENDS to RDEPENDS Chen Qi
@ 2017-09-22 9:34 ` Martin Jansa
2017-09-26 8:38 ` ChenQi
0 siblings, 1 reply; 6+ messages in thread
From: Martin Jansa @ 2017-09-22 9:34 UTC (permalink / raw
To: Chen Qi; +Cc: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 2815 bytes --]
This doesn't seem to work correctly for MACHINEs with empty SERIAL_CONSOLE,
because with empty SERIAL_CONSOLE there is no systemd-serialgetty package
(because it's also empty) and the build fails with:
Collected errors:
* calculate_dependencies_for: Cannot satisfy the following dependencies
for packagegroup-core-boot:
* systemd-serialgetty *
* opkg_solver_install: Cannot install package packagegroup-core-boot.
If you're setting NO_RECOMMENDATIONS to "1", then you should expect that
some recommended functionality will be missing, no?
On Tue, Sep 5, 2017 at 10:07 AM, Chen Qi <Qi.Chen@windriver.com> wrote:
> Set NO_RECOMMENDATIONS to "1", build and start a systemd image, and we
> could not get serial getty spawned, thus causing the user not able to
> login via serial port.
>
> E.g.
> MACHINE=qemux86-64 bitbake core-image-minimal
> runqemu qemux86-64 nographic
>
> And we cannot login onto the system.
>
> Move util-linux-agetty and systemd-serialgetty (determined by
> PACKAGECONFIG)
> from RRECOMMENDS to RDEPENDS to fix the above problem.
>
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> ---
> meta/recipes-core/systemd/systemd_234.bb | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-core/systemd/systemd_234.bb
> b/meta/recipes-core/systemd/systemd_234.bb
> index 2736f38..2695e15 100644
> --- a/meta/recipes-core/systemd/systemd_234.bb
> +++ b/meta/recipes-core/systemd/systemd_234.bb
> @@ -492,13 +492,13 @@ FILES_${PN} = " ${base_bindir}/* \
>
> FILES_${PN}-dev += "${base_libdir}/security/*.la
> ${datadir}/dbus-1/interfaces/ ${sysconfdir}/rpm/macros.systemd"
>
> -RDEPENDS_${PN} += "kmod dbus util-linux-mount udev (= ${EXTENDPKGV})"
> +RDEPENDS_${PN} += "kmod dbus util-linux-mount udev (= ${EXTENDPKGV})
> util-linux-agetty"
> +RDEPENDS_${PN} += "${@bb.utils.contains('PACKAGECONFIG',
> 'serial-getty-generator', '', 'systemd-serialgetty', d)}"
> RDEPENDS_${PN} += "volatile-binds update-rc.d"
>
> -RRECOMMENDS_${PN} += "${@bb.utils.contains('PACKAGECONFIG',
> 'serial-getty-generator', '', 'systemd-serialgetty', d)} \
> - systemd-extra-utils \
> +RRECOMMENDS_${PN} += "systemd-extra-utils \
> systemd-compat-units udev-hwdb \
> - util-linux-agetty util-linux-fsck e2fsprogs-e2fsck
> \
> + util-linux-fsck e2fsprogs-e2fsck \
> kernel-module-autofs4 kernel-module-unix
> kernel-module-ipv6 \
> os-release \
> "
> --
> 1.9.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>
[-- Attachment #2: Type: text/html, Size: 4240 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/2] systemd: change some RRECOMMENDS to RDEPENDS
2017-09-22 9:34 ` Martin Jansa
@ 2017-09-26 8:38 ` ChenQi
0 siblings, 0 replies; 6+ messages in thread
From: ChenQi @ 2017-09-26 8:38 UTC (permalink / raw
To: Martin Jansa; +Cc: Patches and discussions about the oe-core layer
[-- Attachment #1: Type: text/plain, Size: 4048 bytes --]
On 09/22/2017 05:34 PM, Martin Jansa wrote:
> This doesn't seem to work correctly for MACHINEs with
> empty SERIAL_CONSOLE, because with empty SERIAL_CONSOLE there is no
> systemd-serialgetty package (because it's also empty) and the build
> fails with:
>
> Collected errors:
> * calculate_dependencies_for: Cannot satisfy the following
> dependencies for packagegroup-core-boot:
> * systemd-serialgetty *
> * opkg_solver_install: Cannot install package packagegroup-core-boot.
>
> If you're setting NO_RECOMMENDATIONS to "1", then you should expect
> that some recommended functionality will be missing, no?
>
Hi Martin,
Thanks a lot for pointing out this situation.
I've sent out a follow-up fix which sets ALLOW_EMPTY_${PN} = "1" in
systemd-serialtty.bb.
I hope this would be sufficient.
I agree with you that if NO_RECOMMENDATIONS is set to "1", the lack of
some functionality is allowed.
But if the user doesn't set anything related to serial console, and he's
building our reference image (e.g. core-image-minimal) on our supported
BSP (e.g. qemux86-64), the user should not be astonished by being unable
to get to login.
Anyway, I hope my follow-up fix would fix things up.
Best Regards,
Chen Qi
> On Tue, Sep 5, 2017 at 10:07 AM, Chen Qi <Qi.Chen@windriver.com
> <mailto:Qi.Chen@windriver.com>> wrote:
>
> Set NO_RECOMMENDATIONS to "1", build and start a systemd image, and we
> could not get serial getty spawned, thus causing the user not able to
> login via serial port.
>
> E.g.
> MACHINE=qemux86-64 bitbake core-image-minimal
> runqemu qemux86-64 nographic
>
> And we cannot login onto the system.
>
> Move util-linux-agetty and systemd-serialgetty (determined by
> PACKAGECONFIG)
> from RRECOMMENDS to RDEPENDS to fix the above problem.
>
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com
> <mailto:Qi.Chen@windriver.com>>
> ---
> meta/recipes-core/systemd/systemd_234.bb <http://systemd_234.bb>
> | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-core/systemd/systemd_234.bb
> <http://systemd_234.bb> b/meta/recipes-core/systemd/systemd_234.bb
> <http://systemd_234.bb>
> index 2736f38..2695e15 100644
> --- a/meta/recipes-core/systemd/systemd_234.bb <http://systemd_234.bb>
> +++ b/meta/recipes-core/systemd/systemd_234.bb <http://systemd_234.bb>
> @@ -492,13 +492,13 @@ FILES_${PN} = " ${base_bindir}/* \
>
> FILES_${PN}-dev += "${base_libdir}/security/*.la
> ${datadir}/dbus-1/interfaces/ ${sysconfdir}/rpm/macros.systemd"
>
> -RDEPENDS_${PN} += "kmod dbus util-linux-mount udev (= ${EXTENDPKGV})"
> +RDEPENDS_${PN} += "kmod dbus util-linux-mount udev (=
> ${EXTENDPKGV}) util-linux-agetty"
> +RDEPENDS_${PN} += "${@bb.utils.contains('PACKAGECONFIG',
> 'serial-getty-generator', '', 'systemd-serialgetty', d)}"
> RDEPENDS_${PN} += "volatile-binds update-rc.d"
>
> -RRECOMMENDS_${PN} += "${@bb.utils.contains('PACKAGECONFIG',
> 'serial-getty-generator', '', 'systemd-serialgetty', d)} \
> - systemd-extra-utils \
> +RRECOMMENDS_${PN} += "systemd-extra-utils \
> systemd-compat-units udev-hwdb \
> - util-linux-agetty util-linux-fsck
> e2fsprogs-e2fsck \
> + util-linux-fsck e2fsprogs-e2fsck \
> kernel-module-autofs4 kernel-module-unix
> kernel-module-ipv6 \
> os-release \
> "
> --
> 1.9.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> <mailto:Openembedded-core@lists.openembedded.org>
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
> <http://lists.openembedded.org/mailman/listinfo/openembedded-core>
>
>
[-- Attachment #2: Type: text/html, Size: 7031 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-09-26 8:32 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-05 8:07 [PATCH 0/2] systemd: fix dependency issue related to serial getty Chen Qi
2017-09-05 8:07 ` [PATCH 1/2] systemd: change some RRECOMMENDS to RDEPENDS Chen Qi
2017-09-22 9:34 ` Martin Jansa
2017-09-26 8:38 ` ChenQi
2017-09-05 8:08 ` [PATCH 2/2] systemd-serialgetty: remove systemd from RDEPENDS Chen Qi
2017-09-11 1:43 ` [PATCH 0/2] systemd: fix dependency issue related to serial getty ChenQi
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.