Bug#521674: FTBFS: "Def.syn" is not a valid module name.

2009-03-29 Thread Christian Perrier
Quoting Stéphane Glondu (st...@glondu.net):
> tags 521674 + upstream patch
> thanks

Thanks for the patch.

> 
> Stéphane Glondu a écrit :
> > You can workaround this failure by adding "-w x" to the failing command
> > line. It will disable the warning (which is an error because of
> > -warn-error). You can also remove -warn-error (your call).
> 
> I can see in the changelog entry of 4.10-15 that "-warn-error A" has
> already been removed in the past for the same reason. I suggest to do
> the same this time.
> 
> > The proper solution would be to ask upstream to rename the file into a
> > proper module name.
> 
> This still applies.


I notified Daniel this morning when I discovered this bug. I'll see
what is his reaction.

I would very much prefer seeing this fixed upstream and only have to
use the workaround if Daniel doesn't want to fix the issue upstream.




signature.asc
Description: Digital signature


Bug#521674: FTBFS: "Def.syn" is not a valid module name.

2009-03-29 Thread Stéphane Glondu
tags 521674 + upstream patch
thanks

Stéphane Glondu a écrit :
> You can workaround this failure by adding "-w x" to the failing command
> line. It will disable the warning (which is an error because of
> -warn-error). You can also remove -warn-error (your call).

I can see in the changelog entry of 4.10-15 that "-warn-error A" has
already been removed in the past for the same reason. I suggest to do
the same this time.

> The proper solution would be to ask upstream to rename the file into a
> proper module name.

This still applies.


Cheers,

-- 
Stéphane
diff -u geneweb-5.01/debian/changelog geneweb-5.01/debian/changelog
--- geneweb-5.01/debian/changelog
+++ geneweb-5.01/debian/changelog
@@ -1,3 +1,10 @@
+geneweb (5.01-8.1) UNRELEASED; urgency=low
+
+  * Non-maintainer upload
+  * Add patch 014_no-warn-error (Closes: #521674)
+
+ -- Stephane Glondu   Sun, 29 Mar 2009 14:03:15 +0200
+
 geneweb (5.01-8) unstable; urgency=low
 
   * Add "-bindir" to gwsetup's invocation in the wrapper script.
@@ -2372,2 +2378,0 @@
-
-
diff -u geneweb-5.01/debian/patches/series geneweb-5.01/debian/patches/series
--- geneweb-5.01/debian/patches/series
+++ geneweb-5.01/debian/patches/series
@@ -7,0 +8 @@
+014_no-warn-error
only in patch2:
unchanged:
--- geneweb-5.01.orig/debian/patches/014_no-warn-error
+++ geneweb-5.01/debian/patches/014_no-warn-error
@@ -0,0 +1,66 @@
+commit 04f584f0e3d161eec6b566da133f6925f2942d7d
+Author: Stephane Glondu 
+Date:   Sun Mar 29 14:00:48 2009 +0200
+
+Remove "-warn-error A" to work around FTBFS (Closes: #521674)
+
+diff --git a/tools/Makefile.inc.opt b/tools/Makefile.inc.opt
+index 5f7aeb4..594c1f9 100644
+--- a/tools/Makefile.inc.opt
 b/tools/Makefile.inc.opt
+@@ -5,8 +5,8 @@ CAMLP5_COMM=TOP=$(TOP) $(TOP)/tools/camlp5_comm.sh
+ CAMLP5F=-DUNIX
+ CAMLP5D=+camlp5
+ LIBUNIX=-cclib -lunix
+-OCAMLC=ocamlc.opt -warn-error A
+-OCAMLOPT=ocamlopt.opt -warn-error A
++OCAMLC=ocamlc.opt
++OCAMLOPT=ocamlopt.opt
+ OCAMLI=
+ OCAMLD=
+ STRIP=strip
+diff --git a/tools/Makefile.inc.out b/tools/Makefile.inc.out
+index 0102b96..bec0364 100644
+--- a/tools/Makefile.inc.out
 b/tools/Makefile.inc.out
+@@ -5,8 +5,8 @@ CAMLP5_COMM=TOP=$(TOP) $(TOP)/tools/camlp5_comm.sh
+ CAMLP5F=-DUNIX
+ CAMLP5D=+camlp5
+ LIBUNIX=-cclib -lunix
+-OCAMLC=ocamlc -warn-error A
+-OCAMLOPT=ocamlopt -warn-error A
++OCAMLC=ocamlc
++OCAMLOPT=ocamlopt
+ OCAMLI=
+ OCAMLD=
+ STRIP=strip
+diff --git a/tools/Makefile.inc.unix b/tools/Makefile.inc.unix
+index 5f7aeb4..594c1f9 100644
+--- a/tools/Makefile.inc.unix
 b/tools/Makefile.inc.unix
+@@ -5,8 +5,8 @@ CAMLP5_COMM=TOP=$(TOP) $(TOP)/tools/camlp5_comm.sh
+ CAMLP5F=-DUNIX
+ CAMLP5D=+camlp5
+ LIBUNIX=-cclib -lunix
+-OCAMLC=ocamlc.opt -warn-error A
+-OCAMLOPT=ocamlopt.opt -warn-error A
++OCAMLC=ocamlc.opt
++OCAMLOPT=ocamlopt.opt
+ OCAMLI=
+ OCAMLD=
+ STRIP=strip
+diff --git a/tools/Makefile.inc.win b/tools/Makefile.inc.win
+index 38db249..edc61cc 100644
+--- a/tools/Makefile.inc.win
 b/tools/Makefile.inc.win
+@@ -5,8 +5,8 @@ CAMLP5_COMM=TOP=$(TOP) $(TOP)/tools/camlp5_comm.sh
+ CAMLP5F=-DWIN95
+ CAMLP5D=+camlp5
+ LIBUNIX=-cclib -lunix
+-OCAMLC=ocamlc.opt -warn-error A
+-OCAMLOPT=ocamlopt.opt -warn-error A
++OCAMLC=ocamlc.opt
++OCAMLOPT=ocamlopt.opt
+ OCAMLI=
+ OCAMLD=
+ STRIP=true


Bug#521674: FTBFS: "Def.syn" is not a valid module name.

2009-03-29 Thread Stéphane Glondu
Christian Perrier a écrit :
> Trying to recompile the geneweb package in sid gives:
> 
> camlp5r pa_extend.cmo q_MLast.cmo pa_macro.cmo -DUNIX -o def.syn.ppo
> def.syn.ml
> ocamlc.opt -warn-error A -I ../wserver -I ../dag2html  -I +camlp5 -c -impl
> def.syn.ppo
> File "def.syn.ppo", line 1, characters 0-1:
> Warning X: bad source file name: "Def.syn" is not a valid module name.
> File "def.syn.ml", line 1, characters 0-1:
> Error: Error-enabled warnings (1 occurrences)

You can workaround this failure by adding "-w x" to the failing command
line. It will disable the warning (which is an error because of
-warn-error). You can also remove -warn-error (your call).

The proper solution would be to ask upstream to rename the file into a
proper module name.


Cheers,

-- 
Stéphane





--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org



Bug#521674: FTBFS: "Def.syn" is not a valid module name.

2009-03-29 Thread Christian Perrier
Package: geneweb
Version: 5.01-8
Severity: serious
Justification: Fails to build from source

Trying to recompile the geneweb package in sid gives:

camlp5r pa_extend.cmo q_MLast.cmo pa_macro.cmo -DUNIX -o def.syn.ppo
def.syn.ml
ocamlc.opt -warn-error A -I ../wserver -I ../dag2html  -I +camlp5 -c -impl
def.syn.ppo
File "def.syn.ppo", line 1, characters 0-1:
Warning X: bad source file name: "Def.syn" is not a valid module name.
File "def.syn.ml", line 1, characters 0-1:
Error: Error-enabled warnings (1 occurrences)



-- System Information:
Debian Release: squeeze/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)

Kernel: Linux 2.6.26-1-686 (SMP w/1 CPU core)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages geneweb depends on:
ii  adduser   3.110  add and remove users and groups
ii  debconf [debconf-2.0] 1.5.26 Debian configuration management sy
ii  iso-codes 3.7-1  ISO language, territory, currency,
ii  libc6 2.9-6  GNU C Library: Shared libraries
ii  lsb-base  3.2-22 Linux Standard Base 3.2 init scrip
ii  perl-base [perl5-base]5.10.0-19  minimal Perl system

geneweb recommends no packages.

Versions of packages geneweb suggests:
ii  gwsetup5.01-8utilities to configure and manipul
ii  gwtp   5.01-8web interface interacting with Gen
ii  iceweasel [www-browser 3.0.7-1   lightweight web browser based on M
ii  konqueror [www-browser 4:3.5.10.dfsg.1-2 KDE's advanced file manager, web b
ii  lynx   2.8.7pre1-1   Text-mode WWW Browser (transitiona
ii  lynx-cur [www-browser] 2.8.7pre1-1   Text-mode WWW Browser with NLS sup
ii  menu   2.1.41generates programs menu for all me
ii  w3m [www-browser]  0.5.2-2+b1WWW browsable pager with excellent
ii  xemacs21-mule [www-bro 21.4.22-1 highly customizable text editor --
ii  xemacs21-nomule [www-b 21.4.22-1 highly customizable text editor --

-- debconf information excluded



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org