The following commit has been merged in the master branch:
commit d4aa059cdddb497cc4d33c25702f5cdcd265fc27
Author: Stephane Glondu <st...@glondu.net>
Date:   Fri Jul 15 18:47:48 2011 +0200

    Yet another portability issue: assume char is signed by default

diff --git a/debian/patches/0005-Compile-C-stubs-with-fsigned-char.patch 
b/debian/patches/0005-Compile-C-stubs-with-fsigned-char.patch
new file mode 100644
index 0000000..4f38dd5
--- /dev/null
+++ b/debian/patches/0005-Compile-C-stubs-with-fsigned-char.patch
@@ -0,0 +1,24 @@
+From: Stephane Glondu <st...@glondu.net>
+Date: Fri, 15 Jul 2011 18:45:54 +0200
+Subject: Compile C stubs with -fsigned-char
+
+They rely on char being signed by default. They probably should be
+fixed instead...
+---
+ myocamlbuild.ml |    2 ++
+ 1 files changed, 2 insertions(+), 0 deletions(-)
+
+diff --git a/myocamlbuild.ml b/myocamlbuild.ml
+index 0f1653d..51d157d 100644
+--- a/myocamlbuild.ml
++++ b/myocamlbuild.ml
+@@ -494,6 +494,8 @@ Ocamlbuild_plugin.dispatch
+ 
+           flag ["compile";"ocaml"; "cpp"] cpp;
+ 
++          flag ["compile"; "c"] (S[A"-ccopt"; A"-fsigned-char"]);
++
+           dispatch_default e
+       | e ->
+           dispatch_default e
+-- 
diff --git a/debian/patches/series b/debian/patches/series
index 7395b4e..4bd718e 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -2,3 +2,4 @@
 0002-Use-cpp-as-default-preprocessor-on-unknown-OS.patch
 0003-Fix-alignment.patch
 0004-Fix-endianness.patch
+0005-Compile-C-stubs-with-fsigned-char.patch

-- 
bin-prot packaging

_______________________________________________
Pkg-ocaml-maint-commits mailing list
Pkg-ocaml-maint-commits@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-ocaml-maint-commits

Reply via email to