All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/4] bb.codeparser.py: Remove reference for oe.utils.contains
@ 2014-11-12 10:33 Otavio Salvador
  2014-11-12 10:33 ` [PATCH 2/4] toolchain-scripts.bbclass: Export KCFLAGS to ensure sysroot is provided Otavio Salvador
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Otavio Salvador @ 2014-11-12 10:33 UTC (permalink / raw
  To: OpenEmbedded Core Mailing List; +Cc: Otavio Salvador

The oe.utils.contains function has been removed from OE-Core metadata
as the references for it has been replaced to use the
bb.utils.contains.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 bitbake/lib/bb/codeparser.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bitbake/lib/bb/codeparser.py b/bitbake/lib/bb/codeparser.py
index 8b8f91a..21a36f6 100644
--- a/bitbake/lib/bb/codeparser.py
+++ b/bitbake/lib/bb/codeparser.py
@@ -178,7 +178,7 @@ class BufferedLogger(Logger):
 
 class PythonParser():
     getvars = (".getVar", ".appendVar", ".prependVar")
-    containsfuncs = ("bb.utils.contains", "base_contains", "oe.utils.contains", "bb.utils.contains_any")
+    containsfuncs = ("bb.utils.contains", "base_contains", "bb.utils.contains_any")
     execfuncs = ("bb.build.exec_func", "bb.build.exec_task")
 
     def warn(self, func, arg):
-- 
2.1.1



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

* [PATCH 2/4] toolchain-scripts.bbclass: Export KCFLAGS to ensure sysroot is provided
  2014-11-12 10:33 [PATCH 1/4] bb.codeparser.py: Remove reference for oe.utils.contains Otavio Salvador
@ 2014-11-12 10:33 ` Otavio Salvador
  2014-11-12 10:33 ` [PATCH 3/4] sysvinit-inittab: Disable the carrier detect requirement for serial consoles Otavio Salvador
  2014-11-12 10:33 ` [PATCH 4/4] systemd-serialgetty: " Otavio Salvador
  2 siblings, 0 replies; 6+ messages in thread
From: Otavio Salvador @ 2014-11-12 10:33 UTC (permalink / raw
  To: OpenEmbedded Core Mailing List; +Cc: Otavio Salvador

When building the U-Boot the lack of a proper sysroot can trigger
following error:

,----
| arm-poky-linux-gnueabi-ld.bfd: cannot find -lgcc
| make[2]: *** [examples/standalone/hello_world] Error 1
| make[1]: *** [examples/standalone] Error 2
| make: *** [examples] Error 2
`----

Guillaume Fournier has posted a very complete analysis of the
problem[1].

1. https://lists.yoctoproject.org/pipermail/meta-freescale/2014-November/011270.html

The use of KCFLAGS makes the build of U-Boot work out of box, now that
it uses the Linux kernel build system.

Reported-by: Guillaume Fournier <gfournier@brioconcept.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 meta/classes/toolchain-scripts.bbclass | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/classes/toolchain-scripts.bbclass b/meta/classes/toolchain-scripts.bbclass
index 2244cf7..cf99c73 100644
--- a/meta/classes/toolchain-scripts.bbclass
+++ b/meta/classes/toolchain-scripts.bbclass
@@ -68,6 +68,7 @@ toolchain_shared_env_script () {
 	echo 'export CXXFLAGS="${TARGET_CXXFLAGS}"' >> $script
 	echo 'export LDFLAGS="${TARGET_LDFLAGS}"' >> $script
 	echo 'export CPPFLAGS="${TARGET_CPPFLAGS}"' >> $script
+	echo 'export KCFLAGS="--sysroot=$SDKTARGETSYSROOT"' >> $script
 	echo 'export OECORE_DISTRO_VERSION="${DISTRO_VERSION}"' >> $script
 	echo 'export OECORE_SDK_VERSION="${SDK_VERSION}"' >> $script
 	echo 'export ARCH=${ARCH}' >> $script
-- 
2.1.1



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

* [PATCH 3/4] sysvinit-inittab: Disable the carrier detect requirement for serial consoles
  2014-11-12 10:33 [PATCH 1/4] bb.codeparser.py: Remove reference for oe.utils.contains Otavio Salvador
  2014-11-12 10:33 ` [PATCH 2/4] toolchain-scripts.bbclass: Export KCFLAGS to ensure sysroot is provided Otavio Salvador
@ 2014-11-12 10:33 ` Otavio Salvador
  2014-11-26 16:01   ` [dizzy backport] " Javier Viguera
  2014-11-12 10:33 ` [PATCH 4/4] systemd-serialgetty: " Otavio Salvador
  2 siblings, 1 reply; 6+ messages in thread
From: Otavio Salvador @ 2014-11-12 10:33 UTC (permalink / raw
  To: OpenEmbedded Core Mailing List; +Cc: Otavio Salvador

This aligns the params of getty with the ones used in Debian. From the
getty(8) manpage:

,----[ getty(8) manpage ]
|  -L, --local-line
|
|    Force the line to be a local line with no need for carrier
| 	 detect. This can be useful when you have a locally attached
| 	 terminal where the serial line does not set the carrier detect
| 	 signal.
`----

Reported-by: Craig McQueen <craig.mcqueen@beamcommunications.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
Tested-by: Craig McQueen <craig.mcqueen@beamcommunications.com>
---
 meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
index c3244b4..657ef02 100644
--- a/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
+++ b/meta/recipes-core/sysvinit/sysvinit-inittab_2.88dsf.bb
@@ -23,7 +23,7 @@ do_install() {
     do
 	j=`echo ${i} | sed s/\;/\ /g`
 	label=`echo ${i} | sed -e 's/^.*;tty//' -e 's/;.*//'`
-	echo "$label:12345:respawn:${base_sbindir}/getty ${j}" >> ${D}${sysconfdir}/inittab
+	echo "$label:12345:respawn:${base_sbindir}/getty -L ${j}" >> ${D}${sysconfdir}/inittab
     done
 
     if [ "${USE_VT}" = "1" ]; then
-- 
2.1.1



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

* [PATCH 4/4] systemd-serialgetty: Disable the carrier detect requirement for serial consoles
  2014-11-12 10:33 [PATCH 1/4] bb.codeparser.py: Remove reference for oe.utils.contains Otavio Salvador
  2014-11-12 10:33 ` [PATCH 2/4] toolchain-scripts.bbclass: Export KCFLAGS to ensure sysroot is provided Otavio Salvador
  2014-11-12 10:33 ` [PATCH 3/4] sysvinit-inittab: Disable the carrier detect requirement for serial consoles Otavio Salvador
@ 2014-11-12 10:33 ` Otavio Salvador
  2 siblings, 0 replies; 6+ messages in thread
From: Otavio Salvador @ 2014-11-12 10:33 UTC (permalink / raw
  To: OpenEmbedded Core Mailing List; +Cc: Otavio Salvador

This aligns the params of getty with the ones used in Debian. From the
getty(8) manpage:

,----[ getty(8) manpage ]
|  -L, --local-line
|
|    Force the line to be a local line with no need for carrier
|    detect. This can be useful when you have a locally attached
|    terminal where the serial line does not set the carrier detect
|    signal.
`----

Reported-by: Craig McQueen <craig.mcqueen@beamcommunications.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 meta/recipes-core/systemd/systemd-serialgetty/serial-getty@.service | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/systemd/systemd-serialgetty/serial-getty@.service b/meta/recipes-core/systemd/systemd-serialgetty/serial-getty@.service
index 6dd335c..182167b 100644
--- a/meta/recipes-core/systemd/systemd-serialgetty/serial-getty@.service
+++ b/meta/recipes-core/systemd/systemd-serialgetty/serial-getty@.service
@@ -21,7 +21,7 @@ IgnoreOnIsolate=yes
 
 [Service]
 Environment="TERM=xterm"
-ExecStart=-/sbin/agetty -8 --keep-baud %I @BAUDRATE@ $TERM
+ExecStart=-/sbin/agetty -8 -L --keep-baud %I @BAUDRATE@ $TERM
 Type=idle
 Restart=always
 RestartSec=0
-- 
2.1.1



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

* [dizzy backport] [PATCH 3/4] sysvinit-inittab: Disable the carrier detect requirement for serial consoles
  2014-11-12 10:33 ` [PATCH 3/4] sysvinit-inittab: Disable the carrier detect requirement for serial consoles Otavio Salvador
@ 2014-11-26 16:01   ` Javier Viguera
  2014-11-26 22:09     ` akuster
  0 siblings, 1 reply; 6+ messages in thread
From: Javier Viguera @ 2014-11-26 16:01 UTC (permalink / raw
  Cc: openembedded-core

I see this patch has been committed in master. Could it be backported to 
dizzy?

This fixes a problem with some commands (for example busybox 'less' 
applet) that become unresponsive when used in a serial console on some 
hardware platforms (for example a FSL mx28).

A basic: 'less /etc/passwd' just hangs showing nothing and you can only 
stop it with CRTL-C.

As the backport seems pretty safe in general, could it be backported?

-
Regards,

Javier Viguera


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

* Re: [dizzy backport] [PATCH 3/4] sysvinit-inittab: Disable the carrier detect requirement for serial consoles
  2014-11-26 16:01   ` [dizzy backport] " Javier Viguera
@ 2014-11-26 22:09     ` akuster
  0 siblings, 0 replies; 6+ messages in thread
From: akuster @ 2014-11-26 22:09 UTC (permalink / raw
  To: Javier Viguera; +Cc: openembedded-core



On 11/26/2014 08:01 AM, Javier Viguera wrote:
> I see this patch has been committed in master. Could it be backported to
> dizzy?

Will look into.

>
> This fixes a problem with some commands (for example busybox 'less'
> applet) that become unresponsive when used in a serial console on some
> hardware platforms (for example a FSL mx28).
>
> A basic: 'less /etc/passwd' just hangs showing nothing and you can only
> stop it with CRTL-C.
>
> As the backport seems pretty safe in general, could it be backported?

sure.

- Armin
>
> -
> Regards,
>
> Javier Viguera


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

end of thread, other threads:[~2014-11-26 22:09 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-12 10:33 [PATCH 1/4] bb.codeparser.py: Remove reference for oe.utils.contains Otavio Salvador
2014-11-12 10:33 ` [PATCH 2/4] toolchain-scripts.bbclass: Export KCFLAGS to ensure sysroot is provided Otavio Salvador
2014-11-12 10:33 ` [PATCH 3/4] sysvinit-inittab: Disable the carrier detect requirement for serial consoles Otavio Salvador
2014-11-26 16:01   ` [dizzy backport] " Javier Viguera
2014-11-26 22:09     ` akuster
2014-11-12 10:33 ` [PATCH 4/4] systemd-serialgetty: " Otavio Salvador

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.