diff mbox

[v2,2/3] gcc: consolidate fortran options

Message ID 20160318202342.GA23907@spaceflight.com
State Superseded
Headers show

Commit Message

Benjamin Kamath March 18, 2016, 8:23 p.m. UTC
Have BR2_TOOLCHAIN_BUILDROOT_FORTRAN select BR2_INSTALL_LIBFORTAN,
allowing packages and other dependencies to generically check
for fortran support with buildroot and external toolchains.

Signed-off-by: Benjamin Kamath <bkamath@spaceflight.com>
---
 package/gcc/Config.in.host         | 1 +
 package/gcc/gcc-final/gcc-final.mk | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

Comments

Samuel Martin July 1, 2016, 4:40 p.m. UTC | #1
Hi Benjamin,

On Fri, Mar 18, 2016 at 9:23 PM, Benjamin Kamath <kamath.ben@gmail.com> wrote:
> Have BR2_TOOLCHAIN_BUILDROOT_FORTRAN select BR2_INSTALL_LIBFORTAN,
> allowing packages and other dependencies to generically check
> for fortran support with buildroot and external toolchains.
>
> Signed-off-by: Benjamin Kamath <bkamath@spaceflight.com>

I have just reposted an updated of the fortran series which includes
your changes [1].
So, these patches (from May) will be marked as superseded in the
Buildroot patchwork.

[1] http://lists.busybox.net/pipermail/buildroot/2016-July/165662.html

Regards,
diff mbox

Patch

diff --git a/package/gcc/Config.in.host b/package/gcc/Config.in.host
index 4845708..038a0a4 100644
--- a/package/gcc/Config.in.host
+++ b/package/gcc/Config.in.host
@@ -111,6 +111,7 @@  config BR2_TOOLCHAIN_BUILDROOT_CXX
 
 config BR2_TOOLCHAIN_BUILDROOT_FORTRAN
 	bool "Enable Fortran support"
+	select BR2_INSTALL_LIBFORTRAN
 	help
 	  Enable this option if you want your toolchain to support the
 	  Fortran language and you want Fortran libraries to be
diff --git a/package/gcc/gcc-final/gcc-final.mk b/package/gcc/gcc-final/gcc-final.mk
index 69fdc0c..87480c7 100644
--- a/package/gcc/gcc-final/gcc-final.mk
+++ b/package/gcc/gcc-final/gcc-final.mk
@@ -54,7 +54,7 @@  endef
 # Languages supported by the cross-compiler
 GCC_FINAL_CROSS_LANGUAGES-y = c
 GCC_FINAL_CROSS_LANGUAGES-$(BR2_INSTALL_LIBSTDCPP) += c++
-GCC_FINAL_CROSS_LANGUAGES-$(BR2_TOOLCHAIN_BUILDROOT_FORTRAN) += fortran
+GCC_FINAL_CROSS_LANGUAGES-$(BR2_INSTALL_LIBFORTRAN) += fortran
 GCC_FINAL_CROSS_LANGUAGES = $(subst $(space),$(comma),$(GCC_FINAL_CROSS_LANGUAGES-y))
 
 HOST_GCC_FINAL_CONF_OPTS = \
@@ -152,7 +152,7 @@  ifeq ($(BR2_INSTALL_LIBSTDCPP),y)
 HOST_GCC_FINAL_USR_LIBS += libstdc++
 endif
 
-ifeq ($(BR2_TOOLCHAIN_BUILDROOT_FORTRAN),y)
+ifeq ($(BR2_INSTALL_LIBFORTRAN),y)
 HOST_GCC_FINAL_USR_LIBS += libgfortran
 endif