All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [git commit] system: only expose getty options for busybox and sysvinit
@ 2018-01-12 21:07 Thomas Petazzoni
  2018-01-16 19:39 ` Peter Korsgaard
  2018-01-30 22:50 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2018-01-12 21:07 UTC (permalink / raw)
  To: buildroot

commit: https://git.buildroot.net/buildroot/commit/?id=5e23eb5da7b3848cc6b317af9d8c23aac3a13260
branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master

Only busybox and sysvinit handle the BR2_TARGET_GENERIC_GETTY_TERM and
BR2_TARGET_GENERIC_GETTY_OPTIONS options; the other init systems do
not.

So, protect those options behind appropriate dependencies on busybox
or sysvinit.

Fixes #10301.

Reported-by: Michael Heinemann <posted@heine.so>
Suggested-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
 system/Config.in | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/system/Config.in b/system/Config.in
index b971492..d48cf8d 100644
--- a/system/Config.in
+++ b/system/Config.in
@@ -355,12 +355,16 @@ config BR2_TARGET_GENERIC_GETTY_BAUDRATE
 config BR2_TARGET_GENERIC_GETTY_TERM
 	string "TERM environment variable"
 	default "vt100"
+	# currently observed only by busybox and sysvinit
+	depends on BR2_INIT_BUSYBOX || BR2_INIT_SYSV
 	help
 	  Specify a TERM type.
 
 config BR2_TARGET_GENERIC_GETTY_OPTIONS
 	string "other options to pass to getty"
 	default ""
+	# currently observed only by busybox and sysvinit
+	depends on BR2_INIT_BUSYBOX || BR2_INIT_SYSV
 	help
 	  Any other flags you want to pass to getty,
 	  Refer to getty --help for details.

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

* [Buildroot] [git commit] system: only expose getty options for busybox and sysvinit
  2018-01-12 21:07 [Buildroot] [git commit] system: only expose getty options for busybox and sysvinit Thomas Petazzoni
@ 2018-01-16 19:39 ` Peter Korsgaard
  2018-01-30 22:50 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2018-01-16 19:39 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 > commit: https://git.buildroot.net/buildroot/commit/?id=5e23eb5da7b3848cc6b317af9d8c23aac3a13260
 > branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master

 > Only busybox and sysvinit handle the BR2_TARGET_GENERIC_GETTY_TERM and
 > BR2_TARGET_GENERIC_GETTY_OPTIONS options; the other init systems do
 > not.

 > So, protect those options behind appropriate dependencies on busybox
 > or sysvinit.

 > Fixes #10301.

 > Reported-by: Michael Heinemann <posted@heine.so>
 > Suggested-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Committed to 2017.11.x, thanks.

-- 
Bye, Peter Korsgaard

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

* [Buildroot] [git commit] system: only expose getty options for busybox and sysvinit
  2018-01-12 21:07 [Buildroot] [git commit] system: only expose getty options for busybox and sysvinit Thomas Petazzoni
  2018-01-16 19:39 ` Peter Korsgaard
@ 2018-01-30 22:50 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2018-01-30 22:50 UTC (permalink / raw)
  To: buildroot

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 > commit: https://git.buildroot.net/buildroot/commit/?id=5e23eb5da7b3848cc6b317af9d8c23aac3a13260
 > branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/master

 > Only busybox and sysvinit handle the BR2_TARGET_GENERIC_GETTY_TERM and
 > BR2_TARGET_GENERIC_GETTY_OPTIONS options; the other init systems do
 > not.

 > So, protect those options behind appropriate dependencies on busybox
 > or sysvinit.

 > Fixes #10301.

 > Reported-by: Michael Heinemann <posted@heine.so>
 > Suggested-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
 > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>

Committed to 2017.02.x, thanks.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2018-01-30 22:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-12 21:07 [Buildroot] [git commit] system: only expose getty options for busybox and sysvinit Thomas Petazzoni
2018-01-16 19:39 ` Peter Korsgaard
2018-01-30 22:50 ` Peter Korsgaard

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.