Openembedded Core Discussions
 help / color / mirror / Atom feed
* [PATCH 0/1] ncurses: enable-pc-files requires PKG_CONFIG_LIBDIR existed
@ 2014-09-23  4:31 Robert Yang
  2014-09-23  4:31 ` [PATCH 1/1] " Robert Yang
  0 siblings, 1 reply; 3+ messages in thread
From: Robert Yang @ 2014-09-23  4:31 UTC (permalink / raw)
  To: openembedded-core

The following changes since commit e72aac2ae9e5fce1715fa04b7e94034fd06892d9:

  alsa-lib: libasound should runtime depends on alsa-conf (2014-09-03 11:30:11 +0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib rbt/ncurses
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=rbt/ncurses

Robert Yang (1):
  ncurses: enable-pc-files requires PKG_CONFIG_LIBDIR existed

 meta/recipes-core/ncurses/ncurses.inc |    2 ++
 1 file changed, 2 insertions(+)

-- 
1.7.9.5



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

* [PATCH 1/1] ncurses: enable-pc-files requires PKG_CONFIG_LIBDIR existed
  2014-09-23  4:31 [PATCH 0/1] ncurses: enable-pc-files requires PKG_CONFIG_LIBDIR existed Robert Yang
@ 2014-09-23  4:31 ` Robert Yang
  2014-09-23 11:36   ` Burton, Ross
  0 siblings, 1 reply; 3+ messages in thread
From: Robert Yang @ 2014-09-23  4:31 UTC (permalink / raw)
  To: openembedded-core

Fixed ncurses.do_configure:
configure: WARNING: did not find library /path/to/tmp/sysroots/qemuarm/usr/lib/pkgconfig

And then anyone requires ncurses.pc will fail.

The configure.in checks:
[snip]
if test -n "$PKG_CONFIG_LIBDIR" && test -d "$PKG_CONFIG_LIBDIR" ; then
[snip]

Create PKG_CONFIG_LIBDIR in do_configure will fix the problem.

We can reproduce the problem by:
Set SSTATE_DIR=/path/to/sstate-cache
1) In build1, make sure everything is ready in SSTATE_DIR
$ bitbake ncurses
2) In build2, rebuild ncurses only:
$ bitbake ncurses -ccleansstate && bitbake ncurses

Then we will see the warning in log.do_configure.

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 meta/recipes-core/ncurses/ncurses.inc |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/meta/recipes-core/ncurses/ncurses.inc b/meta/recipes-core/ncurses/ncurses.inc
index 521d0e4..10f7dd1 100644
--- a/meta/recipes-core/ncurses/ncurses.inc
+++ b/meta/recipes-core/ncurses/ncurses.inc
@@ -98,6 +98,8 @@ do_configure() {
         # not the case for /dev/null redirections)
         export cf_cv_working_poll=yes
 
+	# The --enable-pc-files requires PKG_CONFIG_LIBDIR existed
+	mkdir -p ${PKG_CONFIG_LIBDIR}
 	( cd ${S}; gnu-configize --force )
 	ncurses_configure "narrowc" || \
 		return 1
-- 
1.7.9.5



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

* Re: [PATCH 1/1] ncurses: enable-pc-files requires PKG_CONFIG_LIBDIR existed
  2014-09-23  4:31 ` [PATCH 1/1] " Robert Yang
@ 2014-09-23 11:36   ` Burton, Ross
  0 siblings, 0 replies; 3+ messages in thread
From: Burton, Ross @ 2014-09-23 11:36 UTC (permalink / raw)
  To: Robert Yang; +Cc: OE-core

On 23 September 2014 05:31, Robert Yang <liezhi.yang@windriver.com> wrote:
> The configure.in checks:
> [snip]
> if test -n "$PKG_CONFIG_LIBDIR" && test -d "$PKG_CONFIG_LIBDIR" ; then
> [snip]
>
> Create PKG_CONFIG_LIBDIR in do_configure will fix the problem.
>
> We can reproduce the problem by:
> Set SSTATE_DIR=/path/to/sstate-cache
> 1) In build1, make sure everything is ready in SSTATE_DIR
> $ bitbake ncurses
> 2) In build2, rebuild ncurses only:
> $ bitbake ncurses -ccleansstate && bitbake ncurses

With this fixed I expect you can remove what looks like a bad
assignment of PKG_CONFIG_LIBDIR in _install_cfgs.  Also note that it's
currently detecting the wrong path at configure time and the
install_cfgs assignment is forcing the right path.

Ross


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

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

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-23  4:31 [PATCH 0/1] ncurses: enable-pc-files requires PKG_CONFIG_LIBDIR existed Robert Yang
2014-09-23  4:31 ` [PATCH 1/1] " Robert Yang
2014-09-23 11:36   ` Burton, Ross

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