[lmi-commits] [lmi] master 8b8a660 3/5: Augment 'show_env' makefile target, exposing a defect

[Greg Chicares]

branch: master
commit 8b8a6608da8cfa27bf61a6fce3d3b0568090a1ed
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Augment 'show_env' makefile target, exposing a defect
    
    Running
      make check_concinnity
    in a new terminal fails. Running
      make show_env
    shows one symptom--even though these variables are set as intended:
      LMI_COMPILER    = "gcc"
      LMI_TRIPLET     = "i686-w64-mingw32"
    this one
      exec_prefix     = "/opt/lmi/_"
    doesn't have the value expected from its definition:
      exec_prefix     := $(prefix)/$(LMI_COMPILER)_$(LMI_TRIPLET)
    because the $(LMI_COMPILER) and $(LMI_TRIPLET) definitions are not in
    effect when $(exec_prefix) is set. Recursive expansion, i.e.:
    -  exec_prefix     := $(prefix)/$(LMI_COMPILER)_$(LMI_TRIPLET)
    +  exec_prefix     = $(prefix)/$(LMI_COMPILER)_$(LMI_TRIPLET)
    would fix that symptom, but wouldn't make 'check_concinnity' work.
    
    Added $(EXEEXT) and $(SHREXT) to the list of environmental variables
    displayed, so that
      make show_env
    shows:
      EXEEXT          = ""
      SHREXT          = ".so"
    demonstrating that $(LMI_COMPILER) and $(LMI_TRIPLET) are also not set
    when 'configuration.make' is included.
    
    Running
      . /opt/lmi/src/lmi/set_toolchain.sh
    whenever a new terminal is started works around this problem, but it is
    better to resolve the defect.
---
 GNUmakefile | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/GNUmakefile b/GNUmakefile
index f455bf9..4d99b2f 100644
--- a/GNUmakefile
+++ b/GNUmakefile
@@ -269,6 +269,8 @@ show_env:
        @printf 'localbindir     = "%s"\n' "$(localbindir)"
        @printf 'locallibdir     = "%s"\n' "$(locallibdir)"
        @printf 'localincludedir = "%s"\n' "$(localincludedir)"
+       @printf 'EXEEXT          = "%s"\n' "$(EXEEXT)"
+       @printf 'SHREXT          = "%s"\n' "$(SHREXT)"
 
 
################################################################################