Providing --with-ocaml-libdir for ./configure. The default is the

stdlib if it's beneath --prefix, and is libdir/ocaml otherwise.

If someone has a better way than this to test whether $B is a path
within $A, I'd love to hear it:

  if test "$A" \< "$B" -a "$B" \< "${A}~"

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@42532 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/Makefile.config.in b/Makefile.config.in
index 778c4e867e48ce9b51c5e5dfac08d90f0f4ed56b..5522232706fb1e9e96dc692a3680bcb4101416d4 100644 (file)
--- a/Makefile.config.in
+++ b/Makefile.config.in
@@ -261,6 +261,7 @@ NOLINKALL := @NOLINKALL@
 HUGE_VAL_SANITY = @HUGE_VAL_SANITY@
 
 # Bindings that we should build
 HUGE_VAL_SANITY = @HUGE_VAL_SANITY@
 
 # Bindings that we should build

-BINDINGS_TO_BUILD = @BINDINGS_TO_BUILD@
-ALL_BINDINGS = @ALL_BINDINGS@
+BINDINGS_TO_BUILD := @BINDINGS_TO_BUILD@
+ALL_BINDINGS      := @ALL_BINDINGS@
+OCAML_LIBDIR      := @OCAML_LIBDIR@

 
 

diff --git a/autoconf/configure.ac b/autoconf/configure.ac
index 9bf2d0e4332abba16df2b94e233c5af18c6df8f2..610d8f903cdd24a170748ba96ab8e61e80bcf45b 100644 (file)
--- a/autoconf/configure.ac
+++ b/autoconf/configure.ac
@@ -450,6 +450,20 @@ case "$enableval" in
   ;;
 esac
 
   ;;
 esac
 

+dnl Allow the ocaml libdir to be overridden. This could go in a configure
+dnl script for bindings/ocaml/configure, except that its auto value depends on
+dnl OCAMLC, which is found here to support tests.
+AC_ARG_WITH([ocaml-libdir],
+  [AS_HELP_STRING([--with-ocaml-libdir],
+    [Specify install location for ocaml bindings (default is stdlib)])],
+  [],
+  [withval=auto])
+case "$withval" in
+  auto) with_ocaml_libdir="$withval" ;;
+  /* | [[A-Za-z]]:[[\\/]]*) with_ocaml_libdir="$withval" ;;
+  *) AC_MSG_ERROR([Invalid path for --with-ocaml-libdir. Provide full path]) ;;
+esac
+

 dnl===-----------------------------------------------------------------------===
 dnl===
 dnl=== SECTION 4: Check for programs we need and that they are the right version
 dnl===-----------------------------------------------------------------------===
 dnl===
 dnl=== SECTION 4: Check for programs we need and that they are the right version


@@ -886,6 +900,44 @@ AC_SUBST(BINDINGS_TO_BUILD,$BINDINGS_TO_BUILD)
 # other files.
 AC_SUBST(ALL_BINDINGS,ocaml)
 
 # other files.
 AC_SUBST(ALL_BINDINGS,ocaml)
 

+# Do any work necessary to ensure that bindings have what they need.
+binding_prereqs_failed=0
+for a_binding in $BINDINGS_TO_BUILD ; do
+  case "$a_binding" in
+  ocaml)
+    if test "x$OCAMLC" = x ; then
+      AC_MSG_WARN([--enable-bindings=ocaml specified, but ocamlc not found. Try configure OCAMLC=/path/to/ocamlc])
+      binding_prereqs_failed=1
+    fi
+    if test "x$OCAMLDEP" = x ; then
+      AC_MSG_WARN([--enable-bindings=ocaml specified, but ocamldep not found. Try configure OCAMLDEP=/path/to/ocamldep])
+      binding_prereqs_failed=1
+    fi
+    if test "x$OCAMLOPT" = x ; then
+      AC_MSG_WARN([--enable-bindings=ocaml specified, but ocamlopt not found. Try configure OCAMLOPT=/path/to/ocamlopt])
+      dnl ocamlopt is optional! 
+    fi
+    if test "x$with_ocaml_libdir" != xauto ; then
+      AC_SUBST(OCAML_LIBDIR,$with_ocaml_libdir)
+    else
+      ocaml_stdlib="`"$OCAMLC" -where`"
+      if test "$LLVM_PREFIX" '<' "$ocaml_stdlib" -a "$ocaml_stdlib" '<' "$LLVM_PREFIX~"
+      then
+        # ocaml stdlib is beneath our prefix; use stdlib
+        AC_SUBST(OCAML_LIBDIR,$ocaml_stdlib)
+      else
+        # ocaml stdlib is outside our prefix; use libdir/ocaml
+        AC_SUBST(OCAML_LIBDIR,$LLVM_LIBDIR/ocaml)
+      fi
+    fi
+    ;;
+  esac
+done
+if test "$binding_prereqs_failed" = 1 ; then
+  AC_MSG_ERROR([Prequisites for bindings not satisfied. Fix them or use configure --disable-bindings.])
+fi
+
+

 dnl===-----------------------------------------------------------------------===
 dnl===
 dnl=== SECTION 10: Specify the output files and generate it
 dnl===-----------------------------------------------------------------------===
 dnl===
 dnl=== SECTION 10: Specify the output files and generate it

diff --git a/bindings/ocaml/Makefile.ocaml b/bindings/ocaml/Makefile.ocaml
index 8ad085ecad3e1cf920099e7200bcdcb6dd3c4271..b157c3194eb64d0a4f81b0d2e60fb900453c0632 100644 (file)
--- a/bindings/ocaml/Makefile.ocaml
+++ b/bindings/ocaml/Makefile.ocaml
@@ -14,12 +14,6 @@
 
 include $(LEVEL)/Makefile.config
 
 
 include $(LEVEL)/Makefile.config
 

-# Find the ocaml stdlib root. /usr/local/lib/ocaml is the default when built
-# from source; distros use something like /usr/lib/ocaml/3.10.0.
-ifndef OCAML_LIBDIR
-OCAML_LIBDIR := $(shell $(OCAMLC) -where)
-endif
-

 # CFLAGS needs to be set before Makefile.rules is included. Yes, ocaml puts its
 # includes under its libdir.
 CFLAGS += -I$(OCAML_LIBDIR)
 # CFLAGS needs to be set before Makefile.rules is included. Yes, ocaml puts its
 # includes under its libdir.
 CFLAGS += -I$(OCAML_LIBDIR)


@@ -27,7 +21,7 @@ CFLAGS += -I$(OCAML_LIBDIR)
 include $(LEVEL)/Makefile.common
 
 # Intentionally ignore PROJ_prefix here. We want the ocaml stdlib. However, the
 include $(LEVEL)/Makefile.common
 
 # Intentionally ignore PROJ_prefix here. We want the ocaml stdlib. However, the

-# user can override this with OCAML_LIBDIR.
+# user can override this with OCAML_LIBDIR or configure --with-ocaml-libdir=.

 PROJ_libocamldir := $(DESTDIR)$(OCAML_LIBDIR)
 OcamlDir := $(LibDir)/ocaml
 
 PROJ_libocamldir := $(DESTDIR)$(OCAML_LIBDIR)
 OcamlDir := $(LibDir)/ocaml
 

diff --git a/bindings/ocaml/bitwriter/llvm_bitwriter.mli b/bindings/ocaml/bitwriter/llvm_bitwriter.mli
index 3b7c6d49589af1437cd66ccbc2b43e286017efb1..08e001bc91024a3e2a3c0bdb6c7798b7773ade4e 100644 (file)
--- a/bindings/ocaml/bitwriter/llvm_bitwriter.mli
+++ b/bindings/ocaml/bitwriter/llvm_bitwriter.mli
@@ -8,7 +8,7 @@
  *===----------------------------------------------------------------------===
  *
  * This interface provides an ocaml API for the LLVM bitcode writer, the
  *===----------------------------------------------------------------------===
  *
  * This interface provides an ocaml API for the LLVM bitcode writer, the

- * classes in the classes in the Bitwriter library.
+ * classes in the Bitwriter library.

  *
  *===----------------------------------------------------------------------===*)
 
  *
  *===----------------------------------------------------------------------===*)