All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Martin Jansa <martin.jansa@gmail.com>
Cc: Peter Kjellerstedt <peter.kjellerstedt@axis.com>,
	OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [PATCHv3 0/5] Add dummy tools to help identify needed dependencies
Date: Thu, 09 Mar 2017 00:44:23 +0000	[thread overview]
Message-ID: <1489020263.22968.88.camel@linuxfoundation.org> (raw)
In-Reply-To: <1489018715.22968.86.camel@linuxfoundation.org>

On Thu, 2017-03-09 at 00:18 +0000, Richard Purdie wrote:
> How badly do people dislike the patch below?
> 
> From: Richard Purdie <richard.purdie@linuxfoundation.org>
> Subject: bitbake/oe-core: Filter contents of PATH
> 
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> 
> diff --git a/bitbake/lib/bb/utils.py b/bitbake/lib/bb/utils.py
> index d6bcfa3..dbb74dd 100644
> --- a/bitbake/lib/bb/utils.py
> +++ b/bitbake/lib/bb/utils.py
> @@ -1526,3 +1526,13 @@ class LogCatcher(logging.Handler):
>          self.messages.append(bb.build.logformatter.format(record))
>      def contains(self, message):
>          return (message in self.messages)
> +
> +def setup_native_bindir(dest, toolsvar, d):
> +    tools = d.getVar(toolsvar).split()
> +    path = os.environ.get("PATH")
> +    mkdirhier(dest)
> +    for tool in tools:
> +        desttool = os.path.join(dest, tool)
> +        if not os.path.exists(desttool):
> +            srctool = which(path, tool)
> +            os.symlink(srctool, desttool)
> diff --git a/meta/conf/layer.conf b/meta/conf/layer.conf
> index 87c235f..21265ed 100644
> --- a/meta/conf/layer.conf
> +++ b/meta/conf/layer.conf
> @@ -59,3 +59,14 @@ SIGGEN_EXCLUDE_SAFE_RECIPE_DEPS += " \
>    oprofile->virtual/kernel \
>  "
>  
> +NATIVETOOLS = " \
> +    bash sh cut sed gcc ld git rm install which find xargs cat true
> mktemp \
> +    grep tar gzip touch cp mv basename dirname tr getopt sort awk
> head tail \
> +    mkdir patch uniq perl python chmod python3 ar strip expr ls make
> as \
> +    ranlib egrep echo chown cpio tee wc wget bzip2 stat date rmdir
> od diff \
> +    md5sum unlzma dd chrpath file pod2man gunzip python2.7 ln g++ [
> \
> +    taskset \

Adding "false true uname test hostname nm objdump objcopy cmp printf
env" gets this going much further. taskset is only above as I have
local patches in my build which use it for other experiments and I
added it just to get things working.

Cheers,

Richard


      reply	other threads:[~2017-03-09  0:44 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-03 11:17 [PATCHv3 0/5] Add dummy tools to help identify needed dependencies Peter Kjellerstedt
2017-03-03 11:17 ` [PATCHv3 1/5] module.bbclass: Add inherit of pkgconfig Peter Kjellerstedt
2017-03-03 11:17 ` [PATCHv3 2/5] linux-libc-headers: " Peter Kjellerstedt
2017-03-03 11:17 ` [PATCHv3 3/5] scripts/dbus-binding-tool: Add a dummy version that always fails Peter Kjellerstedt
2017-03-03 11:17 ` [PATCHv3 4/5] scripts/gdbus-codegen: " Peter Kjellerstedt
2017-03-03 11:17 ` [PATCHv3 5/5] scripts/pkg-config: " Peter Kjellerstedt
2017-03-03 16:23 ` [PATCHv3 0/5] Add dummy tools to help identify needed dependencies Max Krummenacher
2017-03-03 19:52   ` Peter Kjellerstedt
2017-03-08  9:43     ` Peter Kjellerstedt
2017-03-08 17:14       ` Khem Raj
2017-03-08 17:21       ` Burton, Ross
2017-03-08 17:26         ` Mark Hatle
2017-03-08 18:00         ` Richard Purdie
2017-03-08 18:41           ` Martin Jansa
2017-03-09  0:18             ` Richard Purdie
2017-03-09  0:44               ` Richard Purdie [this message]

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=1489020263.22968.88.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=martin.jansa@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=peter.kjellerstedt@axis.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.