[PATCH] common: add dependency !SANDBOX on imd target tool

[PATCH] common: add dependency !SANDBOX on imd target tool

[Antony Pavlov]

The build of bareboximd-target tool fails on sandbox architecture:

      CC      scripts/bareboximd-target
    /tmp/ccoGPulo.o: In function `imd_concat_strings':
    scripts/../common/imd.c:216: undefined reference to `barebox_malloc'
    /tmp/ccoGPulo.o: In function `read_file_2':
    scripts/bareboximd.c:68: undefined reference to `barebox_open'
    scripts/bareboximd.c:74: undefined reference to `barebox_lseek'
    scripts/bareboximd.c:84: undefined reference to `barebox_lseek'
    scripts/bareboximd.c:90: undefined reference to `barebox_malloc'
    scripts/bareboximd.c:99: undefined reference to `barebox_read'
    scripts/bareboximd.c:119: undefined reference to `barebox_free'
    scripts/bareboximd.c:121: undefined reference to `barebox_close'
    /tmp/ccoGPulo.o: In function `imd_command':
    scripts/../common/imd.c:292: undefined reference to `barebox_printf'
    scripts/../common/imd.c:319: undefined reference to `barebox_printf'
    scripts/../common/imd.c:322: undefined reference to `barebox_free'
    scripts/../common/imd.c:324: undefined reference to `barebox_printf'
    /tmp/ccoGPulo.o: In function `usage':
    scripts/bareboximd.c:134: undefined reference to `barebox_printf'
    collect2: error: ld returned 1 exit status
    scripts/Makefile:58: recipe for target 'scripts/bareboximd-target' failed

See also

    commit d4aa01503348a033b1057d8a66c20a4f5819e01f
    Author: Alexander Aring <alex.aring@gmail.com>
    Date:   Tue Nov 19 02:08:13 2013 +0100

        common: add dependency !SANDBOX on target tools

        The build of target tools fails on sandbox architecture.
        We don't need any target tools in this case, so add a dependency.

Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
Cc: Alexander Aring <alex.aring@gmail.com>
---
 common/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/common/Kconfig b/common/Kconfig
index 3b87a3c..7c09e8c 100644
--- a/common/Kconfig
+++ b/common/Kconfig
@@ -542,6 +542,7 @@ config IMD
 config IMD_TARGET
 	bool "build bareboximd target tool"
 	depends on IMD
+	depends on !SANDBOX
 
 config KERNEL_INSTALL_TARGET
 	bool
-- 
2.7.0


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH] common: add dependency !SANDBOX on imd target tool

[Sascha Hauer]

On Thu, Apr 07, 2016 at 10:52:37AM +0300, Antony Pavlov wrote:
> The build of bareboximd-target tool fails on sandbox architecture:
> 
>       CC      scripts/bareboximd-target
>     /tmp/ccoGPulo.o: In function `imd_concat_strings':
>     scripts/../common/imd.c:216: undefined reference to `barebox_malloc'
>     /tmp/ccoGPulo.o: In function `read_file_2':
>     scripts/bareboximd.c:68: undefined reference to `barebox_open'
>     scripts/bareboximd.c:74: undefined reference to `barebox_lseek'
>     scripts/bareboximd.c:84: undefined reference to `barebox_lseek'
>     scripts/bareboximd.c:90: undefined reference to `barebox_malloc'
>     scripts/bareboximd.c:99: undefined reference to `barebox_read'
>     scripts/bareboximd.c:119: undefined reference to `barebox_free'
>     scripts/bareboximd.c:121: undefined reference to `barebox_close'
>     /tmp/ccoGPulo.o: In function `imd_command':
>     scripts/../common/imd.c:292: undefined reference to `barebox_printf'
>     scripts/../common/imd.c:319: undefined reference to `barebox_printf'
>     scripts/../common/imd.c:322: undefined reference to `barebox_free'
>     scripts/../common/imd.c:324: undefined reference to `barebox_printf'
>     /tmp/ccoGPulo.o: In function `usage':
>     scripts/bareboximd.c:134: undefined reference to `barebox_printf'
>     collect2: error: ld returned 1 exit status
>     scripts/Makefile:58: recipe for target 'scripts/bareboximd-target' failed
> 
> See also
> 
>     commit d4aa01503348a033b1057d8a66c20a4f5819e01f
>     Author: Alexander Aring <alex.aring@gmail.com>
>     Date:   Tue Nov 19 02:08:13 2013 +0100
> 
>         common: add dependency !SANDBOX on target tools
> 
>         The build of target tools fails on sandbox architecture.
>         We don't need any target tools in this case, so add a dependency.
> 
> Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com>
> Cc: Alexander Aring <alex.aring@gmail.com>
> ---
>  common/Kconfig | 1 +
>  1 file changed, 1 insertion(+)

Applied, thanks

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox