[Pkg-ocaml-maint-commits] [SCM] coq packaging branch, experimental/master, updated. debian/8.4dfsg-1-2-geabf57d

Stephane Glondu steph at glondu.net
Sat Sep 22 11:23:25 UTC 2012


The following commit has been merged in the experimental/master branch:
commit dcdaa8517bfaedcf4ce97380d39011d976fc1b64
Author: Stephane Glondu <steph at glondu.net>
Date:   Sat Sep 22 12:31:59 2012 +0200

    Upstream bugfix: Fix use of $(HASNATDYNLINK) in coq_makefile output

diff --git a/debian/patches/0002-Fix-use-of-HASNATDYNLINK-in-coq_makefile-output.patch b/debian/patches/0002-Fix-use-of-HASNATDYNLINK-in-coq_makefile-output.patch
new file mode 100644
index 0000000..8e9b9ee
--- /dev/null
+++ b/debian/patches/0002-Fix-use-of-HASNATDYNLINK-in-coq_makefile-output.patch
@@ -0,0 +1,46 @@
+From: Stephane Glondu <steph at glondu.net>
+Date: Sat, 22 Sep 2012 12:24:42 +0200
+Subject: Fix use of $(HASNATDYNLINK) in coq_makefile output
+
+Generated makefiles were broken because
+  $(if ifeq '$(HASNATDYNLINK)' 'true',X)
+always returns X.
+---
+ tools/coq_makefile.ml |   10 +++++++---
+ 1 file changed, 7 insertions(+), 3 deletions(-)
+
+diff --git a/tools/coq_makefile.ml b/tools/coq_makefile.ml
+index eedbf42..bc38136 100644
+--- a/tools/coq_makefile.ml
++++ b/tools/coq_makefile.ml
+@@ -222,7 +222,7 @@ let install (vfiles,(mlifiles,ml4files,mlfiles,mllibfiles,mlpackfiles),_,sds) in
+       print "\n";
+     end;
+     print "install:";
+-    if (not_empty cmxsfiles) then print "$(if ifeq '$(HASNATDYNLINK)' 'true',install-natdynlink)";
++    if (not_empty cmxsfiles) then print "$(if $(HASNATDYNLINK_OR_EMPTY),install-natdynlink)";
+     print "\n";
+     if not_empty vfiles then install_include_by_root "VOFILES" vfiles inc;
+     if (not_empty cmofiles) then
+@@ -543,14 +543,18 @@ let main_targets vfiles (mlifiles,ml4files,mlfiles,mllibfiles,mlpackfiles) other
+       print "CMXSFILES=$(CMXFILES:.cmx=.cmxs) $(CMXAFILES:.cmxa=.cmxs)\n";
+       classify_files_by_root "CMXSFILES" (l1 at l2) inc;
+   end;
+-  print "\n";
++  print "ifeq '$(HASNATDYNLINK)' 'true'\n";
++  print "HASNATDYNLINK_OR_EMPTY := yes\n";
++  print "else\n";
++  print "HASNATDYNLINK_OR_EMPTY :=\n";
++  print "endif\n\n";
+   section "Definition of the toplevel targets.";
+   print "all: ";
+   if !some_vfile then print "$(VOFILES) ";
+   if !some_mlfile || !some_ml4file || !some_mlpackfile then print "$(CMOFILES) ";
+   if !some_mllibfile then print "$(CMAFILES) ";
+   if !some_mlfile || !some_ml4file || !some_mllibfile || !some_mlpackfile
+-  then print "$(if ifeq '$(HASNATDYNLINK)' 'true',$(CMXSFILES)) ";
++  then print "$(if $(HASNATDYNLINK_OR_EMPTY),$(CMXSFILES)) ";
+   print_list "\\\n  " other_targets; print "\n\n";
+   if !some_mlifile then
+     begin
+-- 
diff --git a/debian/patches/series b/debian/patches/series
index 53d51a1..f1d97c9 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1 +1,2 @@
 0001-test-suite-success-Nsatz.v-comment-out-Ceva.patch
+0002-Fix-use-of-HASNATDYNLINK-in-coq_makefile-output.patch

-- 
coq packaging



More information about the Pkg-ocaml-maint-commits mailing list