Re: [OE-core] [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices

[Mike Looijmans]

On 12/17/2014 11:08 PM, Gary Thomas wrote:

On 2014-12-17 15:03, Burton, Ross wrote:

Hi Mike,

On 5 December 2014 at 09:58, Mike Looijmans mailto:mike.looijm...@topic.nl>> wrote:

Upon inserting a USB stick or similar device, mdev will run
an automounter script that mounts valid partitions on
/media/. The script first checks /etc/fstab entries
so that mounting on UUID or LABEL or using custom mount options
is still possible. If /etc/fstab does not contain particular
mount options, the script will create (and remove) the mountpoint
automatically.
The script also supports full disk partitions (devices without
partition table).


Thanks for this, but can this script be put into a separate recipe to make
it even easier for people who don't want it, or want their own version?  The
prior art here is the udev
automounting script which is in udev-extraconf.


Also it would be nice if the path matched the most recent
use in udev which is /run/media/


Of course, these locations need to move around every few years, otherwise 
people would be expecting things to remain in place...


I'd rather keep using the /media symlink, as it gives distributions a chance 
to move things around.


On the satellite settop boxes, /media is a 64k tmpfs mount. This is because 
when a mount fails, the box may end up writing a TV recording to wherever 
/media happens to be. It that's flash or a big tmpfs, the box will commit 
suicide in mere seconds by either filling the flash or eating up the precious 
RAM. And for some things, the box really needs a big tmpfs, so just reducing 
the volatiles' sizes wasn't an option.


Having said that, if this is blocking, I'll do a s./media./run/media.g on the 
file, getting this mainlined has higher priority.


M.


Met vriendelijke groet / kind regards,

Mike Looijmans
System Expert


TOPIC Embedded Systems
Eindhovenseweg 32-C, NL-5683 KH Best
Postbus 440, NL-5680 AK Best
Telefoon: (+31) (0) 499 33 69 79
Telefax:  (+31) (0) 499 33 69 70
E-mail: mike.looijm...@topic.nl
Website: www.topic.nl

Please consider the environment before printing this e-mail

Topic zoekt gedreven (embedded) software specialisten!
http://topic.nl/vacatures/topic-zoekt-software-engineers/

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices

[Mike Looijmans]

On 12/17/2014 11:03 PM, Burton, Ross wrote:

Hi Mike,

On 5 December 2014 at 09:58, Mike Looijmans mailto:mike.looijm...@topic.nl>> wrote:

Upon inserting a USB stick or similar device, mdev will run
an automounter script that mounts valid partitions on
/media/. The script first checks /etc/fstab entries
so that mounting on UUID or LABEL or using custom mount options
is still possible. If /etc/fstab does not contain particular
mount options, the script will create (and remove) the mountpoint
automatically.
The script also supports full disk partitions (devices without
partition table).

Thanks for this, but can this script be put into a separate recipe to make it
even easier for people who don't want it, or want their own version?  The
prior art here is the udev automounting script which is in udev-extraconf.


Putting the script into its own package is no problem, but how does one 
arrange this with mdev.conf?


For the automounter to work, mdev.conf needs extra entries to call the script 
when block devices appear/disappear.


Unlike udev, mdev only has one config file.


Met vriendelijke groet / kind regards,

Mike Looijmans
System Expert


TOPIC Embedded Systems
Eindhovenseweg 32-C, NL-5683 KH Best
Postbus 440, NL-5680 AK Best
Telefoon: (+31) (0) 499 33 69 79
Telefax:  (+31) (0) 499 33 69 70
E-mail: mike.looijm...@topic.nl
Website: www.topic.nl

Please consider the environment before printing this e-mail

Topic zoekt gedreven (embedded) software specialisten!
http://topic.nl/vacatures/topic-zoekt-software-engineers/

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/1] dhcp: fix for systemd service file

[ChenQi]

On 12/17/2014 06:21 PM, Enrico Scholz wrote:

ChenQi  writes:


1. Set Type=forking for dhcpd.service to make it start up correctly.
2. Add PIDFile=/var/run/dhcpd.pid

why forking? Starting dhcpd with '-f --no-pid' should chose the more
simple non-forking variant without a clumsy.pidfile.


Enrico


V2 sent, using '-f --no-pid'.

Regards,
Chen Qi
--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH V2 0/1] dhcp: fix for systemd service file

[Chen Qi]

The following changes since commit e6c6d3fcfd2faf867e8145d25c1ba197fb9ee6b5:

  build-appliance-image: Update to dizzy head revision (2014-09-23 22:10:21 
+0100)

are available in the git repository at:

  git://git.pokylinux.org/poky-contrib ChenQi/systemd-dhcpd
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/systemd-dhcpd

Chen Qi (1):
  dhcp: fix for systemd service file

 meta/recipes-connectivity/dhcp/files/dhcpd.service | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH V2 1/1] dhcp: fix for systemd service file

[Chen Qi]

1. Add '-f --no-pid' option to make dhcpd start correctly
2. Do not force existence of the environment file.

Signed-off-by: Chen Qi 
---
 meta/recipes-connectivity/dhcp/files/dhcpd.service | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-connectivity/dhcp/files/dhcpd.service 
b/meta/recipes-connectivity/dhcp/files/dhcpd.service
index 8648f1a..484f46a 100644
--- a/meta/recipes-connectivity/dhcp/files/dhcpd.service
+++ b/meta/recipes-connectivity/dhcp/files/dhcpd.service
@@ -4,8 +4,8 @@ After=network.target
 After=time-sync.target
 
 [Service]
-EnvironmentFile=@SYSCONFDIR@/default/dhcp-server
-ExecStart=-@SBINDIR@/dhcpd -q $INTERFACES
+EnvironmentFile=-@SYSCONFDIR@/default/dhcp-server
+ExecStart=-@SBINDIR@/dhcpd -f --no-pid -q $INTERFACES
 
 [Install]
 WantedBy=multi-user.target
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH v2] file: upgrade to 5.21

[Li xin]

file 5.21 fix CVE-2014-8116 and CVE-2014-8117.

Signed-off-by: Li Xin 
---
 .../file/file/debian-742262-backport.patch | 32 
 .../recipes-devtools/file/file/debian-742262.patch | 19 
 meta/recipes-devtools/file/file_5.18.bb| 35 --
 meta/recipes-devtools/file/file_5.21.bb| 35 ++
 4 files changed, 67 insertions(+), 54 deletions(-)
 create mode 100644 meta/recipes-devtools/file/file/debian-742262-backport.patch
 delete mode 100644 meta/recipes-devtools/file/file/debian-742262.patch
 delete mode 100644 meta/recipes-devtools/file/file_5.18.bb
 create mode 100644 meta/recipes-devtools/file/file_5.21.bb

diff --git a/meta/recipes-devtools/file/file/debian-742262-backport.patch 
b/meta/recipes-devtools/file/file/debian-742262-backport.patch
new file mode 100644
index 000..8c68d62
--- /dev/null
+++ b/meta/recipes-devtools/file/file/debian-742262-backport.patch
@@ -0,0 +1,32 @@
+From 1d147dd57a63003da63a2df32030b17d5f1c08b6 Mon Sep 17 00:00:00 2001
+From: Li xin 
+Date: Thu, 18 Dec 2014 10:39:29 +0900
+Subject: [PATCH] The awk pattern was checked *before* the Perl pattern, so the
+  perl script with BEGIN{...} would be reported as awk, this patch fixes it.
+
+Upstream-Status: Backport [debian]
+
+the author of this patch is Christoph Biedl 
+and Robert Yang 
+
+Signed-off-by: Li Xin 
+---
+ magic/Magdir/commands | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/magic/Magdir/commands b/magic/Magdir/commands
+index 3d97489..689caa5 100644
+--- a/magic/Magdir/commands
 b/magic/Magdir/commands
+@@ -57,7 +57,7 @@
+ 0 string/wt   #!\ /usr/bin/awkawk script text executable
+ !:mimetext/x-awk
+ 0 regex/4096  =^\\s{0,100}BEGIN\\s{0,100}[{]  awk or perl script text
+-
++!:strength - 12
+ # AT&T Bell Labs' Plan 9 shell
+ 0 string/wt   #!\ /bin/rc Plan 9 rc shell script text executable
+ 
+-- 
+1.8.4.2
+
diff --git a/meta/recipes-devtools/file/file/debian-742262.patch 
b/meta/recipes-devtools/file/file/debian-742262.patch
deleted file mode 100644
index d57258c..000
--- a/meta/recipes-devtools/file/file/debian-742262.patch
+++ /dev/null
@@ -1,19 +0,0 @@
-The awk pattern was checked *before* the Perl pattern, so the perl
-script with BEGIN{...} would be reported as awk, this patch fixes it.
-
-Upstream-Status: Backport [debian]
-
-Signed-off-by: Christoph Biedl 
-Signed-off-by: Robert Yang 

-diff --git a/magic/Magdir/commands b/magic/Magdir/commands
 a/magic/Magdir/commands
-+++ b/magic/Magdir/commands
-@@ -50,6 +50,7 @@
- 0 string/wt   #!\ /usr/bin/awkawk script text executable
- !:mimetext/x-awk
- 0 regex   =^\\s{0,100}BEGIN\\s{0,100}[{]  awk script text
-+!:strength - 12
- 
- # AT&T Bell Labs' Plan 9 shell
- 0 string/wt   #!\ /bin/rc Plan 9 rc shell script text executable
diff --git a/meta/recipes-devtools/file/file_5.18.bb 
b/meta/recipes-devtools/file/file_5.18.bb
deleted file mode 100644
index 9068b93..000
--- a/meta/recipes-devtools/file/file_5.18.bb
+++ /dev/null
@@ -1,35 +0,0 @@
-SUMMARY = "File classification tool"
-DESCRIPTION = "File attempts to classify files depending \
-on their contents and prints a description if a match is found."
-HOMEPAGE = "http://www.darwinsys.com/file/";
-SECTION = "console/utils"
-
-# two clause BSD
-LICENSE = "BSD"
-LIC_FILES_CHKSUM = 
"file://COPYING;beginline=2;md5=6a7382872edb68d33e1a9398b6e03188"
-
-DEPENDS = "zlib file-native"
-DEPENDS_class-native = "zlib-native"
-
-SRC_URI = "ftp://ftp.astron.com/pub/file/file-${PV}.tar.gz \
-   file://debian-742262.patch \
-  "
-
-SRC_URI[md5sum] = "d420d8f2990cd344673acfbf8d76ff5a"
-SRC_URI[sha256sum] = 
"6519fb706d583231c2419592ebecdbb21d33c62eaf7a1a0b24ddfcb80c08bf07"
-
-inherit autotools
-
-FILES_${PN} += "${datadir}/misc/*.mgc"
-
-do_install_append_class-native() {
-   create_cmdline_wrapper ${D}/${bindir}/file \
-   --magic-file ${datadir}/misc/magic.mgc
-}
-
-do_install_append_class-nativesdk() {
-   create_cmdline_wrapper ${D}/${bindir}/file \
-   --magic-file ${datadir}/misc/magic.mgc
-}
-
-BBCLASSEXTEND = "native nativesdk"
diff --git a/meta/recipes-devtools/file/file_5.21.bb 
b/meta/recipes-devtools/file/file_5.21.bb
new file mode 100644
index 000..74acfeb
--- /dev/null
+++ b/meta/recipes-devtools/file/file_5.21.bb
@@ -0,0 +1,35 @@
+SUMMARY = "File classification tool"
+DESCRIPTION = "File attempts to classify files depending \
+on their contents and prints a description if a match is found."
+HOMEPAGE = "http://www.darwinsys.com/file/";
+SECTION = "console/utils"
+
+# two clause BSD
+LICENSE = "BSD"
+LIC_FILES_CHKSUM = 
"file://COPYING;beginline=2;md5=6a7382872edb68d33e1a9398b6e03188"
+
+DEPENDS = "zlib file-native"
+DEPENDS_class-native = "zlib-native"
+
+SRC_URI = "ftp://ftp.astron.com/pub/file/file-${PV}.tar.gz \
+file://debian-742262-backp

[OE-core] [PATCH] file: upgrade to 5.21

[Li xin]

file 5.21 fix CVE-2014-8116 and CVE-2014-8117.

Signed-off-by: Li Xin 
---
 .../file/file/debian-742262-backport.patch | 32 
 .../recipes-devtools/file/file/debian-742262.patch | 19 
 meta/recipes-devtools/file/file_5.18.bb| 35 --
 meta/recipes-devtools/file/file_5.21.bb| 35 ++
 4 files changed, 67 insertions(+), 54 deletions(-)
 create mode 100644 meta/recipes-devtools/file/file/debian-742262-backport.patch
 delete mode 100644 meta/recipes-devtools/file/file/debian-742262.patch
 delete mode 100644 meta/recipes-devtools/file/file_5.18.bb
 create mode 100644 meta/recipes-devtools/file/file_5.21.bb

diff --git a/meta/recipes-devtools/file/file/debian-742262-backport.patch 
b/meta/recipes-devtools/file/file/debian-742262-backport.patch
new file mode 100644
index 000..8c68d62
--- /dev/null
+++ b/meta/recipes-devtools/file/file/debian-742262-backport.patch
@@ -0,0 +1,32 @@
+From 1d147dd57a63003da63a2df32030b17d5f1c08b6 Mon Sep 17 00:00:00 2001
+From: Li xin 
+Date: Thu, 18 Dec 2014 10:39:29 +0900
+Subject: [PATCH] The awk pattern was checked *before* the Perl pattern, so the
+  perl script with BEGIN{...} would be reported as awk, this patch fixes it.
+
+Upstream-Status: Backport [debian]
+
+the author of this patch is Christoph Biedl 
+and Robert Yang 
+
+Signed-off-by: Li Xin 
+---
+ magic/Magdir/commands | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/magic/Magdir/commands b/magic/Magdir/commands
+index 3d97489..689caa5 100644
+--- a/magic/Magdir/commands
 b/magic/Magdir/commands
+@@ -57,7 +57,7 @@
+ 0 string/wt   #!\ /usr/bin/awkawk script text executable
+ !:mimetext/x-awk
+ 0 regex/4096  =^\\s{0,100}BEGIN\\s{0,100}[{]  awk or perl script text
+-
++!:strength - 12
+ # AT&T Bell Labs' Plan 9 shell
+ 0 string/wt   #!\ /bin/rc Plan 9 rc shell script text executable
+ 
+-- 
+1.8.4.2
+
diff --git a/meta/recipes-devtools/file/file/debian-742262.patch 
b/meta/recipes-devtools/file/file/debian-742262.patch
deleted file mode 100644
index d57258c..000
--- a/meta/recipes-devtools/file/file/debian-742262.patch
+++ /dev/null
@@ -1,19 +0,0 @@
-The awk pattern was checked *before* the Perl pattern, so the perl
-script with BEGIN{...} would be reported as awk, this patch fixes it.
-
-Upstream-Status: Backport [debian]
-
-Signed-off-by: Christoph Biedl 
-Signed-off-by: Robert Yang 

-diff --git a/magic/Magdir/commands b/magic/Magdir/commands
 a/magic/Magdir/commands
-+++ b/magic/Magdir/commands
-@@ -50,6 +50,7 @@
- 0 string/wt   #!\ /usr/bin/awkawk script text executable
- !:mimetext/x-awk
- 0 regex   =^\\s{0,100}BEGIN\\s{0,100}[{]  awk script text
-+!:strength - 12
- 
- # AT&T Bell Labs' Plan 9 shell
- 0 string/wt   #!\ /bin/rc Plan 9 rc shell script text executable
diff --git a/meta/recipes-devtools/file/file_5.18.bb 
b/meta/recipes-devtools/file/file_5.18.bb
deleted file mode 100644
index 9068b93..000
--- a/meta/recipes-devtools/file/file_5.18.bb
+++ /dev/null
@@ -1,35 +0,0 @@
-SUMMARY = "File classification tool"
-DESCRIPTION = "File attempts to classify files depending \
-on their contents and prints a description if a match is found."
-HOMEPAGE = "http://www.darwinsys.com/file/";
-SECTION = "console/utils"
-
-# two clause BSD
-LICENSE = "BSD"
-LIC_FILES_CHKSUM = 
"file://COPYING;beginline=2;md5=6a7382872edb68d33e1a9398b6e03188"
-
-DEPENDS = "zlib file-native"
-DEPENDS_class-native = "zlib-native"
-
-SRC_URI = "ftp://ftp.astron.com/pub/file/file-${PV}.tar.gz \
-   file://debian-742262.patch \
-  "
-
-SRC_URI[md5sum] = "d420d8f2990cd344673acfbf8d76ff5a"
-SRC_URI[sha256sum] = 
"6519fb706d583231c2419592ebecdbb21d33c62eaf7a1a0b24ddfcb80c08bf07"
-
-inherit autotools
-
-FILES_${PN} += "${datadir}/misc/*.mgc"
-
-do_install_append_class-native() {
-   create_cmdline_wrapper ${D}/${bindir}/file \
-   --magic-file ${datadir}/misc/magic.mgc
-}
-
-do_install_append_class-nativesdk() {
-   create_cmdline_wrapper ${D}/${bindir}/file \
-   --magic-file ${datadir}/misc/magic.mgc
-}
-
-BBCLASSEXTEND = "native nativesdk"
diff --git a/meta/recipes-devtools/file/file_5.21.bb 
b/meta/recipes-devtools/file/file_5.21.bb
new file mode 100644
index 000..c61954b
--- /dev/null
+++ b/meta/recipes-devtools/file/file_5.21.bb
@@ -0,0 +1,35 @@
+SUMMARY = "File classification tool"
+DESCRIPTION = "File attempts to classify files depending \
+on their contents and prints a description if a match is found."
+HOMEPAGE = "http://www.darwinsys.com/file/";
+SECTION = "console/utils"
+
+# two clause BSD
+LICENSE = "BSD"
+LIC_FILES_CHKSUM = 
"file://COPYING;beginline=2;md5=6a7382872edb68d33e1a9398b6e03188"
+
+DEPENDS = "zlib file-native"
+DEPENDS_class-native = "zlib-native"
+
+SRC_URI = "ftp://ftp.astron.com/pub/file/file-${PV}.tar.gz \
+file://debian-742262.patch

[OE-core] [PATCH] Upgrade qemu to version 2.2.0

[Kai Kang]

Hi Ross,

This upgrade includes the signle step support for aarch64, so I'll drop the
patch which add the single step support in the qemuarm64 serial commits.

Kai Kang (1):
  qemu: upgrade to 2.2.0

 .../files/Qemu-Arm-versatilepb-Add-memory-size-checking.patch  | 10 +++---
 meta/recipes-devtools/qemu/{qemu_2.1.2.bb => qemu_2.2.0.bb}|  4 ++--
 2 files changed, 9 insertions(+), 5 deletions(-)
 rename meta/recipes-devtools/qemu/{qemu_2.1.2.bb => qemu_2.2.0.bb} (82%)

-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] qemu: upgrade to 2.2.0

[Kai Kang]

Upgrade qemu from version 2.1.2 to 2.2.0.

Update Qemu-Arm-versatilepb-Add-memory-size-checking.patch for new
version qemu.

Signed-off-by: Kai Kang 
---
 .../files/Qemu-Arm-versatilepb-Add-memory-size-checking.patch  | 10 +++---
 meta/recipes-devtools/qemu/{qemu_2.1.2.bb => qemu_2.2.0.bb}|  4 ++--
 2 files changed, 9 insertions(+), 5 deletions(-)
 rename meta/recipes-devtools/qemu/{qemu_2.1.2.bb => qemu_2.2.0.bb} (82%)

diff --git 
a/meta/recipes-devtools/qemu/files/Qemu-Arm-versatilepb-Add-memory-size-checking.patch
 
b/meta/recipes-devtools/qemu/files/Qemu-Arm-versatilepb-Add-memory-size-checking.patch
index 7f1c5a9..b0c2ea5 100644
--- 
a/meta/recipes-devtools/qemu/files/Qemu-Arm-versatilepb-Add-memory-size-checking.patch
+++ 
b/meta/recipes-devtools/qemu/files/Qemu-Arm-versatilepb-Add-memory-size-checking.patch
@@ -14,6 +14,10 @@ Signed-off-by: Jiang Lu 
 Updated it on 2014-01-15 for rebasing
 
 Signed-off-by: Robert Yang 
+
+Update it when upgrade qemu to 2.2.0
+
+Signed-off-by: Kai Kang 
 ---
  hw/arm/versatilepb.c |6 ++
  1 file changed, 6 insertions(+)
@@ -22,7 +26,7 @@ diff --git a/hw/arm/versatilepb.c b/hw/arm/versatilepb.c
 index b48d84c..ad2cd5a 100644
 --- a/hw/arm/versatilepb.c
 +++ b/hw/arm/versatilepb.c
-@@ -199,6 +199,12 @@ static void versatile_init(QEMUMachineInitArgs *args, int 
board_id)
+@@ -198,6 +198,12 @@ static void versatile_init(MachineState *machine, int 
board_id)
  fprintf(stderr, "Unable to find CPU definition\n");
  exit(1);
  }
@@ -32,9 +36,9 @@ index b48d84c..ad2cd5a 100644
 +((unsigned int)ram_size / (1 << 20)));
 +exit(1);
 +}
- memory_region_init_ram(ram, NULL, "versatile.ram", machine->ram_size);
+ memory_region_init_ram(ram, NULL, "versatile.ram", machine->ram_size,
+&error_abort);
  vmstate_register_ram_global(ram);
- /* ??? RAM should repeat to fill physical memory space.  */
 -- 
 1.7.10.4
 
diff --git a/meta/recipes-devtools/qemu/qemu_2.1.2.bb 
b/meta/recipes-devtools/qemu/qemu_2.2.0.bb
similarity index 82%
rename from meta/recipes-devtools/qemu/qemu_2.1.2.bb
rename to meta/recipes-devtools/qemu/qemu_2.2.0.bb
index 55ae7b6..209b910 100644
--- a/meta/recipes-devtools/qemu/qemu_2.1.2.bb
+++ b/meta/recipes-devtools/qemu/qemu_2.2.0.bb
@@ -8,8 +8,8 @@ SRC_URI += "file://configure-fix-Darwin-target-detection.patch \
 file://Qemu-Arm-versatilepb-Add-memory-size-checking.patch \
 "
 SRC_URI_prepend = "http://wiki.qemu-project.org/download/${BP}.tar.bz2";
-SRC_URI[md5sum] = "0ff197c4ed4b695620bc4734e77c888f"
-SRC_URI[sha256sum] = 
"fd10f5e45cf5a736fa5a3e1c279ae9821534e700beb7d1aab88a07648a394885"
+SRC_URI[md5sum] = "f7a5e2da22d057eb838a91da7aff43c8"
+SRC_URI[sha256sum] = 
"b68c9b6c7c694f5489b5a6bffe993cd976ffbb78e7d178eb3bc016caf460039c"
 
 COMPATIBLE_HOST_class-target_mips64 = "null"
 
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 09/24] libpng: add neon option for aarch64

[Kang Kai]

On 2014年12月17日 19:04, Burton, Ross wrote:

I find Koen's patch for libpng neater so could you merge that into your
qemuarm64 branch?  The autobuilder has been doing some builds so this
should be the last stage before it's all merged.


OK. I'll do it.

--Kai



Ross

On 17 December 2014 at 08:47, Kang Kai  wrote:

  On 2014年12月17日 03:30, Koen Kooi wrote:

  Op 7 nov. 2014, om 09:51 heeft Kai Kang  
 het volgende geschreven:

Add neon option for aarch64 just as for arm. It fails with:

| ./.libs/libpng16.so: undefined reference to `png_init_filter_functions_neon'
| collect2: error: ld returned 1 exit status

Signed-off-by: Kai Kang  
---
meta/recipes-multimedia/libpng/libpng_1.6.13.bb | 1 +
1 file changed, 1 insertion(+)

diff --git a/meta/recipes-multimedia/libpng/libpng_1.6.13.bb 
b/meta/recipes-multimedia/libpng/libpng_1.6.13.bb
index 0c6fd1f..3d32bfe 100644
--- a/meta/recipes-multimedia/libpng/libpng_1.6.13.bb
+++ b/meta/recipes-multimedia/libpng/libpng_1.6.13.bb
@@ -20,6 +20,7 @@ inherit autotools binconfig-disabled pkgconfig

# Work around missing symbols
EXTRA_OECONF_append_arm = " ${@bb.utils.contains("TUNE_FEATURES", "neon", 
"--enable-arm-neon=on", "--enable-arm-neon=off" ,d)}"
+EXTRA_OECONF_append_aarch64 = " ${@bb.utils.contains("TUNE_FEATURES", "neon", 
"--enable-arm-neon=on", "--enable-arm-neon=off" ,d)}"

  You're introducing the same bug in a different way: overeager use of 
qualifiers. Just drop the _arm bit and it works.



OK. I'll remove the _arm to fix this issue.

Thanks.

--
Regards,
Neil | Kai Kang





--
Regards,
Neil | Kai Kang

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] terminal: Disable shopt errexit

[Richard Tollerton]

`sh -e` causes any errors encountered during shell execution to abort
the shell. For terminal sessions like devshell, this causes errors
encountered while executing do_terminal to instantaneously kill the
session. This is explicitly not what we want for interactive/debugging
situations like with terminal, so just remove the '-e' from the sh
invocation.

Signed-off-by: Richard Tollerton 
---
 meta/classes/terminal.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/terminal.bbclass b/meta/classes/terminal.bbclass
index e577c6d..51e9aa3 100644
--- a/meta/classes/terminal.bbclass
+++ b/meta/classes/terminal.bbclass
@@ -23,7 +23,7 @@ def emit_terminal_func(command, envdata, d):
 bb.utils.mkdirhier(os.path.dirname(runfile))
 
 with open(runfile, 'w') as script:
-script.write('#!/bin/sh -e\n')
+script.write('#!/bin/sh\n')
 bb.data.emit_func(cmd_func, script, envdata)
 script.write(cmd_func)
 script.write("\n")
-- 
2.1.3

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] glibc-testing: explicitly specify system headers location for g++

[Juro Bystricky]

The following glibc test programs fail to build when compiled using g++:
  /elf/tst-unique3cc
  /elf/tst-unique4cc
  /elf/tst-unique3lib.cc
  /elf/tst-unique3lib2.cc

The build fails because g++ system include files were not found. g++ is
invoked with -nostdinc option, so location of system include files must
be specified explicitly. glibc configure attempts to determine the
location, but this often fails for a number of reasons.

This patch tries to determine the location of g++ system include headers
just before running the glibc test suite. It queries configparms for
sysroot and the g++ compiler being used for building the test programs.
Then it queries g++ for version and machine information and based on this
information it assumes include headers in corresponding locations.

The header locations are assumed to be:
/usr/include/c++/
/usr/include/c++//

The header locations are subsequently written out to configparms.

[YOCTO #7081]

Signed-off-by: Juro Bystricky 
---
 meta/recipes-core/glibc/glibc-testing.inc | 19 +++
 1 file changed, 19 insertions(+)

diff --git a/meta/recipes-core/glibc/glibc-testing.inc 
b/meta/recipes-core/glibc/glibc-testing.inc
index ec16fe1..204c926 100644
--- a/meta/recipes-core/glibc/glibc-testing.inc
+++ b/meta/recipes-core/glibc/glibc-testing.inc
@@ -68,6 +68,25 @@ fi
 
fgrep tcbootstrap ${B}/config.make > ${B}/configparms
sed -i -e "s/\-tcbootstrap//g" ${B}/configparms
+
+# g++ uses flag -nostdinc, so the locations of system include headers must be 
explicitly specified.
+# If the locations are not already specified in config.make, then we provide 
the following locations:
+#/usr/include/c++/
+#/usr/include/c++//
+
+cxxincludes=\`cat ${B}/config.make | gawk '\$1 == "c++-sysincludes"' | gawk 
-F"=" '{print \$2}' | sed "s/[ \t]\?//g"\`
+
+if [ -z "\$cxxincludes" ]; then
+  sysroot=\`cat ${B}/configparms | sed -n "/CXX/p" | sed -e 
"s/^.*--sysroot=//"\`
+  cxx=\`cat ${B}/configparms | gawk '\$1 ~ /^CXX/' | gawk -F"=" '{print \$2}' 
| gawk '{print \$1}'\`
+  cxxmachine=\`\$cxx -dumpmachine\`
+  cxxversion=\`\$cxx -dumpversion\`
+  if [ -d "\$sysroot/usr/include/c++/\$cxxversion" ] && [ -d 
"\$sysroot/usr/include/c++/\$cxxversion/\$cxxmachine" ]; then
+# if the folders exist, pass the new value of c++-sysincludes via 
configparms
+echo "c++-sysincludes = -isystem \$sysroot/usr/include/c++/\$cxxversion 
-isystem \$sysroot/usr/include/c++/\$cxxversion/\$cxxmachine" >> 
${B}/configparms
+  fi
+fi
+
 wrapper="${S}/scripts/cross-test-ssh.sh \$target"
 localedef="${STAGING_BINDIR_NATIVE}/cross-localedef --little-endian 
--uint32-align=4"
 make tests-clean
-- 
1.9.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 3/3] xorg-font-common: fix sysroot injection for encoding maps

[Richard Tollerton]

The xorg font builds use `pkg-config --variable=mapdir fontutil` to
locate encoding maps. This variable ought to be sysroot-relative, but
neither pkg-config nor font-util nor the fonts themselves provide any
facility to add the sysroot back in.

We're presently adding the sysroot by by twiddling MAPFILES_PATH in
configure.ac. This is broken; it's actually defined in aclocal.m4,
because the definition is provided by fontutil.m4. Another (more
speculative) criticism is that it also hardcodes a build-specific
absolute path into builds which might (incorrectly) encode it into
target-installable packages.

A somewhat more robust, focused, and clear solution is to override
UTIL_DIR on the make command line. (UTIL_DIR, not MAPFILES_PATH, is what
is actually referenced in the build.)

Signed-off-by: Richard Tollerton 
---
 meta/recipes-graphics/xorg-font/xorg-font-common.inc | 8 +---
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/meta/recipes-graphics/xorg-font/xorg-font-common.inc 
b/meta/recipes-graphics/xorg-font/xorg-font-common.inc
index 55adde3..d3b995a 100644
--- a/meta/recipes-graphics/xorg-font/xorg-font-common.inc
+++ b/meta/recipes-graphics/xorg-font/xorg-font-common.inc
@@ -15,13 +15,7 @@ S = "${WORKDIR}/${XORG_PN}-${PV}"
 
 inherit autotools pkgconfig
 
-EXTRA_OEMAKE += "FCCACHE=/bin/true"
-
-do_configure_prepend() {
-if [ -f "${S}"/configure.ac ] ; then
-sed -i 
"s#^MAPFILES_PATH=.*#MAPFILES_PATH=\"${STAGING_DIR_TARGET}/\$(pkg-config 
--variable=mapdir fontutil)\"#g" "${S}"/configure.ac
-fi
-}
+EXTRA_OEMAKE += "FCCACHE=/bin/true 
UTIL_DIR=${STAGING_DIR_TARGET}\$\(MAPFILES_PATH\)"
 
 do_install_append() {
find ${D}${libdir}/X11/fonts -type f -name fonts.dir | xargs rm -f
-- 
2.1.3

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 0/3] xorg-font: stop trying to use build machine's encoding maps

[Richard Tollerton]

Several xorg-font builds were added to meta-oe recently. These builds tend to
fail on several Linux distros because they referenced encoding maps (e.g.
map-ISO8859-1) at native paths outside the sysroot, which don't exist; the build
expects /usr/share/fonts/X11/util/map-ISO8859-1 to exist, but the distro
actually installed it at /usr/share/fonts/util/map-ISO8859-1. Of course, the
correct behavior is for the build to reference the appropriate path under the
appropriate sysroot.

There are two different root causes. The first and by far the most important
cause is that we're explicitly blacklisting ${datadir}/fonts from staging, so it
was never possible for these maps to be sourced from sysroots in the first
place. This seems like a silly thing to do, and I could not find any
justification for it in the commit history, so I propose getting rid of it.

Once that was out of the way, I cleaned up some existing chicanery involving
where to look for the maps. In particular, presently font-util is hardcoding the
path to point at the native sysroot, for *all* architectures. Again, this seems
silly, I can't find any justification for it, and it breaks builds; I propose
getting rid of it.

These changes will require the font recipes in meta-oe need to be updated to
explicitly depend on font-util in addition to font-util-native; I'll post those
patches separately. (Yes, these font builds are arch-specific right now; making
them allarch is something I'm also working on.)

Richard Tollerton (3):
  staging.bbclass: Install ${datadir}/fonts/ into sysroots
  font-util: remove MAPFILES_PATH override
  xorg-font-common: fix sysroot injection for encoding maps

 meta/classes/staging.bbclass | 2 +-
 meta/recipes-graphics/xorg-font/font-util_1.3.0.bb   | 4 
 meta/recipes-graphics/xorg-font/xorg-font-common.inc | 8 +---
 3 files changed, 2 insertions(+), 12 deletions(-)

-- 
2.1.3

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 1/3] staging.bbclass: Install ${datadir}/fonts/ into sysroots

[Richard Tollerton]

font-util installs encoding maps into /usr/share/fonts/X11. This path
may not exist on the host (many distros install these into
/usr/share/fonts/util) so they must exist in the sysroot in order for
e.g. `pkg-config --variable=mapdir fontutil` to work correctly. However,
currently /usr/share/fonts is explicitly not installed into sysroots, so
the encoding maps were not getting found.

Fix this by no longer deleting ${datadir}/fonts/ from sysroots.

Signed-off-by: Richard Tollerton 
---
 meta/classes/staging.bbclass | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes/staging.bbclass b/meta/classes/staging.bbclass
index 57b2743..c4a2cb1 100644
--- a/meta/classes/staging.bbclass
+++ b/meta/classes/staging.bbclass
@@ -50,7 +50,7 @@ sysroot_stage_dirs() {
sysroot_stage_dir $from${datadir} $to${datadir}
# We don't care about docs/info/manpages/locales
rm -rf $to${mandir}/ $to${docdir}/ $to${infodir}/ 
${to}${datadir}/locale/
-   rm -rf $to${datadir}/applications/ $to${datadir}/fonts/ 
$to${datadir}/pixmaps/
+   rm -rf $to${datadir}/applications/ $to${datadir}/pixmaps/
 }
 
 sysroot_stage_all() {
-- 
2.1.3

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 2/3] font-util: remove MAPFILES_PATH override

[Richard Tollerton]

MAPFILES_PATH is a configuration variable that is used by packages using
font-util, which tells those packages how to compute it in fontutil.m4.
Presently, we're manually twiddling things so that MAPFILES_PATH will
consult the native sysroot, when building under *any* architecture.
This complicates building other packages immensely, and also generates
broken on-target packages, because the contents of font-util-dev will
reference the native sysroot on the build machine (!).

We don't even need to twiddle MAPFILES_PATH anymore so just delete it.

This code also had a path bug (referencing /usr/lib/pkg-config instead
of /usr/lib/pkgconfig) which is also fixed by deletion.

Signed-off-by: Richard Tollerton 
---
 meta/recipes-graphics/xorg-font/font-util_1.3.0.bb | 4 
 1 file changed, 4 deletions(-)

diff --git a/meta/recipes-graphics/xorg-font/font-util_1.3.0.bb 
b/meta/recipes-graphics/xorg-font/font-util_1.3.0.bb
index 8b42991..a1b4628 100644
--- a/meta/recipes-graphics/xorg-font/font-util_1.3.0.bb
+++ b/meta/recipes-graphics/xorg-font/font-util_1.3.0.bb
@@ -16,10 +16,6 @@ RDEPENDS_${PN}_class-native = "mkfontdir-native 
mkfontscale-native"
 
 PR = "${INC_PR}.0"
 
-do_configure_prepend() {
-sed -i 
"s#MAPFILES_PATH=\`pkg-config#MAPFILES_PATH=\`PKG_CONFIG_PATH=\"${STAGING_LIBDIR_NATIVE}/pkg-config\"
 pkg-config#g" ${S}/fontutil.m4.in
-}
-
 BBCLASSEXTEND = "native"
 
 SRC_URI[md5sum] = "ddfc8a89d597651408369d940d03d06b"
-- 
2.1.3

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH] font-util: Fix incorrect PKG_CONFIG_PATH

[Richard Tollerton]

Richard Tollerton  writes:

> PKG_CONFIG_PATH always defaults to /usr/lib/pkgconfig, and the host
> /usr/lib/pkgconfig is always checked as a fallback; however,
> PKG_CONFIG_PATH is currently (incorrectly) set to /usr/lib/pkg-config in
> the sysroot, which doesn't exist. On host distros where the font
> encoding maps are stored under a different path than OE, this will break
> font builds, because ucs2any will attempt to read the sysroot's encoding
> maps with the host paths.
>
> Signed-off-by: Richard Tollerton 
> ---
>  meta/recipes-graphics/xorg-font/font-util_1.3.0.bb | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

I'm withdrawing this patch, because I found a better solution to the
problems I've encountered which completely obviate this approach. A
superseding patchset will be sent out shortly.


-- 
Richard Tollerton 
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices

[Gary Thomas]

On 2014-12-17 15:03, Burton, Ross wrote:

Hi Mike,

On 5 December 2014 at 09:58, Mike Looijmans mailto:mike.looijm...@topic.nl>> wrote:

Upon inserting a USB stick or similar device, mdev will run
an automounter script that mounts valid partitions on
/media/. The script first checks /etc/fstab entries
so that mounting on UUID or LABEL or using custom mount options
is still possible. If /etc/fstab does not contain particular
mount options, the script will create (and remove) the mountpoint
automatically.
The script also supports full disk partitions (devices without
partition table).


Thanks for this, but can this script be put into a separate recipe to make it 
even easier for people who don't want it, or want their own version?  The prior 
art here is the udev
automounting script which is in udev-extraconf.


Also it would be nice if the path matched the most recent
use in udev which is /run/media/

--

Gary Thomas |  Consulting for the
MLB Associates  |Embedded world

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices

[Burton, Ross]

On 17 December 2014 at 22:03, Burton, Ross  wrote:
>
> Thanks for this, but can this script be put into a separate recipe to make
> it even easier for people who don't want it, or want their own version?
> The prior art here is the udev automounting script which is in
> udev-extraconf.


That embarrassing moment when you realise that Otavio said the same thing
twelve days earlier. :)
Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices

[Burton, Ross]

Hi Mike,

On 5 December 2014 at 09:58, Mike Looijmans  wrote:
>
> Upon inserting a USB stick or similar device, mdev will run
> an automounter script that mounts valid partitions on
> /media/. The script first checks /etc/fstab entries
> so that mounting on UUID or LABEL or using custom mount options
> is still possible. If /etc/fstab does not contain particular
> mount options, the script will create (and remove) the mountpoint
> automatically.
> The script also supports full disk partitions (devices without
> partition table).
>

Thanks for this, but can this script be put into a separate recipe to make
it even easier for people who don't want it, or want their own version?
The prior art here is the udev automounting script which is in
udev-extraconf.

Cheers,
Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/2] busybox-mdev: Support automatic mounting of block devices

[Burton, Ross]

On 5 December 2014 at 09:58, Mike Looijmans  wrote:
>
> The automatic mounting can be disabled by adding a line
> MDEV_AUTOMOUNT=y
> into /etc/default/mdev
>

Presumably you meant =n there, right?

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH] vala: Add Upstream-Status to patch

[Saul Wold]

Signed-off-by: Saul Wold 
---
 .../0001-git-version-gen-don-t-append-dirty-if-we-re-not-in-g.patch | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/meta/recipes-devtools/vala/vala/0001-git-version-gen-don-t-append-dirty-if-we-re-not-in-g.patch
 
b/meta/recipes-devtools/vala/vala/0001-git-version-gen-don-t-append-dirty-if-we-re-not-in-g.patch
index e6c7941..ac9dc8c 100644
--- 
a/meta/recipes-devtools/vala/vala/0001-git-version-gen-don-t-append-dirty-if-we-re-not-in-g.patch
+++ 
b/meta/recipes-devtools/vala/vala/0001-git-version-gen-don-t-append-dirty-if-we-re-not-in-g.patch
@@ -7,6 +7,8 @@ Subject: [PATCH] git-version-gen: don't append -dirty if we're 
not in git
 * for example if we have some dirty directory and we unpack clean vala tarball 
in it, then it will append -dirty
 
 Signed-off-by: Martin Jansa 
+
+Upstream-Status: Inappropriate [OE-Specific]
 ---
  build-aux/git-version-gen |   25 ++---
  1 files changed, 14 insertions(+), 11 deletions(-)
-- 
1.8.3.1

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 1/2] which: add missing Upstream-Status

[Ross Burton]

Signed-off-by: Ross Burton 
---
 meta/recipes-extended/which/which-2.20/automake.patch |5 +
 1 file changed, 5 insertions(+)

diff --git a/meta/recipes-extended/which/which-2.20/automake.patch 
b/meta/recipes-extended/which/which-2.20/automake.patch
index 92365dd..ef00bc9 100644
--- a/meta/recipes-extended/which/which-2.20/automake.patch
+++ b/meta/recipes-extended/which/which-2.20/automake.patch
@@ -1,3 +1,8 @@
+Update autoconf prologue to use "foreign" strictness.
+
+Upstream-Status: Pending
+Signed-off-by: Ross Burton 
+
 diff --git a/configure.ac b/configure.ac
 index d974461..a20dfa8 100644
 --- a/configure.ac
-- 
1.7.10.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [PATCH 2/2] oprofile: add missing Upstream-Status

[Ross Burton]

Signed-off-by: Ross Burton 
---
 meta/recipes-kernel/oprofile/oprofile/automake-foreign.patch |5 +
 1 file changed, 5 insertions(+)

diff --git a/meta/recipes-kernel/oprofile/oprofile/automake-foreign.patch 
b/meta/recipes-kernel/oprofile/oprofile/automake-foreign.patch
index e82a381..b9bb6c5 100644
--- a/meta/recipes-kernel/oprofile/oprofile/automake-foreign.patch
+++ b/meta/recipes-kernel/oprofile/oprofile/automake-foreign.patch
@@ -1,3 +1,8 @@
+oprofile doesn't want GNU-levels of automake strictness so tell it to be 
"foreign".
+
+Upstream-Status: Pending
+Signed-off-by: Ross Burton 
+
 diff --git a/configure.ac b/configure.ac
 index 5740585..cf6c316 100644
 --- a/configure.ac
-- 
1.7.10.4

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

[OE-core] [dizzy][PATCH] Fixed openssh ptest

[Liviu Gheorghisan]

ptests for openssh, apart from a lot of .sh scripts, also
require 2 executables to be cross-compiled for the target
and deployed there, that will be called from the test
scripts.

Those 2 files were not built when cross-compiling
(Makefile target all:), so I added them to the build,
under the target all_test.

This new make target is intended to be run from
do_compile_ptest(), that builds everything needed for
performing the ptests.

Because both these 2 executables required for ptest
are linked with libssh (LDFLAGS += -lssh), I also
added it as dependency for each of them.

Now we have 2 separate make targets:
all: builds all openssh, except for the ptest related files,
that is invoked from do_compile().

all_test: builds only the files required for ptests, that is
invoked from do_compile_ptest().

All this stuff fixes the following openssh ptest error:

/bin/sh: /usr/lib/openssh/ptest/regress/modpipe: No such file or directory
unexpected error mac chacha20-poly1...@openssh.com at 2909
test integrity: 10 errors: mac 0 padding 0 length 0
chacha20-poly1...@openssh.com: no mac errors
chacha20-poly1...@openssh.com: expected 10 mac errors, got 0
FAIL:  integrity
make: *** [t-exec] Error 1
Makefile:168: recipe for target 't-exec' failed
make: Target 'tests' not remade because of errors.

Signed-off-by: Liviu Gheorghisan 
---
 .../openssh/openssh-ptest-fix-sshconnect.patch | 57 ++
 meta/recipes-connectivity/openssh/openssh_6.6p1.bb |  7 ++-
 2 files changed, 63 insertions(+), 1 deletion(-)
 create mode 100644 
meta/recipes-connectivity/openssh/openssh/openssh-ptest-fix-sshconnect.patch

diff --git 
a/meta/recipes-connectivity/openssh/openssh/openssh-ptest-fix-sshconnect.patch 
b/meta/recipes-connectivity/openssh/openssh/openssh-ptest-fix-sshconnect.patch
new file mode 100644
index 000..0913701
--- /dev/null
+++ 
b/meta/recipes-connectivity/openssh/openssh/openssh-ptest-fix-sshconnect.patch
@@ -0,0 +1,57 @@
+Fixed openssh ptest
+
+ptests for openssh, apart from a lot of .sh scripts, also
+require 2 executables to be cross-compiled for the target
+and deployed there, that will be called from the test
+scripts.
+
+Those 2 files were not built when cross-compiling
+(Makefile target all:), so I added them to the build,
+under the target all_test.
+
+This new make target is intended to be run from
+do_compile_ptest(), that builds everything needed for
+performing the ptests.
+
+Because both these 2 executables required for ptest
+are linked with libssh (LDFLAGS += -lssh), I also
+added it as dependency for each of them.
+
+Upstream-status: Inappropriate
+
+Signed-off-by: Liviu Gheorghisan 
+---
+--- a/Makefile.in  2014-12-10 16:19:37.513373454 +0100
 b/Makefile.in  2014-12-10 16:25:23.262764227 +0100
+@@ -130,6 +130,8 @@ FIXALGORITHMSCMD= $(SHELL) $(srcdir)/fix
+ 
+ all: $(CONFIGFILES) $(MANPAGES) $(TARGETS)
+ 
++all_test: regress/modpipe$(EXEEXT) regress/setuid-allowed$(EXEEXT)
++
+ $(LIBSSH_OBJS): Makefile.in config.h
+ $(SSHOBJS): Makefile.in config.h
+ $(SSHDOBJS): Makefile.in config.h
+@@ -394,18 +396,18 @@ uninstall:
+   -rm -f $(DESTDIR)$(mandir)/$(mansubdir)8/ssh-pkcs11-helper.8
+   -rm -f $(DESTDIR)$(mandir)/$(mansubdir)1/slogin.1
+ 
+-regress/modpipe$(EXEEXT): $(srcdir)/regress/modpipe.c
++regress/modpipe$(EXEEXT): libssh.a
+   [ -d `pwd`/regress ]  ||  mkdir -p `pwd`/regress
+   [ -f `pwd`/regress/Makefile ]  || \
+   ln -s `cd $(srcdir) && pwd`/regress/Makefile `pwd`/regress/Makefile
+-  $(CC) $(CFLAGS) $(CPPFLAGS) -o $@ $? \
++  $(CC) $(CFLAGS) $(CPPFLAGS) -o $@ $(srcdir)/regress/modpipe.c \
+   $(LDFLAGS) -lssh -lopenbsd-compat -lssh -lopenbsd-compat $(LIBS)
+ 
+-regress/setuid-allowed$(EXEEXT): $(srcdir)/regress/setuid-allowed.c
++regress/setuid-allowed$(EXEEXT): libssh.a
+   [ -d `pwd`/regress ]  ||  mkdir -p `pwd`/regress
+   [ -f `pwd`/regress/Makefile ]  || \
+   ln -s `cd $(srcdir) && pwd`/regress/Makefile `pwd`/regress/Makefile
+-  $(CC) $(CFLAGS) $(CPPFLAGS) -o $@ $? \
++  $(CC) $(CFLAGS) $(CPPFLAGS) -o $@ $(srcdir)/regress/setuid-allowed.c \
+   $(LDFLAGS) -lssh -lopenbsd-compat -lssh -lopenbsd-compat $(LIBS)
+ 
+ tests interop-tests:  $(TARGETS) regress/modpipe$(EXEEXT) 
regress/setuid-allowed$(EXEEXT)
diff --git a/meta/recipes-connectivity/openssh/openssh_6.6p1.bb 
b/meta/recipes-connectivity/openssh/openssh_6.6p1.bb
index 047a895..fe9ee02 100644
--- a/meta/recipes-connectivity/openssh/openssh_6.6p1.bb
+++ b/meta/recipes-connectivity/openssh/openssh_6.6p1.bb
@@ -25,7 +25,8 @@ SRC_URI = 
"ftp://ftp.openbsd.org/pub/OpenBSD/OpenSSH/portable/openssh-${PV}.tar.
file://run-ptest \
file://openssh-CVE-2014-2532.patch \
file://openssh-CVE-2014-2653.patch \
-   file://auth2-none.c-avoid-authenticate-empty-passwords-to-m.patch"
+   file://auth2-none.c-avoid-authenticate-empty-passwords-to-m.patch \
+   file://openssh-ptest-fix-sshconne

Re: [OE-core] [PATCH 13/24] libunwind: add aarch64 support

[Kang Kai]

On 2014年12月17日 17:23, Fathi Boudra wrote:

On 17 December 2014 at 10:53, Kang Kai  wrote:

On 2014年12月09日 04:56, Burton, Ross wrote:

On 7 November 2014 at 08:51, Kai Kang  wrote:


- Support-building-with-older-compilers (fix undefined reference to
  `unreachable' on older compilers)


This patch is missing a signed-off-by header, and considering that oe-core
has gcc 4.9 do we still need this, or is it specific to a commercial
product that is using an older toolchain?

we still need this.


It is ok to drop this patch for gcc 4.9. I'll do it.

no, it isn't ok to drop this patch. aarch64 support is still needed in
libunwind.


I mean Support-building-with-older-compilers.patch. Do we still need it?

--Kai




--Kai


Ross



--
Regards,
Neil | Kai Kang





--
Regards,
Neil | Kai Kang

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 0/4] Import Python Modules for Toaster

[Burton, Ross]

On 10 December 2014 at 05:44, Saul Wold  wrote:
>
> These 4 python modules are needed by toaster, since we want to have a
> complete environment, we need add these python modules to oe-core so
> that toaster can run.
>

I'm not very comfortable with pulling pieces from meta-python or
meta-openstack and merging them into oe-core just for running Toaster on
Poky.  I think a better solution would be to ensure that meta-python has
everything required and then if we want to continue with the one-repo model
for Poky use combo-layer to copy the relevant recipes from meta-python into
poky.

Ross
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 09/24] libpng: add neon option for aarch64

[Burton, Ross]

I find Koen's patch for libpng neater so could you merge that into your
qemuarm64 branch?  The autobuilder has been doing some builds so this
should be the last stage before it's all merged.

Ross

On 17 December 2014 at 08:47, Kang Kai  wrote:
>
>  On 2014年12月17日 03:30, Koen Kooi wrote:
>
>  Op 7 nov. 2014, om 09:51 heeft Kai Kang  
>  het volgende geschreven:
>
> Add neon option for aarch64 just as for arm. It fails with:
>
> | ./.libs/libpng16.so: undefined reference to `png_init_filter_functions_neon'
> | collect2: error: ld returned 1 exit status
>
> Signed-off-by: Kai Kang  
> ---
> meta/recipes-multimedia/libpng/libpng_1.6.13.bb | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/meta/recipes-multimedia/libpng/libpng_1.6.13.bb 
> b/meta/recipes-multimedia/libpng/libpng_1.6.13.bb
> index 0c6fd1f..3d32bfe 100644
> --- a/meta/recipes-multimedia/libpng/libpng_1.6.13.bb
> +++ b/meta/recipes-multimedia/libpng/libpng_1.6.13.bb
> @@ -20,6 +20,7 @@ inherit autotools binconfig-disabled pkgconfig
>
> # Work around missing symbols
> EXTRA_OECONF_append_arm = " ${@bb.utils.contains("TUNE_FEATURES", "neon", 
> "--enable-arm-neon=on", "--enable-arm-neon=off" ,d)}"
> +EXTRA_OECONF_append_aarch64 = " ${@bb.utils.contains("TUNE_FEATURES", 
> "neon", "--enable-arm-neon=on", "--enable-arm-neon=off" ,d)}"
>
>  You're introducing the same bug in a different way: overeager use of 
> qualifiers. Just drop the _arm bit and it works.
>
>
>
> OK. I'll remove the _arm to fix this issue.
>
> Thanks.
>
> --
> Regards,
> Neil | Kai Kang
>
>
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/1] dhcp: fix for systemd service file

[Enrico Scholz]

ChenQi  writes:

>> 1. Set Type=forking for dhcpd.service to make it start up correctly.
>> 2. Add PIDFile=/var/run/dhcpd.pid

why forking? Starting dhcpd with '-f --no-pid' should chose the more
simple non-forking variant without a clumsy.pidfile.


Enrico
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 1/1] dhcp: fix for systemd service file

[ChenQi]

ping

On 09/28/2014 04:59 PM, Chen Qi wrote:

1. Set Type=forking for dhcpd.service to make it start up correctly.
2. Add PIDFile=/var/run/dhcpd.pid
3. Do not force existence of the environment file.

Signed-off-by: Chen Qi 
---
  meta/recipes-connectivity/dhcp/files/dhcpd.service | 4 +++-
  1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/dhcp/files/dhcpd.service 
b/meta/recipes-connectivity/dhcp/files/dhcpd.service
index 8648f1a..4215173 100644
--- a/meta/recipes-connectivity/dhcp/files/dhcpd.service
+++ b/meta/recipes-connectivity/dhcp/files/dhcpd.service
@@ -4,7 +4,9 @@ After=network.target
  After=time-sync.target
  
  [Service]

-EnvironmentFile=@SYSCONFDIR@/default/dhcp-server
+Type=forking
+PIDFile=/var/run/dhcpd.pid
+EnvironmentFile=-@SYSCONFDIR@/default/dhcp-server
  ExecStart=-@SBINDIR@/dhcpd -q $INTERFACES
  
  [Install]


--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 13/24] libunwind: add aarch64 support

[Fathi Boudra]

On 17 December 2014 at 10:53, Kang Kai  wrote:
> On 2014年12月09日 04:56, Burton, Ross wrote:
>>
>> On 7 November 2014 at 08:51, Kai Kang  wrote:
>>
>>>- Support-building-with-older-compilers (fix undefined reference to
>>>  `unreachable' on older compilers)
>>>
>> This patch is missing a signed-off-by header, and considering that oe-core
>> has gcc 4.9 do we still need this, or is it specific to a commercial
>> product that is using an older toolchain?

we still need this.

>
> It is ok to drop this patch for gcc 4.9. I'll do it.

no, it isn't ok to drop this patch. aarch64 support is still needed in
libunwind.

> --Kai
>
>>
>> Ross
>>
>
>
> --
> Regards,
> Neil | Kai Kang
-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 13/24] libunwind: add aarch64 support

[Kang Kai]

On 2014年12月09日 04:56, Burton, Ross wrote:

On 7 November 2014 at 08:51, Kai Kang  wrote:


   - Support-building-with-older-compilers (fix undefined reference to
 `unreachable' on older compilers)


This patch is missing a signed-off-by header, and considering that oe-core
has gcc 4.9 do we still need this, or is it specific to a commercial
product that is using an older toolchain?


It is ok to drop this patch for gcc 4.9. I'll do it.

--Kai



Ross




--
Regards,
Neil | Kai Kang

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 16/24] packagegroup-core-tools-profile: remove sysprof for aarch64

[Kang Kai]

On 2014年12月10日 20:27, Burton, Ross wrote:

On 10 December 2014 at 01:39, Kang Kai  wrote:


I think I misunderstood you. Do you mean update like this:

SYSPROF = "${@bb.utils.contains('DISTRO_FEATURES', 'x11', 'sysprof', '',
d)}"
SYSPROF_aarch64 = ""

RRECOMMENDS_${PN} = "\
   ... \
  ${SYSPROF} \

"

I'll update it.


Yep, that's it.


I didn't find the usage of SYSPROF in lttng recipes. Neither  in this 
serial commits nor the oe-core master branch.

Where did you find it?

Thanks,
Kai




Ross




--
Regards,
Neil | Kai Kang

--
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Re: [OE-core] [PATCH 09/24] libpng: add neon option for aarch64

[Kang Kai]

On 2014年12月17日 03:30, Koen Kooi wrote:

Op 7 nov. 2014, om 09:51 heeft Kai Kang  het volgende 
geschreven:

Add neon option for aarch64 just as for arm. It fails with:

| ./.libs/libpng16.so: undefined reference to `png_init_filter_functions_neon'
| collect2: error: ld returned 1 exit status

Signed-off-by: Kai Kang 
---
meta/recipes-multimedia/libpng/libpng_1.6.13.bb | 1 +
1 file changed, 1 insertion(+)

diff --git a/meta/recipes-multimedia/libpng/libpng_1.6.13.bb 
b/meta/recipes-multimedia/libpng/libpng_1.6.13.bb
index 0c6fd1f..3d32bfe 100644
--- a/meta/recipes-multimedia/libpng/libpng_1.6.13.bb
+++ b/meta/recipes-multimedia/libpng/libpng_1.6.13.bb
@@ -20,6 +20,7 @@ inherit autotools binconfig-disabled pkgconfig

# Work around missing symbols
EXTRA_OECONF_append_arm = " ${@bb.utils.contains("TUNE_FEATURES", "neon", 
"--enable-arm-neon=on", "--enable-arm-neon=off" ,d)}"
+EXTRA_OECONF_append_aarch64 = " ${@bb.utils.contains("TUNE_FEATURES", "neon", 
"--enable-arm-neon=on", "--enable-arm-neon=off" ,d)}"

You're introducing the same bug in a different way: overeager use of 
qualifiers. Just drop the _arm bit and it works.



OK. I'll remove the _arm to fix this issue.

Thanks.

--
Regards,
Neil | Kai Kang

-- 
___
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core