sane-project-website/old-archive/1998-04/0059.html

<!-- received="Sun Apr  5 12:21:47 1998 PDT" -->
<!-- sent="Sun, 5 Apr 1998 21:21:08 +0200 (CEST)" -->
<!-- name="=?ISO-8859-1?Q?Mikko_Ty=F6l=E4j=E4rvi?=" -->
<!-- email="mikko@securitydynamics.com" -->
<!-- subject="sane 0.72pre meets FreeBSD" -->
<!-- id="" -->
<!-- inreplyto="" -->
<title>sane-devel: sane 0.72pre meets FreeBSD</title>
<h1>sane 0.72pre meets FreeBSD</h1>
<b>=?ISO-8859-1?Q?Mikko_Ty=F6l=E4j=E4rvi?=</b> (<a href="mailto:mikko@securitydynamics.com"><i>mikko@securitydynamics.com</i></a>)<br>
<i>Sun, 5 Apr 1998 21:21:08 +0200 (CEST)</i>
<p>
<ul>
<li> <b>Messages sorted by:</b> <a href="date.html#59">[ date ]</a><a href="index.html#59">[ thread ]</a><a href="subject.html#59">[ subject ]</a><a href="author.html#59">[ author ]</a>
<!-- next="start" -->
<li> <b>Next message:</b> <a href="0060.html">Christian Bucher: "I don´t like that"</a>
<li> <b>Previous message:</b> <a href="0058.html">John Brush: "Re: Q: support for digi cameras"</a>
<!-- nextthread="start" -->
<!-- reply="end" -->
</ul>
<!-- body="start" -->
The patches below are all (?) that is required to compile and pass<br>
initial tests on FreeBSD 2.2.5 with gtk 0.99.9 and gimp 0.99.23.<br>
<p>
I haven't had time to test on FreeBSD 2.2.6 or 3.0 yet, but I'd be<br>
surprised if the result is any different.<br>
<p>
<p>
Question 0:<br>
<p>
Is the entire build supposed to pass without any compiler warnings?<br>
If so, there are a few things that could be improved:<br>
<p>
 - getopt.c does not include headers to get declarations of getpid()<br>
   and strdup().  Well, it sort of does, but there are enough #ifdefs<br>
   to make sure it does not happen.  But I guess this file is simply<br>
   grabbed out of GNU libs, and should not be changed.  No big deal.<br>
<p>
 - sane provides strndup() for systems that does not have it, but<br>
   fails to declare it anywhere.  It could be done in config.h, which<br>
   would require to include &lt;sys/types.h&gt; somewhere too (to get size_t).<br>
<p>
<p>
Question 1:<br>
<p>
Is it necessary to keep all the loadable backends in the standard<br>
library path?  I'm getting an uneasy feeling when sane singlehandedly<br>
doubles the number of entries in my /usr/local/lib, with something<br>
that could be considered as internal to sane.<br>
<p>
Am I missing something, or could they be moved to (for example)<br>
/usr/local/lib/sane/?  Aren't the backends just dlopen()ed?<br>
<p>
<p>
Patch 0:<br>
<p>
The test for libgimp/gimp.h does not use CFLAGS, which are updated to<br>
contain the X11 include directory.  It uses CPPFLAGS.  This way gimp.h<br>
is located automagically in case gimp is in /usr/X11R6 or so.<br>
<p>
Maybe CFLAGS and CPPFLAGS should be restored further down, as the<br>
comment suggests (but which never happens).<br>
<p>
--- sane-pre0.72.org/configure.in	Fri Apr  3 09:52:35 1998<br>
+++ sane-pre0.72/configure.in	Sun Apr  5 20:17:51 1998<br>
@@ -108,20 +108,21 @@<br>
 <br>
 # Change CFLAGS temporarily so that C_SWITCH_X_SITE gets used<br>
 # for the tests that follow.  We set it back to REAL_CFLAGS later on.<br>
 <br>
 HAVE_X11=yes<br>
 if test "${HAVE_X11}" = "yes"; then<br>
   DEFS="$C_SWITCH_X_SITE $DEFS"<br>
   LDFLAGS="$LDFLAGS $LD_SWITCH_X_SITE"<br>
   LIBS="$LIBX $LIBS"<br>
   CFLAGS="$C_SWITCH_X_SITE $CFLAGS"<br>
+  CPPFLAGS="$C_SWITCH_X_SITE $CPPFLAGS"<br>
 <br>
   # On Solaris, arrange for LD_RUN_PATH to point to the X libraries for tests.<br>
   # This is handled by LD_SWITCH_X_SITE_AUX during the real build,<br>
   # but it's more convenient here to set LD_RUN_PATH<br>
   # since this also works on hosts that don't understand LD_SWITCH_X_SITE_AUX.<br>
   if test "${x_libraries}" != NONE &amp;&amp; test -n "${x_libraries}"; then<br>
     LD_RUN_PATH=$x_libraries${LD_RUN_PATH+:}$LD_RUN_PATH<br>
     export LD_RUN_PATH<br>
   fi<br>
<p>
<p>
<p>
Patch 1:<br>
<p>
--- sane-pre0.72.org/backend/coolscan.c	Fri Apr  3 10:36:11 1998<br>
+++ sane-pre0.72/backend/coolscan.c	Sun Apr  5 20:01:13 1998<br>
@@ -66,7 +66,7 @@<br>
 <br>
 /*#include "sane/config.h"*/<br>
 <br>
-#include &lt;alloca.h&gt;<br>
+#include &lt;lalloca.h&gt;<br>
 #include &lt;errno.h&gt;<br>
 #include &lt;fcntl.h&gt;<br>
 #include &lt;limits.h&gt;<br>
<p>
<p>
	Regards,<br>
	/Mikko<br>
<p>
<p>
 Mikko Tyo"la"ja"rvi________________________________mikko@securitydynamics.com<br>
 SecurityDynamics<br>
<p>
<p>
<pre>
--
Source code, list archive, and docs: <a href="http://www.mostang.com/sane/">http://www.mostang.com/sane/</a>
To unsubscribe: echo unsubscribe sane-devel | mail <a href="mailto:majordomo@mostang.com">majordomo@mostang.com</a>
</pre>
<!-- body="end" -->
<p>
<ul>
<!-- next="start" -->
<li> <b>Next message:</b> <a href="0060.html">Christian Bucher: "I don´t like that"</a>
<li> <b>Previous message:</b> <a href="0058.html">John Brush: "Re: Q: support for digi cameras"</a>
<!-- nextthread="start" -->
<!-- reply="end" -->
</ul>