Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Saul Wold <sgw@linux.intel.com>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.linuxtogo.org>
Cc: Zhenfeng.Zhao@windriver.com
Subject: Re: [PATCH 1/1] at 3.1.13: get 'at' depends on 'flex-native' explicitly
Date: Thu, 09 Aug 2012 16:57:08 +0300	[thread overview]
Message-ID: <5023C1B4.7040602@linux.intel.com> (raw)
In-Reply-To: <ab9afe9c0eab7dc2d9832cd2afabe1eededc7cae.1344480978.git.wenzong.fan@windriver.com>

On 08/09/2012 05:59 AM, wenzong.fan@windriver.com wrote:
> From: Wenzong Fan <wenzong.fan@windriver.com>
>
> 'at' needs flex to generate source file lex.yy.c, but it doesn't
> depend on flex-native directly; This maybe cause a do_compile
> failure if flex-native is not built or included in project.
>
> Error message is:
>      error: lex.yy.c: No such file or directory
>
> Get 'at' depends on flex-native to fix this potential issue.
>
> [YOCTO #2919]
>
> Signed-off-by: Wenzong Fan <wenzong.fan@windriver.com>
> ---
>   meta/recipes-extended/at/at_3.1.13.bb |    4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-extended/at/at_3.1.13.bb b/meta/recipes-extended/at/at_3.1.13.bb
> index ce9396a..a391d6e 100644
> --- a/meta/recipes-extended/at/at_3.1.13.bb
> +++ b/meta/recipes-extended/at/at_3.1.13.bb
> @@ -4,14 +4,14 @@ the system load levels drop to a particular level."
>   SECTION = "base"
>   LICENSE="GPLv2+"
>   LIC_FILES_CHKSUM = "file://COPYING;md5=4325afd396febcb659c36b49533135d4"
> -DEPENDS = "flex initscripts \
> +DEPENDS = "flex flex-native initscripts \
Ok, I understand the flex-native dependency, that makes sense, but I 
wonder why there was a dependency on flex originally?  Is that actually 
needed?

Sau!

>   	${@base_contains('DISTRO_FEATURES', 'pam', 'libpam', '', d)}"
>   RDEPENDS_${PN} = "${@base_contains('DISTRO_FEATURES', 'pam', '${PAM_DEPS}', '', d)}"
>   PAM_DEPS = "libpam libpam-runtime pam-plugin-env pam-plugin-limits"
>
>   RCONFLICTS_${PN} = "atd"
>   RREPLACES_${PN} = "atd"
> -PR = "r2"
> +PR = "r3"
>
>   SRC_URI = "${DEBIAN_MIRROR}/main/a/at/at_${PV}.orig.tar.gz \
>       file://configure.patch \
>



  parent reply	other threads:[~2012-08-09 14:10 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-09  2:59 [PATCH 0/1] at 3.1.13: get 'at' depends on 'flex-native' explicitly wenzong.fan
2012-08-09  2:59 ` [PATCH 1/1] " wenzong.fan
2012-08-09 13:44   ` Paul Eggleton
2012-08-09 13:57   ` Saul Wold [this message]
     [not found]   ` <1880382.26omCZ6a7m@helios>
2012-08-10  1:49     ` wenzong fan
2012-08-16 18:59 ` [PATCH 0/1] " Saul Wold

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5023C1B4.7040602@linux.intel.com \
    --to=sgw@linux.intel.com \
    --cc=Zhenfeng.Zhao@windriver.com \
    --cc=openembedded-core@lists.linuxtogo.org \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox