# problem with sane 1.00 and NetBSD

Thorsten Frueauf (frueauf@ira.uka.de)
Sun, 29 Nov 1998 21:15:12 +0100 (MET)

- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
- Next message: Hugo van der Kooij: "SANE 1.00 breaks on Solaris v2.6"
- Previous message: Dave Barron: "ACROSS scanner compatible with ???"
- Next in thread: wolfgang Rapp: "Re: problem with sane 1.00 and NetBSD"
- Reply: wolfgang Rapp: "Re: problem with sane 1.00 and NetBSD"

Hello!

First sorry if this is the false way to report problems/bugs, but
I did not find an explicit email adress for such feedback, so I choosed
this maillist. But please note that I am not subscribed to this list.

I found two major problems while integrating the new sane 1.00 release
into pkgsrc from NetBSD (similar to the FreeBSD ports collection):

1) in backend/snapscan.c gets a Linux-only header file included
(at least I suspect that its Linux-only) which tries to find
out what MAXINT is.

2) in backend/abaton.c, s->val[OPT_X_RESOLUTION] is missing the .w member.

The appended patches solve those problems. I also want to note that
the DC210 backend does not compile, as it uses a constant which
I could not find in any NetBSD header file: TCSBRKP.
Its used in backend/dc210.c line 351, and I don't know an aquivalent.

Greets
Thorsten

--- backend/snapscan.c-orig Thu Oct 29 08:02:29 1998
+++ backend/snapscan.c Sat Nov 28 12:16:28 1998
@@ -55,7 +55,6 @@
#include <sys/time.h>
#include <sys/types.h>
#include <sys/stat.h>
-#include <values.h>
#include <limits.h>
#include <fcntl.h>
#include <unistd.h>
@@ -3150,7 +3149,7 @@
{
int result = write (pss->tfd, other_buf, pss->read_bytes);
- if (result < 0 || result == MAXINT)
+ if (result < 0 || result == INT_MAX)
{
DBG (DL_MAJOR_ERROR, "%s: write of test data to file failed.\n",
me);
--- backend/abaton.c-orig Sun Nov 22 03:54:14 1998
+++ backend/abaton.c Sat Nov 28 11:20:23 1998
@@ -1113,7 +1113,7 @@
case OPT_Y_RESOLUTION:
if (s->val[OPT_PREVIEW].w || s->val[OPT_RESOLUTION_BIND].w) {
- s->val[OPT_X_RESOLUTION] = *(SANE_Word *)val;
+ s->val[OPT_X_RESOLUTION].w = *(SANE_Word *)val;
if (info)
*info |= SANE_INFO_RELOAD_OPTIONS;
}

    --
    Source code, list archive, and docs: http://www.mostang.com/sane/
    To unsubscribe: echo unsubscribe sane-devel | mail majordomo@mostang.com

- Next message: Hugo van der Kooij: "SANE 1.00 breaks on Solaris v2.6"
- Previous message: Dave Barron: "ACROSS scanner compatible with ???"
- Next in thread: wolfgang Rapp: "Re: problem with sane 1.00 and NetBSD"
- Reply: wolfgang Rapp: "Re: problem with sane 1.00 and NetBSD"