[PATCH] connman: fix build with automake 1.12

[PATCH] connman: fix build with automake 1.12

[Ross Burton]

Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 meta/recipes-connectivity/connman/connman.inc |    6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-connectivity/connman/connman.inc b/meta/recipes-connectivity/connman/connman.inc
index 12378e9..bb6f532 100644
--- a/meta/recipes-connectivity/connman/connman.inc
+++ b/meta/recipes-connectivity/connman/connman.inc
@@ -20,7 +20,7 @@ DEPENDS  = "dbus glib-2.0 ppp iptables gnutls \
             ${@base_contains('DISTRO_FEATURES', '3g','ofono', '', d)} \
             "
 
-INC_PR = "r11"
+INC_PR = "r12"
 
 TIST = "--enable-tist"
 TIST_powerpc = ""
@@ -72,6 +72,10 @@ do_install_append() {
 
 	# We don't need to package an empty directory
 	rmdir ${D}${libdir}/connman/scripts
+
+	# Automake 1.12 won't install empty directories, but we need the
+	# plugins directory to be present for ownership
+	mkdir -p ${D}${libdir}/connman/plugins
 }
 
 # These used to be plugins, but now they are core
-- 
1.7.10

Re: [PATCH] connman: fix build with automake 1.12

[Colin Walters]

On Wed, 2012-07-11 at 16:39 +0100, Ross Burton wrote:

> +	# Automake 1.12 won't install empty directories, but we need the
> +	# plugins directory to be present for ownership
> +	mkdir -p ${D}${libdir}/connman/plugins
>  }

Did you consider patching upstream to do this in install-data-local
rule?  That way everyone gets the fix (Debian/Gentoo/Fedora etc.), not
just you.

Re: [PATCH] connman: fix build with automake 1.12

[Burton, Ross]

On 11 July 2012 23:00, Colin Walters <walters@verbum.org> wrote:
> On Wed, 2012-07-11 at 16:39 +0100, Ross Burton wrote:
>
>> +     # Automake 1.12 won't install empty directories, but we need the
>> +     # plugins directory to be present for ownership
>> +     mkdir -p ${D}${libdir}/connman/plugins
>>  }
>
> Did you consider patching upstream to do this in install-data-local
> rule?  That way everyone gets the fix (Debian/Gentoo/Fedora etc.), not
> just you.

Yes, and I also considered that this surely can't be restricted to
connman and automake should have some clean way of forcing an empty
directory to exist.  I'm also pretty certain the automake maintainers
response would be "do it in install-data-local", so this should go to
connman...

Ross

Re: [PATCH] connman: fix build with automake 1.12

[Saul Wold]

On 07/11/2012 08:39 AM, Ross Burton wrote:
> Signed-off-by: Ross Burton <ross.burton@intel.com>
> ---
>   meta/recipes-connectivity/connman/connman.inc |    6 +++++-
>   1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/meta/recipes-connectivity/connman/connman.inc b/meta/recipes-connectivity/connman/connman.inc
> index 12378e9..bb6f532 100644
> --- a/meta/recipes-connectivity/connman/connman.inc
> +++ b/meta/recipes-connectivity/connman/connman.inc
> @@ -20,7 +20,7 @@ DEPENDS  = "dbus glib-2.0 ppp iptables gnutls \
>               ${@base_contains('DISTRO_FEATURES', '3g','ofono', '', d)} \
>               "
>
> -INC_PR = "r11"
> +INC_PR = "r12"
>
>   TIST = "--enable-tist"
>   TIST_powerpc = ""
> @@ -72,6 +72,10 @@ do_install_append() {
>
>   	# We don't need to package an empty directory
>   	rmdir ${D}${libdir}/connman/scripts
> +
> +	# Automake 1.12 won't install empty directories, but we need the
> +	# plugins directory to be present for ownership
> +	mkdir -p ${D}${libdir}/connman/plugins
>   }
>
>   # These used to be plugins, but now they are core
>


Merged this change into OE-Core, other connman changes are pending with 
some clean up requests.

Thanks
	Sau!