From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Fri, 20 Mar 2015 22:45:08 +0100 Subject: [Buildroot] [PATCH 1/2] bfin: remove the dependency of flat shared mode to shared library In-Reply-To: <1426669156-24250-1-git-send-email-sonic.adi@gmail.com> References: <1426669156-24250-1-git-send-email-sonic.adi@gmail.com> Message-ID: <20150320224508.73c0c84b@free-electrons.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Dear Sonic Zhang, On Wed, 18 Mar 2015 16:59:15 +0800, Sonic Zhang wrote: > From: Sonic Zhang > > Blackfin flat shared library isn't generated and linked in the same way > as the ELF shared library. So, don't mark it depend on the static library > only. > > Signed-off-by: Sonic Zhang > --- > arch/Config.in | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/Config.in b/arch/Config.in > index 16ad8be..6179960 100644 > --- a/arch/Config.in > +++ b/arch/Config.in > @@ -295,7 +295,6 @@ config BR2_BINFMT_FLAT_SEP_DATA > > config BR2_BINFMT_FLAT_SHARED > bool "Shared binary" > - select BR2_BINFMT_SUPPORTS_SHARED If you don't select BR2_BINFMT_SUPPORTS_SHARED, then the only available option amongst: static libs, static libs + shared libs, shared libs is "static libs", which means that the "-static" flag is always passed. If that's OK, then I'm fine with the patch. The second sentence of your commit log is confusing however. You say "don't mark it depend on the static library only". But it is exactly what your patch is doing: since it's no longer select BR2_BINFMT_SUPPORTS_SHARED, it means that after your patch is applied, when BR2_BINFMT_FLAT_SHARED is used, the *only* possibility is static library only. Could you clarify? Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com