[Uisp-dev] building uisp on OS X

[Jake McGuire]

( i hope this isn't a double-post )

I'm trying to build uisp on OS X (planning to run on a PowerBook using 
an USB->RS-232 dongle).

I ran into a couple of issues:

bootstrap: wants exactly automake version 1.4 and autoconf version 2.13. 
I have automake 1.6 and autoconf 2.5 - modifying the checks in boostrap 
so that they accept my versions didn't create any obvious difficulties. 
I could install back-level tools, but is there any reason these checks 
can't be made a bit more lenient?

src/DAPA.C:  the logic around NO_DIRECT_IO is a bit backwards.  If you 
use -DNO_DIRECT_IO, the macros ioport_* are not defined.

src/DAPA.C: the macro par_release sometimes gets defined to (0).  This 
generates a warning for "statement has no effect" which is treated as an 
error because the par_release is called without looking at the return 
value.  If the return value of par_release is not going to be checked, 
it should probably be defined to nothing.

proposed DAPA.C diffs:

[localhost:~/uisp/src] jake% cvs diff -ub DAPA.C
cvs server: Diffing .
Index: DAPA.C
===================================================================
RCS file: /cvsroot/uisp/uisp/src/DAPA.C,v
retrieving revision 1.7
diff -u -b -r1.7 DAPA.C
--- DAPA.C      15 Nov 2002 21:42:18 -0000      1.7
+++ DAPA.C      22 Nov 2002 01:05:27 -0000
@@ -107,6 +107,13 @@

#endif

+#else /* NO_DIRECT_IO */
+
+#define ioport_read(port)         (0xFF)
+#define ioport_write(port, val)
+#define ioport_enable(port, num)  (-1)
+#define ioport_disable(port, num) (0)
+
#endif /* NO_DIRECT_IO */

#include <unistd.h>
@@ -139,7 +146,7 @@
#define par_write_data(fd, ptr)  ioctl(fd, PPISDATA, ptr)
#define par_write_ctrl(fd, ptr)  ioctl(fd, PPISCTRL, ptr)
/* par_set_dir not defined, par_write_ctrl used instead */
-#define par_release(fd)          (0)
+#define par_release(fd)

#else

@@ -149,7 +156,7 @@
#define par_read_status(fd, ptr)
#define par_write_data(fd, ptr)
#define par_write_ctrl(fd, ptr)
-#define par_release(fd)          (0)
+#define par_release(fd)

#endif