Issue 635147  has been merged into this issue.

coreboot is failing because chromes-mrc failed to produce FSP.fd. Why it failed but reporting success is another bug/story.

make -C GnuGenBootSector
make[1]: Entering directory '/build/chell/tmp/portage/sys-boot/chromeos-mrc-0.0.1-r209/work/chromeos-mrc-0.0.1/edk2/BaseTools/Source/C/GnuGenBootSector'
mkdir ../bin
x86_64-cros-linux-gnu-gcc  -c -MD -fshort-wchar -fno-strict-aliasing -Wall -Werror -Wno-deprecated-declarations -nostdlib -c -g  -I .. -I ../Include/Common -I ../Include/ -I ../Include/IndustryStandard -I ../Common/ -I .. -I . -I ../Include/X64/  GnuGenBootSector.c -o GnuGenBootSector.o
x86_64-cros-linux-gnu-gcc -o ../bin/GnuGenBootSector  GnuGenBootSector.o -L../libs -lCommon
/usr/x86_64-pc-linux-gnu/x86_64-cros-linux-gnu/binutils-bin/2.25.51/ld.gold.real: error: GnuGenBootSector.o: requires dynamic R_X86_64_32 reloc which may overflow at runtime; recompile with -fPIC
/usr/x86_64-pc-linux-gnu/x86_64-cros-linux-gnu/binutils-bin/2.25.51/ld.gold.real: error: ../libs/libCommon.a(CommonLib.o): requires dynamic R_X86_64_32 reloc which may overflow at runtime; recompile with -fPIC
/usr/x86_64-pc-linux-gnu/x86_64-cros-linux-gnu/binutils-bin/2.25.51/ld.gold.real: error: ../libs/libCommon.a(EfiUtilityMsgs.o): requires dynamic R_X86_64_32 reloc which may overflow at runtime; recompile with -fPIC
/usr/x86_64-pc-linux-gnu/x86_64-cros-linux-gnu/binutils-bin/2.25.51/ld.gold.real: error: ../libs/libCommon.a(ParseInf.o): requires dynamic R_X86_64_32 reloc which may overflow at runtime; recompile with -fPIC
collect2: error: ld returned 1 exit status
../Makefiles/app.makefile:24: recipe for target '../bin/GnuGenBootSector' failed
make[1]: *** [../bin/GnuGenBootSector] Error 1
 
GnuGenBootSector is a host tool which is supposed to be compiled by host cc. However, this CL calls cros_use_gcc (because clang doesn't work) which sets CC to target cc:
https://crrev.com/5c089b936d2e10aa8ea836bb01b046ba81cb38a7

The simplest way to fix it would be replacing "cros_use_gcc" by "unset CC CXX". Any objections?

host tools should be using a diff var than CC or CXX like BUILD_CC.  then the ebuild should be using `tc-export BUILD_CC` to set those up.  by unseting the vars, you're now breaking the target build too by letting it use the native compiler instead of the cross-compiler.

This CL was tested before being uploaded. The default 'base' profile doesn't set CC nor CXX. Unsetting them should only revert what's done in the 'llvm' profile and shouldn't break anything [1].

In fact, the package in question doesn't produce target executables nor libraries. I'm surprised that cros_use_gcc could satisfy the build tests.

[1] By the way, this might be a good example not to complicate CFLAGS in build profiles' make.conf, since host tools and target executables may require different flags.

because it "happens" to "work" by default does not make it correct or right.  using the target CC to build tools that run on the host is *never* correct.

there is a similar separation between build flags -- use BUILD_CFLAGS and BUILD_CPPFLAGS and BUILD_LDFLAGS and such rather than CFLAGS/etc...

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/third_party/intel-framework/+/dd835610fa3b1910b2c1d8ce12fb15e836574928

commit dd835610fa3b1910b2c1d8ce12fb15e836574928
Author: Liming Gao <liming.gao@intel.com>
Date: Mon Aug 22 06:28:50 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/third_party/intel-framework/+/dd835610fa3b1910b2c1d8ce12fb15e836574928

commit dd835610fa3b1910b2c1d8ce12fb15e836574928
Author: Liming Gao <liming.gao@intel.com>
Date: Mon Aug 22 06:28:50 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/third_party/intel-framework/+/2eea6f1615bfcf9c8613c69e807908c26c0aea06

commit 2eea6f1615bfcf9c8613c69e807908c26c0aea06
Author: Liming Gao <liming.gao@intel.com>
Date: Mon Aug 22 06:32:23 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/third_party/intel-framework/+/7f2a5930e64d2b1ea85d36ed5cd144c7ba228c3b

commit 7f2a5930e64d2b1ea85d36ed5cd144c7ba228c3b
Author: Liming Gao <liming.gao@intel.com>
Date: Wed Aug 31 04:44:50 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/third_party/intel-framework/+/f2ef5673a17c62a00487254b7d0640a42518b3e3

commit f2ef5673a17c62a00487254b7d0640a42518b3e3
Author: Liming Gao <liming.gao@intel.com>
Date: Wed Aug 31 04:46:17 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/third_party/intel-framework/+/6b169ba18806ba0af34990c786ce3bfb4483e544

commit 6b169ba18806ba0af34990c786ce3bfb4483e544
Author: Liming Gao <liming.gao@intel.com>
Date: Thu Sep 01 07:47:04 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/third_party/intel-framework/+/8910876670b6947cb8e3cd9ece25a868a582ec27

commit 8910876670b6947cb8e3cd9ece25a868a582ec27
Author: Liming Gao <liming.gao@intel.com>
Date: Thu Sep 01 07:59:57 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/overlays/chromeos-partner-overlay/+/f317dce762378754eedd8a4b99c41375c1fa0c87

commit f317dce762378754eedd8a4b99c41375c1fa0c87
Author: Ting-Yuan Huang <laszio@google.com>
Date: Wed Aug 10 19:22:15 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/overlays/chromeos-partner-overlay/+/f317dce762378754eedd8a4b99c41375c1fa0c87

commit f317dce762378754eedd8a4b99c41375c1fa0c87
Author: Ting-Yuan Huang <laszio@google.com>
Date: Wed Aug 10 19:22:15 2016