Package: fairymax
Severity: minor
Usertags: clang-ftbfs
User: pkg-llvm-t...@lists.alioth.debian.org
Tag: patch


Hello,

Using the rebuild infrastructure, your package fails to build with clang 
(instead of gcc).

Thanks,
Arthur
diff -Naur fairymax.orig/fairymax-4.8q/debian/changelog fairymax/fairymax-4.8q/debian/changelog 
--- fairymax.orig/fairymax-4.8q/debian/changelog	2014-03-16 15:53:53.427410944 -0500
+++ fairymax/fairymax-4.8q/debian/changelog	2014-03-29 18:08:25.149987219 -0500
@@ -1,3 +1,13 @@
+fairymax (4.8q-3) unstable; urgency=low
+
+  * Fix FTBFS with clang
+    - Fixed the non-void function should return a value in
+      fairymax.c
+      maxqi.c
+    - Whenever there was a return; I replaced it with return 0;
+
+ -- Arthur Marble <art...@info9.net>  Sat, 29 Mar 2014 18:08:25 -0500
+
 fairymax (4.8q-2) unstable; urgency=low
 
   * Upload to unstable
diff -Naur fairymax.orig/fairymax-4.8q/debian/patches/clang-ftbfs.diff fairymax/fairymax-4.8q/debian/patches/clang-ftbfs.diff 
--- fairymax.orig/fairymax-4.8q/debian/patches/clang-ftbfs.diff	1969-12-31 18:00:00.000000000 -0600
+++ fairymax/fairymax-4.8q/debian/patches/clang-ftbfs.diff	2014-03-16 15:58:15.043415500 -0500
@@ -0,0 +1,58 @@
+--- a/fairymax.c
++++ b/fairymax.c
+@@ -473,7 +473,7 @@ int LoadGame(char *name)
+                if(c == EOF) {
+                    printf("telluser variant %s not supported\n", name);
+                    fclose(f);
+-                   return; /* keep old settings */
++                   return 0; /* keep old settings */
+                }
+            }
+            currentVariant = count;
+@@ -596,7 +596,7 @@ int main(int argc, char **argv)
+                         continue;
+ 		}
+ 		if (!fgets(line, 256, stdin))
+-			return;
++			return 0;
+ 		if (line[0] == '\n')
+ 			continue;
+ 		sscanf(line, "%s", command);
+@@ -648,7 +648,7 @@ int main(int argc, char **argv)
+ 		}
+ 		if (!strcmp(command, "quit"))
+                         /* exit engine */
+-			return;
++			return 0;
+ 		if (!strcmp(command, "force")) {
+                         /* computer plays neither */
+                         Computer = EMPTY;
+--- a/maxqi.c
++++ b/maxqi.c
+@@ -422,7 +422,7 @@ int LoadGame(char *name)
+                if(c == EOF) {
+                    printf("telluser variant %s not supported\n", name);
+                    fclose(f);
+-                   return; /* keep old settings */
++                   return 0; /* keep old settings */
+                }
+            }
+            currentVariant = count;
+@@ -531,7 +531,7 @@ int main(int argc, char **argv)
+                         continue;
+ 		}
+ 		if (!fgets(line, 256, stdin))
+-			return;
++			return 0;
+ 		if (line[0] == '\n')
+ 			continue;
+ 		sscanf(line, "%s", command);
+@@ -583,7 +583,7 @@ int main(int argc, char **argv)
+ 		}
+ 		if (!strcmp(command, "quit"))
+                         /* exit engine */
+-			return;
++			return 0;
+ 		if (!strcmp(command, "force")) {
+                         /* computer plays neither */
+                         Computer = EMPTY;
diff -Naur fairymax.orig/fairymax-4.8q/debian/patches/series fairymax/fairymax-4.8q/debian/patches/series 
--- fairymax.orig/fairymax-4.8q/debian/patches/series	1969-12-31 18:00:00.000000000 -0600
+++ fairymax/fairymax-4.8q/debian/patches/series	2014-03-16 15:54:09.487411224 -0500
@@ -0,0 +1 @@
+clang-ftbfs.diff

Reply via email to