All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ti-dsplink: Fixed location of toolchain
@ 2011-06-14  3:55 Joel A Fernandes
  2011-06-14  5:51 ` Denys Dmytriyenko
  2011-06-14  7:36 ` Koen Kooi
  0 siblings, 2 replies; 7+ messages in thread
From: Joel A Fernandes @ 2011-06-14  3:55 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Kridner, Jason, Koen

ti-dsplink: Fixed location of toolchain to build ti-dsplink.

Signed-off-by: Joel A Fernandes <agnel.joel@gmail.com>
---
 recipes/ti/ti-dsplink.inc |   34 +++++++++++++++++-----------------
 1 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/recipes/ti/ti-dsplink.inc b/recipes/ti/ti-dsplink.inc
index c48ec41..bbdc35a 100644
--- a/recipes/ti/ti-dsplink.inc
+++ b/recipes/ti/ti-dsplink.inc
@@ -94,23 +94,23 @@ do_compile() {
     # Build the gpp user space library
     cd ${DSPLINK}/gpp/src/api && make \
       CROSS_COMPILE="${TARGET_PREFIX}" \
-      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
-      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
-      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ld" \
-      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
-      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
+      CC="${CC}" \
+      AR="${AR}" \
+      LD="${LD}" \
+      COMPILER="${CC}" \
+      ARCHIVER="${AR}" \
       KERNEL_DIR="${STAGING_KERNEL_DIR}" \
       all
 
     # Build the gpp kernel space (debug and release)
     cd ${DSPLINK}/gpp/src && make \
-      OBJDUMP="${TARGET_PREFIX}objdump" \
+      OBJDUMP="${OBJDUMP}" \
       CROSS_COMPILE="${TARGET_PREFIX}" \
-      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
-      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
-      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ld" \
-      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
-      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
+      CC="${CC}" \
+      AR="${AR}" \
+      LD="${LD}" \
+      COMPILER="${CC}" \
+      ARCHIVER="${AR}" \
       KERNEL_DIR="${STAGING_KERNEL_DIR}" \
       BASE_BUILDOS="${STAGING_KERNEL_DIR}" \
       all
@@ -122,12 +122,12 @@ do_compile() {
       OSINC_PLATFORM="${TOOLCHAIN_PATH}/lib/gcc/${TARGET_SYS}/$(${TARGET_PREFIX}gcc -dumpversion)/include" \
       OSINC_TARGET="${BASE_TOOLCHAIN}/target/usr/include" \
       CROSS_COMPILE="${TARGET_PREFIX}" \
-      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
-      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
-      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
-      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
-      LINKER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
-      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
+      CC="${CC}" \
+      AR="${AR}" \
+      LD="${LD}" \
+      COMPILER="${CC}" \
+      ARCHIVER="${AR}" \
+      LINKER="${CC}" \
       KERNEL_DIR="${STAGING_KERNEL_DIR}" \
       all
 



^ permalink raw reply related	[flat|nested] 7+ messages in thread
* Re: [PATCH] ti-dsplink: Fixed location of toolchain
  2011-06-14  3:55 [PATCH] ti-dsplink: Fixed location of toolchain Joel A Fernandes
@ 2011-06-14  5:51 ` Denys Dmytriyenko
  2011-06-14  7:36 ` Koen Kooi
  1 sibling, 0 replies; 7+ messages in thread
From: Denys Dmytriyenko @ 2011-06-14  5:51 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Kridner, Jason, Koen

On Mon, Jun 13, 2011 at 10:55:58PM -0500, Joel A Fernandes wrote:
> ti-dsplink: Fixed location of toolchain to build ti-dsplink.
> 
> Signed-off-by: Joel A Fernandes <agnel.joel@gmail.com>

Have you tested this with an external toolchain?

-- 
Denys


> ---
>  recipes/ti/ti-dsplink.inc |   34 +++++++++++++++++-----------------
>  1 files changed, 17 insertions(+), 17 deletions(-)
> 
> diff --git a/recipes/ti/ti-dsplink.inc b/recipes/ti/ti-dsplink.inc
> index c48ec41..bbdc35a 100644
> --- a/recipes/ti/ti-dsplink.inc
> +++ b/recipes/ti/ti-dsplink.inc
> @@ -94,23 +94,23 @@ do_compile() {
>      # Build the gpp user space library
>      cd ${DSPLINK}/gpp/src/api && make \
>        CROSS_COMPILE="${TARGET_PREFIX}" \
> -      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> -      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ld" \
> -      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> +      CC="${CC}" \
> +      AR="${AR}" \
> +      LD="${LD}" \
> +      COMPILER="${CC}" \
> +      ARCHIVER="${AR}" \
>        KERNEL_DIR="${STAGING_KERNEL_DIR}" \
>        all
>  
>      # Build the gpp kernel space (debug and release)
>      cd ${DSPLINK}/gpp/src && make \
> -      OBJDUMP="${TARGET_PREFIX}objdump" \
> +      OBJDUMP="${OBJDUMP}" \
>        CROSS_COMPILE="${TARGET_PREFIX}" \
> -      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> -      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ld" \
> -      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> +      CC="${CC}" \
> +      AR="${AR}" \
> +      LD="${LD}" \
> +      COMPILER="${CC}" \
> +      ARCHIVER="${AR}" \
>        KERNEL_DIR="${STAGING_KERNEL_DIR}" \
>        BASE_BUILDOS="${STAGING_KERNEL_DIR}" \
>        all
> @@ -122,12 +122,12 @@ do_compile() {
>        OSINC_PLATFORM="${TOOLCHAIN_PATH}/lib/gcc/${TARGET_SYS}/$(${TARGET_PREFIX}gcc -dumpversion)/include" \
>        OSINC_TARGET="${BASE_TOOLCHAIN}/target/usr/include" \
>        CROSS_COMPILE="${TARGET_PREFIX}" \
> -      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> -      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      LINKER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> +      CC="${CC}" \
> +      AR="${AR}" \
> +      LD="${LD}" \
> +      COMPILER="${CC}" \
> +      ARCHIVER="${AR}" \
> +      LINKER="${CC}" \
>        KERNEL_DIR="${STAGING_KERNEL_DIR}" \
>        all
>  
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel



^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] ti-dsplink: Fixed location of toolchain
  2011-06-14  3:55 [PATCH] ti-dsplink: Fixed location of toolchain Joel A Fernandes
  2011-06-14  5:51 ` Denys Dmytriyenko
@ 2011-06-14  7:36 ` Koen Kooi
  2011-06-14 15:38   ` Denys Dmytriyenko
  1 sibling, 1 reply; 7+ messages in thread
From: Koen Kooi @ 2011-06-14  7:36 UTC (permalink / raw)
  To: openembedded-devel

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 14-06-11 05:55, Joel A Fernandes wrote:
> ti-dsplink: Fixed location of toolchain to build ti-dsplink.

It sadly really needs an absolute path to the tools :(

> 
> Signed-off-by: Joel A Fernandes <agnel.joel@gmail.com>
> ---
>  recipes/ti/ti-dsplink.inc |   34 +++++++++++++++++-----------------
>  1 files changed, 17 insertions(+), 17 deletions(-)
> 
> diff --git a/recipes/ti/ti-dsplink.inc b/recipes/ti/ti-dsplink.inc
> index c48ec41..bbdc35a 100644
> --- a/recipes/ti/ti-dsplink.inc
> +++ b/recipes/ti/ti-dsplink.inc
> @@ -94,23 +94,23 @@ do_compile() {
>      # Build the gpp user space library
>      cd ${DSPLINK}/gpp/src/api && make \
>        CROSS_COMPILE="${TARGET_PREFIX}" \
> -      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> -      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ld" \
> -      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> +      CC="${CC}" \
> +      AR="${AR}" \
> +      LD="${LD}" \
> +      COMPILER="${CC}" \
> +      ARCHIVER="${AR}" \
>        KERNEL_DIR="${STAGING_KERNEL_DIR}" \
>        all
>  
>      # Build the gpp kernel space (debug and release)
>      cd ${DSPLINK}/gpp/src && make \
> -      OBJDUMP="${TARGET_PREFIX}objdump" \
> +      OBJDUMP="${OBJDUMP}" \
>        CROSS_COMPILE="${TARGET_PREFIX}" \
> -      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> -      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ld" \
> -      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> +      CC="${CC}" \
> +      AR="${AR}" \
> +      LD="${LD}" \
> +      COMPILER="${CC}" \
> +      ARCHIVER="${AR}" \
>        KERNEL_DIR="${STAGING_KERNEL_DIR}" \
>        BASE_BUILDOS="${STAGING_KERNEL_DIR}" \
>        all
> @@ -122,12 +122,12 @@ do_compile() {
>        OSINC_PLATFORM="${TOOLCHAIN_PATH}/lib/gcc/${TARGET_SYS}/$(${TARGET_PREFIX}gcc -dumpversion)/include" \
>        OSINC_TARGET="${BASE_TOOLCHAIN}/target/usr/include" \
>        CROSS_COMPILE="${TARGET_PREFIX}" \
> -      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> -      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      LINKER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> -      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> +      CC="${CC}" \
> +      AR="${AR}" \
> +      LD="${LD}" \
> +      COMPILER="${CC}" \
> +      ARCHIVER="${AR}" \
> +      LINKER="${CC}" \
>        KERNEL_DIR="${STAGING_KERNEL_DIR}" \
>        all

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFN9w96MkyGM64RGpERAkk+AJ9kAE91Op6450wG1xySGpAa3drFXQCfSNcO
F1rBpao1N5wwWAlNk6kwiqU=
=x1pA
-----END PGP SIGNATURE-----




^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] ti-dsplink: Fixed location of toolchain
  2011-06-14  7:36 ` Koen Kooi
@ 2011-06-14 15:38   ` Denys Dmytriyenko
  2011-06-14 15:47     ` Koen Kooi
  0 siblings, 1 reply; 7+ messages in thread
From: Denys Dmytriyenko @ 2011-06-14 15:38 UTC (permalink / raw)
  To: openembedded-devel

On Tue, Jun 14, 2011 at 09:36:26AM +0200, Koen Kooi wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 14-06-11 05:55, Joel A Fernandes wrote:
> > ti-dsplink: Fixed location of toolchain to build ti-dsplink.
> 
> It sadly really needs an absolute path to the tools :(

So, what was this patch trying to fix anyway? Is there something broken?

-- 
Denys


> > Signed-off-by: Joel A Fernandes <agnel.joel@gmail.com>
> > ---
> >  recipes/ti/ti-dsplink.inc |   34 +++++++++++++++++-----------------
> >  1 files changed, 17 insertions(+), 17 deletions(-)
> > 
> > diff --git a/recipes/ti/ti-dsplink.inc b/recipes/ti/ti-dsplink.inc
> > index c48ec41..bbdc35a 100644
> > --- a/recipes/ti/ti-dsplink.inc
> > +++ b/recipes/ti/ti-dsplink.inc
> > @@ -94,23 +94,23 @@ do_compile() {
> >      # Build the gpp user space library
> >      cd ${DSPLINK}/gpp/src/api && make \
> >        CROSS_COMPILE="${TARGET_PREFIX}" \
> > -      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> > -      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> > -      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ld" \
> > -      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> > -      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> > +      CC="${CC}" \
> > +      AR="${AR}" \
> > +      LD="${LD}" \
> > +      COMPILER="${CC}" \
> > +      ARCHIVER="${AR}" \
> >        KERNEL_DIR="${STAGING_KERNEL_DIR}" \
> >        all
> >  
> >      # Build the gpp kernel space (debug and release)
> >      cd ${DSPLINK}/gpp/src && make \
> > -      OBJDUMP="${TARGET_PREFIX}objdump" \
> > +      OBJDUMP="${OBJDUMP}" \
> >        CROSS_COMPILE="${TARGET_PREFIX}" \
> > -      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> > -      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> > -      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ld" \
> > -      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> > -      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> > +      CC="${CC}" \
> > +      AR="${AR}" \
> > +      LD="${LD}" \
> > +      COMPILER="${CC}" \
> > +      ARCHIVER="${AR}" \
> >        KERNEL_DIR="${STAGING_KERNEL_DIR}" \
> >        BASE_BUILDOS="${STAGING_KERNEL_DIR}" \
> >        all
> > @@ -122,12 +122,12 @@ do_compile() {
> >        OSINC_PLATFORM="${TOOLCHAIN_PATH}/lib/gcc/${TARGET_SYS}/$(${TARGET_PREFIX}gcc -dumpversion)/include" \
> >        OSINC_TARGET="${BASE_TOOLCHAIN}/target/usr/include" \
> >        CROSS_COMPILE="${TARGET_PREFIX}" \
> > -      CC="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> > -      LD="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> > -      AR="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> > -      COMPILER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> > -      LINKER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}gcc" \
> > -      ARCHIVER="${TOOLCHAIN_PATH}/bin/${TARGET_PREFIX}ar" \
> > +      CC="${CC}" \
> > +      AR="${AR}" \
> > +      LD="${LD}" \
> > +      COMPILER="${CC}" \
> > +      ARCHIVER="${AR}" \
> > +      LINKER="${CC}" \
> >        KERNEL_DIR="${STAGING_KERNEL_DIR}" \
> >        all
> 
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.5 (Darwin)
> 
> iD8DBQFN9w96MkyGM64RGpERAkk+AJ9kAE91Op6450wG1xySGpAa3drFXQCfSNcO
> F1rBpao1N5wwWAlNk6kwiqU=
> =x1pA
> -----END PGP SIGNATURE-----
> 
> 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel



^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] ti-dsplink: Fixed location of toolchain
  2011-06-14 15:38   ` Denys Dmytriyenko
@ 2011-06-14 15:47     ` Koen Kooi
  2011-06-14 19:31       ` Denys Dmytriyenko
  0 siblings, 1 reply; 7+ messages in thread
From: Koen Kooi @ 2011-06-14 15:47 UTC (permalink / raw)
  To: openembedded-devel

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 14-06-11 17:38, Denys Dmytriyenko wrote:
> On Tue, Jun 14, 2011 at 09:36:26AM +0200, Koen Kooi wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> On 14-06-11 05:55, Joel A Fernandes wrote:
>>> ti-dsplink: Fixed location of toolchain to build ti-dsplink.
>>
>> It sadly really needs an absolute path to the tools :(
> 
> So, what was this patch trying to fix anyway? Is there something broken?

Using an external toolchain without your import magic and then
overriding TOOLCHAINPATH in local.conf is broken. You might call it
broken by design :)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFN94J7MkyGM64RGpERAufjAJ0byRdnVm9SmVJ0Hlqsyv+BJv75uwCggF42
qeHxFXcjNMNr11nIr+Knmio=
=mZHT
-----END PGP SIGNATURE-----




^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] ti-dsplink: Fixed location of toolchain
  2011-06-14 15:47     ` Koen Kooi
@ 2011-06-14 19:31       ` Denys Dmytriyenko
  2011-06-14 19:38         ` Koen Kooi
  0 siblings, 1 reply; 7+ messages in thread
From: Denys Dmytriyenko @ 2011-06-14 19:31 UTC (permalink / raw)
  To: openembedded-devel

On Tue, Jun 14, 2011 at 05:47:07PM +0200, Koen Kooi wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> On 14-06-11 17:38, Denys Dmytriyenko wrote:
> > On Tue, Jun 14, 2011 at 09:36:26AM +0200, Koen Kooi wrote:
> >> -----BEGIN PGP SIGNED MESSAGE-----
> >> Hash: SHA1
> >>
> >> On 14-06-11 05:55, Joel A Fernandes wrote:
> >>> ti-dsplink: Fixed location of toolchain to build ti-dsplink.
> >>
> >> It sadly really needs an absolute path to the tools :(
> > 
> > So, what was this patch trying to fix anyway? Is there something broken?
> 
> Using an external toolchain without your import magic and then
> overriding TOOLCHAINPATH in local.conf is broken. You might call it
> broken by design :)

As you know, the design is hindered by requirements, so ease up! :)

As of the "import magic" - there is nothing magical about it, it just sets up 
TOOLCHAIN_PATH and TOOLCHAIN_SYSPATH automatically. Otherwise you'd have to 
set at least TOOLCHAIN_PATH manually and it works. So, I still don't see what 
exactly is broken and needs to be fixed.

BTW, the requirements for absolute toolchain path come from dsplink and not 
the external toolchain, AFAICR...

-- 
Denys



^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH] ti-dsplink: Fixed location of toolchain
  2011-06-14 19:31       ` Denys Dmytriyenko
@ 2011-06-14 19:38         ` Koen Kooi
  0 siblings, 0 replies; 7+ messages in thread
From: Koen Kooi @ 2011-06-14 19:38 UTC (permalink / raw)
  To: openembedded-devel

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 14-06-11 21:31, Denys Dmytriyenko wrote:
> On Tue, Jun 14, 2011 at 05:47:07PM +0200, Koen Kooi wrote:
>> -----BEGIN PGP SIGNED MESSAGE-----
>> Hash: SHA1
>>
>> On 14-06-11 17:38, Denys Dmytriyenko wrote:
>>> On Tue, Jun 14, 2011 at 09:36:26AM +0200, Koen Kooi wrote:
>>>> -----BEGIN PGP SIGNED MESSAGE-----
>>>> Hash: SHA1
>>>>
>>>> On 14-06-11 05:55, Joel A Fernandes wrote:
>>>>> ti-dsplink: Fixed location of toolchain to build ti-dsplink.
>>>>
>>>> It sadly really needs an absolute path to the tools :(
>>>
>>> So, what was this patch trying to fix anyway? Is there something broken?
>>
>> Using an external toolchain without your import magic and then
>> overriding TOOLCHAINPATH in local.conf is broken. You might call it
>> broken by design :)
> 
> As you know, the design is hindered by requirements, so ease up! :)
> 
> As of the "import magic" - there is nothing magical about it, it just sets up 
> TOOLCHAIN_PATH and TOOLCHAIN_SYSPATH automatically. Otherwise you'd have to 
> set at least TOOLCHAIN_PATH manually and it works. So, I still don't see what 
> exactly is broken and needs to be fixed.

Joel setup his tree the wrong way

> BTW, the requirements for absolute toolchain path come from dsplink and not 
> the external toolchain, AFAICR...

dsplink and others, yes.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Darwin)

iD8DBQFN97jDMkyGM64RGpERAimXAKC70bo3sUyDf+/jzJux/n8mXRurYgCgqsEt
RT2PAYT1EIeptDARW/MoEcE=
=Yfp9
-----END PGP SIGNATURE-----




^ permalink raw reply	[flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-06-14 19:42 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-14  3:55 [PATCH] ti-dsplink: Fixed location of toolchain Joel A Fernandes
2011-06-14  5:51 ` Denys Dmytriyenko
2011-06-14  7:36 ` Koen Kooi
2011-06-14 15:38   ` Denys Dmytriyenko
2011-06-14 15:47     ` Koen Kooi
2011-06-14 19:31       ` Denys Dmytriyenko
2011-06-14 19:38         ` Koen Kooi

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.