[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#118138: seaview; Please build-depend on libforms-dev

tags 118138 + patch

Peter S Galbraith <psg@debian.org> wrote:
> As the Debian maintainer of the XForms, I'm asking you to build
> against version 0.89 (e.g. build-depend on libforms-dev) unless
> you have a very compelling reason.

One small fix needs to be made to seaview to get it to build with
xforms 0.89.  In the forms.h that came with xforms 0.88 TRUE and
FALSE were defined and seaview depends on these existing.  I have
made a small patch to define these in seaview's seaview.h if they
have not already been defined.

diff -Naur seaview-0.0.orig/debian/control seaview-0.0/debian/control
--- seaview-0.0.orig/debian/control	Thu Nov  8 17:25:10 2001
+++ seaview-0.0/debian/control	Thu Nov  8 16:52:50 2001
@@ -2,7 +2,7 @@
 Section: contrib/misc
 Priority: optional
 Maintainer: Debian QA Group <packages@qa.debian.org>
-Build-Depends: debhelper (>> 2.0.0), libforms0.88-dev
+Build-Depends: debhelper (>> 2.0.0), libforms-dev
 Standards-Version: 3.5.2
 Package: seaview
diff -Naur seaview-0.0.orig/seaview.h seaview-0.0/seaview.h
--- seaview-0.0.orig/seaview.h	Wed Sep 30 08:36:29 1998
+++ seaview-0.0/seaview.h	Thu Nov  8 17:17:02 2001
@@ -2,6 +2,15 @@
 #define SEAVIEW_H
 #include "forms.h"
+/* The forms.h from XForms 0.88 defined TRUE and FALSE; later versions
+ * did not and so we will define them if necessary.
+ */
+#ifndef FALSE
+#define FALSE	0
+#define TRUE	1
 #define fl_drw_text_cursor(a,x,y,w,h,c,st,si,t,cc,p)  \

Since this package is maintained by the QA group I'll NMU it in
the next day or two unless someone has something else to point
out.  Thanks.

Doug Porter <dsp@waterspout.com>

Reply to: