Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [CONSOLIDATED PULL 05/32] tcl: fix networking breakage
Date: Fri, 19 Aug 2011 09:31:50 -0500	[thread overview]
Message-ID: <1313764310.13995.103.camel@rex> (raw)
In-Reply-To: <811a2cc3d1f9d025f99f7decfa6d3603cb3ca5f2.1313700595.git.sgw@linux.intel.com>

On Thu, 2011-08-18 at 13:55 -0700, Saul Wold wrote:
> From: Koen Kooi <koen@dominion.thruhere.net>
> 
> Without providing these site entries tcl networking doesn't work on the target.
> 
> Original OE commit:
> 
> commit 0c340037644f87054462f62b50cd397c162615b7
> Author: Koen Kooi <koen@openembedded.org>
> Date:   Wed May 26 13:04:18 2010 +0200
> 
>     tcl: add 2 entries to arm-linux site file (possibly only glibc specific, need to check)
> 
> Signed-off-by: Koen Kooi <koen@dominion.thruhere.net>
> ---
>  meta/recipes-devtools/tcltk/tcl_8.5.9.bb |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/meta/recipes-devtools/tcltk/tcl_8.5.9.bb b/meta/recipes-devtools/tcltk/tcl_8.5.9.bb
> index 2eabb4d..1b4ae83 100644
> --- a/meta/recipes-devtools/tcltk/tcl_8.5.9.bb
> +++ b/meta/recipes-devtools/tcltk/tcl_8.5.9.bb
> @@ -11,7 +11,7 @@ LIC_FILES_CHKSUM = "file://../license.terms;md5=a47a9be26d03f925fc1fbd2784f27e11
>      file://../win/license.terms;md5=a47a9be26d03f925fc1fbd2784f27e11 \
>      "
>  
> -PR = "r1"
> +PR = "r2"
>  
>  BASE_SRC_URI = "${SOURCEFORGE_MIRROR}/tcl/tcl${PV}-src.tar.gz \
>                  file://tcl-add-soname.patch"
> @@ -30,7 +30,9 @@ inherit autotools
>  
>  DEPENDS_virtclass-native = ""
>  
> -EXTRA_OECONF = "--enable-threads"
> +EXTRA_OECONF = "--enable-threads \
> +                tcl_cv_api_getaddrinfo=yes \
> +               "
>  

Why are we adding these in the recipe rather than the site files? I can
see the reasons why we might but it does seem to be a change in
behaviour/policy?

Cheers,

Richard




  reply	other threads:[~2011-08-19 14:36 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-18 20:55 [CONSOLIDATED PULL 00/32] Various Fixes, Updates Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 01/32] image.bbclass, kernel.bbclass: create warning file about deleting deploydir files Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 02/32] sanity.bbclass: add optional untested host distro warning Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 03/32] qt4: allow a reduction in build time Saul Wold
2011-08-19 14:34   ` Richard Purdie
2011-08-18 20:55 ` [CONSOLIDATED PULL 04/32] glib-2.0: explicitly disable dtrace and systemtap for native varaint Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 05/32] tcl: fix networking breakage Saul Wold
2011-08-19 14:31   ` Richard Purdie [this message]
2011-08-18 20:55 ` [CONSOLIDATED PULL 06/32] perl-native: Update find_perl to find the perl wrapper Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 07/32] busybox: Change ${PN} to ${BPN} in file names Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 08/32] sysvinit: " Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 09/32] qemu: modify search paths for libgl Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 10/32] scripts/runqemu: " Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 11/32] mtd-utils: Added recipe of newer version 1.4.4 Saul Wold
2011-08-19 14:28   ` Richard Purdie
2011-08-18 20:55 ` [CONSOLIDATED PULL 12/32] libxkbcommon: new Saul Wold
2011-08-19 14:27   ` Richard Purdie
2011-08-19 14:33     ` Phil Blundell
2011-08-19 14:40       ` Martin Jansa
2011-08-19 16:08         ` Richard Purdie
2011-08-18 20:55 ` [CONSOLIDATED PULL 13/32] libxslt: fix packaging of static libraries Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 14/32] at: make at usable for common user Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 15/32] conf, recipes: Redefine LINKER_HASH_STYLE Saul Wold
2011-08-19  9:24   ` Phil Blundell
2011-08-19 14:26     ` Richard Purdie
2011-08-19 15:27       ` Khem Raj
2011-08-19 16:13         ` Richard Purdie
2011-08-19 16:35           ` Khem Raj
2011-08-18 20:55 ` [CONSOLIDATED PULL 16/32] gcc-4.6: Use --with-linker-hash-style configure option Saul Wold
2011-08-19 16:23   ` Phil Blundell
2011-08-19 18:29     ` Khem Raj
2011-08-19 18:55       ` Khem Raj
2011-08-18 20:55 ` [CONSOLIDATED PULL 17/32] task-core-tools: Use LTTNGUST variable instead of hardcoding lttng-ust Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 18/32] gdb: Add lttng-ust to DEPENDS only for eglibc Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 19/32] task-core-boot: allow distribution to override default dev_manager provider with VIRTUAL-RUNTIME_dev_manager Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 20/32] bluez4: update recipe from meta-oe Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 21/32] initrdscripts: fix init-live.sh Saul Wold
2011-08-22  4:30   ` Tom Zanussi
2011-08-18 20:55 ` [CONSOLIDATED PULL 22/32] curl: Use BPN instead of PN for style like lib${PN} Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 23/32] ea-acl: " Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 24/32] attr: " Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 25/32] acl: " Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 26/32] procps: Fix lib path to support multilib Saul Wold
2011-08-19 14:18   ` Richard Purdie
2011-08-18 20:55 ` [CONSOLIDATED PULL 27/32] eglibc: check dependencies among eglibc options Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 28/32] mailx: update license and distro tracking fileds Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 29/32] slang: add homepage and update distro tracking fields Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 30/32] alsa-tools: update license and add " Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 31/32] newt: update " Saul Wold
2011-08-18 20:55 ` [CONSOLIDATED PULL 32/32] file: new file version 5.07 Saul Wold
2011-08-19 16:14 ` [CONSOLIDATED PULL 00/32] Various Fixes, Updates Richard Purdie

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=1313764310.13995.103.camel@rex \
    --to=richard.purdie@linuxfoundation.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