Bug 259583 – sys-fs/mtd-utils-20080907 fails to cross compile during src_install

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Robert Piasek (RETIRED) 2009-02-19 12:20:21 UTC]

someone added needed CC="$(tc-getCC)" to src_compile, but forgot about src_install.


Reproducible: Always

[Comment 1 Robert Piasek (RETIRED) 2009-02-19 12:21:17 UTC]

Created attachment 182539 [details, diff]
fixes src_install

this patch fixes src_install

[Comment 2 SpanKY 2009-02-20 22:17:39 UTC]

erm, no ... the compiler should not be run in src_install

[Comment 3 solar (RETIRED) 2009-02-20 22:59:37 UTC]

=mtd-utils-20080102 (cross-works)
=mtd-utils-20080907 (cross-fails) 
 patch from comment #1 made no difference even if it's wrong.

mtd-utils now depends on uuid/uuid.h which is only provided by sys-libs/e2fsprogs-libs but no proper depend is listed.

Cross tested using:
CHOST="powerpc-pc-linux-uclibc"
CBUILD="x86_64-pc-linux-gnu"

[Comment 4 Robert Piasek (RETIRED) 2009-02-21 00:25:27 UTC]

(In reply to comment #3)
> =mtd-utils-20080102 (cross-works)
> =mtd-utils-20080907 (cross-fails) 
>  patch from comment #1 made no difference even if it's wrong.
> 
> mtd-utils now depends on uuid/uuid.h which is only provided by
> sys-libs/e2fsprogs-libs but no proper depend is listed.
> 
> Cross tested using:
> CHOST="powerpc-pc-linux-uclibc"
> CBUILD="x86_64-pc-linux-gnu"
> 

(In reply to comment #3)
> =mtd-utils-20080102 (cross-works)
> =mtd-utils-20080907 (cross-fails) 
>  patch from comment #1 made no difference even if it's wrong.
> 
> mtd-utils now depends on uuid/uuid.h which is only provided by
> sys-libs/e2fsprogs-libs but no proper depend is listed.
> 
> Cross tested using:
> CHOST="powerpc-pc-linux-uclibc"
> CBUILD="x86_64-pc-linux-gnu"
> 

without this patch HOST compiler is used for src_install

Starting src_install                                                                                                                                              
make -j2 install DESTDIR=/var/tmp/paludis/sys-fs-mtd-utils-20080907/image/                                                                                            
mkdir -p .                                                                                                                                                            
gcc -I./include   -Os -pipe -march=armv4t -mtune=arm920t -fomit-frame-pointer -I/usr/armv4tl-softfloat-linux-gnueabi/usr/include/ -I/usr/armv4tl-softfloat-linux-gnueabi/include/ -Wall -c -o ftl_format.o ftl_format.c -g -Wp,-MD,./.ftl_format.c.dep                                                                                      
mkdir -p .                                                                                                                                                            
gcc -I./include   -Os -pipe -march=armv4t -mtune=arm920t -fomit-frame-pointer -I/usr/armv4tl-softfloat-linux-gnueabi/usr/include/ -I/usr/armv4tl-softfloat-linux-gnueabi/include/ -Wall -c -o flash_erase.o flash_erase.c -g -Wp,-MD,./.flash_erase.c.dep                                                                                   
flash_erase.c:1: error: bad value (armv4t) for -march= switch                                                                                                         
flash_erase.c:1: error: bad value (arm920t) for -mtune= switch                                                                                                        
ftl_format.c:1: error: bad value (armv4t) for -march= switch                                                                                                          
ftl_format.c:1: error: bad value (arm920t) for -mtune= switch         

[Comment 5 SpanKY 2009-02-21 01:45:31 UTC]

then the package needs to be fixed to not run the compiler in src_install

[Comment 6 Maksim 'max_posedon' Melnikau 2009-05-07 10:37:40 UTC]

One more related error, 
sys-libs/e2fsprogs-libs should be added in rdeps

[Comment 7 SpanKY 2009-07-03 05:54:33 UTC]

ive updated the snapshot and src_install doesnt seem to run any toolchain commands, so things should be fixed now