Openembedded Core Discussions
 help / color / mirror / Atom feed
* [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC
@ 2015-04-21 10:53 Martin Jansa
  2015-04-21 10:53 ` [dizzy][PATCH 2/3] postinst-intercepts/update_font_cache: fix ownership of fontconfig cache Martin Jansa
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Martin Jansa @ 2015-04-21 10:53 UTC (permalink / raw)
  To: openembedded-core

* I don't see any usage for libpulse-* packages
* adding '-' resolves the issue when we have separate recipe for
  pulseaudio-modules-droid which isn't built to satisfy RDEPENDS
  with the same name, because generic pulseaudio recipe seems to
  RPROVIDE it through PACKAGES_DYNAMIC

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
index db144a9..99cad76 100644
--- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
+++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
@@ -79,7 +79,7 @@ PACKAGES =+ "libpulsecore libpulsecommon libpulse libpulse-simple libpulse-mainl
 #upgrade path:
 RREPLACES_pulseaudio-server = "libpulse-bin libpulse-conf"
 
-PACKAGES_DYNAMIC += "^pulseaudio-lib.* ^pulseaudio-module.* ^libpulse-lib.* ^libpulse-module.* "
+PACKAGES_DYNAMIC += "^pulseaudio-lib-.* ^pulseaudio-module-.*"
 
 FILES_libpulsecore = "${libdir}/libpulsecore*.so"
 FILES_libpulsecommon = "${libdir}/pulseaudio/libpulsecommon*.so"
-- 
2.3.5



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

* [dizzy][PATCH 2/3] postinst-intercepts/update_font_cache: fix ownership of fontconfig cache
  2015-04-21 10:53 [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC Martin Jansa
@ 2015-04-21 10:53 ` Martin Jansa
  2015-04-21 10:53 ` [dizzy][PATCH 3/3] fontcache: allow to pass different fontconfig cache dir Martin Jansa
  2015-04-27 11:04 ` [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC Martin Jansa
  2 siblings, 0 replies; 5+ messages in thread
From: Martin Jansa @ 2015-04-21 10:53 UTC (permalink / raw)
  To: openembedded-core

From: Jonathan Liu <net147@gmail.com>

The file ownership of the cache files in /var/cache/fontconfig needs to
be set to root:root otherwise it inherits the user and group id of the
build user.

[YOCTO #7411]

Signed-off-by: Jonathan Liu <net147@gmail.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
---
 meta/classes/fontcache.bbclass                | 2 +-
 scripts/postinst-intercepts/update_font_cache | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/meta/classes/fontcache.bbclass b/meta/classes/fontcache.bbclass
index dcd1f04..dfbdfa1 100644
--- a/meta/classes/fontcache.bbclass
+++ b/meta/classes/fontcache.bbclass
@@ -12,7 +12,7 @@ FONT_EXTRA_RDEPENDS ?= "fontconfig-utils"
 fontcache_common() {
 if [ "x$D" != "x" ] ; then
 	$INTERCEPT_DIR/postinst_intercept update_font_cache ${PKG} mlprefix=${MLPREFIX} bindir=${bindir} \
-		libdir=${libdir} base_libdir=${base_libdir}
+		libdir=${libdir} base_libdir=${base_libdir} localstatedir=${localstatedir}
 else
 	fc-cache
 fi
diff --git a/scripts/postinst-intercepts/update_font_cache b/scripts/postinst-intercepts/update_font_cache
index 78f3365..3907f25 100644
--- a/scripts/postinst-intercepts/update_font_cache
+++ b/scripts/postinst-intercepts/update_font_cache
@@ -2,5 +2,6 @@
 
 PSEUDO_UNLOAD=1 qemuwrapper -L $D -E LD_LIBRARY_PATH=$D/${libdir}:$D/${base_libdir}\
 					$D${bindir}/fc-cache --sysroot=$D
+chown -R root:root $D${localstatedir}/cache/fontconfig
 
 
-- 
2.3.5



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

* [dizzy][PATCH 3/3] fontcache: allow to pass different fontconfig cache dir
  2015-04-21 10:53 [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC Martin Jansa
  2015-04-21 10:53 ` [dizzy][PATCH 2/3] postinst-intercepts/update_font_cache: fix ownership of fontconfig cache Martin Jansa
@ 2015-04-21 10:53 ` Martin Jansa
  2015-04-27 11:04 ` [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC Martin Jansa
  2 siblings, 0 replies; 5+ messages in thread
From: Martin Jansa @ 2015-04-21 10:53 UTC (permalink / raw)
  To: openembedded-core

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
---
 meta/classes/fontcache.bbclass                        | 4 ++--
 meta/recipes-graphics/fontconfig/fontconfig_2.11.1.bb | 4 +++-
 scripts/postinst-intercepts/update_font_cache         | 4 +---
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/meta/classes/fontcache.bbclass b/meta/classes/fontcache.bbclass
index dfbdfa1..d122387 100644
--- a/meta/classes/fontcache.bbclass
+++ b/meta/classes/fontcache.bbclass
@@ -8,11 +8,11 @@ inherit qemu
 
 FONT_PACKAGES ??= "${PN}"
 FONT_EXTRA_RDEPENDS ?= "fontconfig-utils"
-
+FONTCONFIG_CACHE_DIR ?= "${localstatedir}/cache/fontconfig"
 fontcache_common() {
 if [ "x$D" != "x" ] ; then
 	$INTERCEPT_DIR/postinst_intercept update_font_cache ${PKG} mlprefix=${MLPREFIX} bindir=${bindir} \
-		libdir=${libdir} base_libdir=${base_libdir} localstatedir=${localstatedir}
+		libdir=${libdir} base_libdir=${base_libdir} fontconfigcachedir=${FONTCONFIG_CACHE_DIR}
 else
 	fc-cache
 fi
diff --git a/meta/recipes-graphics/fontconfig/fontconfig_2.11.1.bb b/meta/recipes-graphics/fontconfig/fontconfig_2.11.1.bb
index 797b321..7c5b4b4 100644
--- a/meta/recipes-graphics/fontconfig/fontconfig_2.11.1.bb
+++ b/meta/recipes-graphics/fontconfig/fontconfig_2.11.1.bb
@@ -37,6 +37,8 @@ DEBIAN_NOAUTONAME_fontconfig-utils = "1"
 
 inherit autotools pkgconfig
 
-EXTRA_OECONF = " --disable-docs --with-default-fonts=${datadir}/fonts"
+FONTCONFIG_CACHE_DIR ?= "${localstatedir}/cache/fontconfig"
+
+EXTRA_OECONF = " --disable-docs --with-default-fonts=${datadir}/fonts --with-cache-dir=${FONTCONFIG_CACHE_DIR}"
 
 BBCLASSEXTEND = "native"
diff --git a/scripts/postinst-intercepts/update_font_cache b/scripts/postinst-intercepts/update_font_cache
index 3907f25..c8c6018 100644
--- a/scripts/postinst-intercepts/update_font_cache
+++ b/scripts/postinst-intercepts/update_font_cache
@@ -2,6 +2,4 @@
 
 PSEUDO_UNLOAD=1 qemuwrapper -L $D -E LD_LIBRARY_PATH=$D/${libdir}:$D/${base_libdir}\
 					$D${bindir}/fc-cache --sysroot=$D
-chown -R root:root $D${localstatedir}/cache/fontconfig
-
-
+chown -R root:root $D${fontconfigcachedir}
-- 
2.3.5



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

* Re: [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC
  2015-04-21 10:53 [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC Martin Jansa
  2015-04-21 10:53 ` [dizzy][PATCH 2/3] postinst-intercepts/update_font_cache: fix ownership of fontconfig cache Martin Jansa
  2015-04-21 10:53 ` [dizzy][PATCH 3/3] fontcache: allow to pass different fontconfig cache dir Martin Jansa
@ 2015-04-27 11:04 ` Martin Jansa
  2015-04-27 12:10   ` Otavio Salvador
  2 siblings, 1 reply; 5+ messages in thread
From: Martin Jansa @ 2015-04-27 11:04 UTC (permalink / raw)
  To: openembedded-core

On Tue, Apr 21, 2015 at 12:53:01PM +0200, Martin Jansa wrote:
> * I don't see any usage for libpulse-* packages
> * adding '-' resolves the issue when we have separate recipe for
>   pulseaudio-modules-droid which isn't built to satisfy RDEPENDS
>   with the same name, because generic pulseaudio recipe seems to
>   RPROVIDE it through PACKAGES_DYNAMIC

ping for all 3 changes

> 
> Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
> ---
>  meta/recipes-multimedia/pulseaudio/pulseaudio.inc | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> index db144a9..99cad76 100644
> --- a/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> +++ b/meta/recipes-multimedia/pulseaudio/pulseaudio.inc
> @@ -79,7 +79,7 @@ PACKAGES =+ "libpulsecore libpulsecommon libpulse libpulse-simple libpulse-mainl
>  #upgrade path:
>  RREPLACES_pulseaudio-server = "libpulse-bin libpulse-conf"
>  
> -PACKAGES_DYNAMIC += "^pulseaudio-lib.* ^pulseaudio-module.* ^libpulse-lib.* ^libpulse-module.* "
> +PACKAGES_DYNAMIC += "^pulseaudio-lib-.* ^pulseaudio-module-.*"
>  
>  FILES_libpulsecore = "${libdir}/libpulsecore*.so"
>  FILES_libpulsecommon = "${libdir}/pulseaudio/libpulsecommon*.so"
> -- 
> 2.3.5
> 

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com


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

* Re: [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC
  2015-04-27 11:04 ` [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC Martin Jansa
@ 2015-04-27 12:10   ` Otavio Salvador
  0 siblings, 0 replies; 5+ messages in thread
From: Otavio Salvador @ 2015-04-27 12:10 UTC (permalink / raw)
  To: Martin Jansa; +Cc: Patches and discussions about the oe-core layer

On Mon, Apr 27, 2015 at 8:04 AM, Martin Jansa <martin.jansa@gmail.com> wrote:
> On Tue, Apr 21, 2015 at 12:53:01PM +0200, Martin Jansa wrote:
>> * I don't see any usage for libpulse-* packages
>> * adding '-' resolves the issue when we have separate recipe for
>>   pulseaudio-modules-droid which isn't built to satisfy RDEPENDS
>>   with the same name, because generic pulseaudio recipe seems to
>>   RPROVIDE it through PACKAGES_DYNAMIC
>
> ping for all 3 changes

Ack for all 3 changes.

-- 
Otavio Salvador                             O.S. Systems
http://www.ossystems.com.br        http://code.ossystems.com.br
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750


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

end of thread, other threads:[~2015-04-27 12:09 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-21 10:53 [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC Martin Jansa
2015-04-21 10:53 ` [dizzy][PATCH 2/3] postinst-intercepts/update_font_cache: fix ownership of fontconfig cache Martin Jansa
2015-04-21 10:53 ` [dizzy][PATCH 3/3] fontcache: allow to pass different fontconfig cache dir Martin Jansa
2015-04-27 11:04 ` [dizzy][PATCH 1/3] pulseaudio: use stricter PACKAGES_DYNAMIC Martin Jansa
2015-04-27 12:10   ` Otavio Salvador

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox