Re: [PATCH 2/3] stream: Add append for TISDK customizations

[Denys Dmytriyenko <denys@ti.com>]

On Wed, Mar 04, 2015 at 02:05:15PM -0600, Franklin S. Cooper Jr wrote:
> From: "Franklin S. Cooper Jr" <fcooper@ti.com>
> 
> * Use an SDK branch that slightly tweaks the makefile to include generic
>   optimizations that should be used by all TI platforms.
> * Also manually set certain variables to insure no additional/unexpected
>   compiler optimization are pulled in.
> 
> Signed-off-by: Franklin S. Cooper Jr <fcooper@ti.com>
> ---
>  .../recipes-benchmark/stream/stream_git.bbappend   |   10 ++++++++++
>  1 file changed, 10 insertions(+)
>  create mode 100644 meta-arago-distro/recipes-benchmark/stream/stream_git.bbappend
> 
> diff --git a/meta-arago-distro/recipes-benchmark/stream/stream_git.bbappend b/meta-arago-distro/recipes-benchmark/stream/stream_git.bbappend
> new file mode 100644
> index 0000000..83adfb6
> --- /dev/null
> +++ b/meta-arago-distro/recipes-benchmark/stream/stream_git.bbappend
> @@ -0,0 +1,10 @@
> +PR_append = "-arago0"
> +
> +BRANCH = "sdk"
> +SRCREV = "cfd73a33c0ceef1f8532ce6a71de3bcf9435f6a2"
> +
> +do_compile_prepend() {
> +    #Explicitly set some variables to insure no unexpected optimizations are being passed in.
> +    export CFLAGS="-isystem${EXTERNAL_TOOLCHAIN}/${ELT_TARGET_SYS}/include"
> +    export LDFLAGS="-L${EXTERNAL_TOOLCHAIN}/${ELT_TARGET_SYS}/lib -Wl,-rpath-link,${EXTERNAL_TOOLCHAIN}/${ELT_TARGET_SYS}/lib"

NAK, this assumes external Linaro toolchain. Can you make it more generic? Why 
can't you use the sysroots?

-- 
Denys