[Buildroot] [PATCH] toolchain: add a harmless link when sysroot detection is not accurate

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Fri Jan 22 13:15:47 UTC 2016


Dear Vicente Olivert Riera,

On Fri, 22 Jan 2016 12:14:22 +0000, Vicente Olivert Riera wrote:
> Sometimes is not possible to detect if the sysroots are nested or side
> by side. For instance this happens for MIPS big endian, where the
> sysroot and the arch-sysroot directories are the same.
> 
> For those cases where the sysroot detection is not accurate we add a
> harmless symlink necessary for certain toolchains.
> 
> Signed-off-by: Vicente Olivert Riera <Vincent.Riera at imgtec.com>

Gaah, even more magic in this code.

Can you update the comment above the function, and also in the commit
log explain which toolchain is affected, and give outputs of
-print-file-name that explain the problem ?

It is really important to document all those things, because we
progressively adjust this sorcery to accommodate all toolchains, but
when we'll want to refactor this stuff, it will be very useful to have
some explanation as to why each case was handled this way.

The more I look at this external toolchain mess, the more I think we
should have less magic, and simply for each toolchain have some
explicit code in BR that says "this toolchain is organized this way".
Yes, it means that we will no longer automagically accommodate to
custom external toolchains...

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com



More information about the buildroot mailing list