<div dir="ltr">Hi Thomas,<br><div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Apr 17, 2013 at 4:56 PM, Thomas Petazzoni <span dir="ltr"><<a href="mailto:thomas.petazzoni@free-electrons.com" target="_blank">thomas.petazzoni@free-electrons.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Dear Sonic Zhang,<br>
<div class="im"><br>
On Wed, 17 Apr 2013 16:16:21 +0800, Sonic Zhang wrote:<br>
<br>
> > # When the FDPIC binary format is used, we force the installation<br>
> > # of the corresponding libraries. When a different binary format is<br>
> > # used, we offer the option of installing the FDPIC shared libraries<br>
> > # (typically in addition to the FLAT shared ones).<br>
> > config BR2_BFIN_INSTALL_FDPIC_SHARED<br>
> >         bool "Install FDPIC shared libraries" if !BR2_BINFMT_FDPIC<br>
> >         default y if BR2_BINFMT_FDPIC<br>
> >         help<br>
> >           ...<br>
><br>
> Because current library installation scripts in<br>
> toolchain-external/<a href="http://ext-tool.mk" target="_blank">ext-tool.mk</a> always install FDPIC libraries when<br>
> BINFMT_FDPIC is selected. No necessary to set<br>
> BR2_BFIN_INSTALL_FDPIC_SHARED y in this case.<br>
<br>
</div>Ok. But then why do we have to duplicate this code in <a href="http://ext-tool.mk" target="_blank">ext-tool.mk</a>. If<br>
it already has the code to copy the FDPIC libraries when FDPIC is<br>
selected as the primary binary format, we should be able to re-use this<br>
code when FLAT shared is used as the primary binary format, but we want<br>
the FDPIC libraries to be installed in addition.<br></blockquote></div><br></div><div class="gmail_extra">Yes, I do reuse the function copy_toolchain_lib_root() in the code to install shared FDPIC libraries when FLAT format is selected. But, rest part of the library installation code depends on current TOOLCHAIN_EXTERNAL_CC, which is different from the FDPIC one in this case. And the following sysroot to staging installation code depends on macro SYSROOT_DIR, ARCH_SYSROOT_DIR, ARCH_SUBDIR, ARCH_LIB_DIR and SUPPORT_LIB_DIR, which are based on current TOOLCHAIN_EXTERNAL_CC as well.<br>
<br></div><div class="gmail_extra">I feel the individual FDPIC installation code for FLAT format is a simple and clear solution.<br><br></div><div class="gmail_extra">Regards,<br><br>Sonic<br></div></div></div>