FreeTDS Development Group

["Varley, David(CBorn at Alcoa)" <David.Varley AT alcoa.com.au>]

Freddy,

I just pulled the latest from CVS with your changes, the mods to
src/tds/Makefile.am seem to break things for me.

When I clean and run autogen.sh, I get in part the following:


Running autoheader...
WARNING: Using auxiliary files such as `acconfig.h', `config.h.bot'
WARNING: and `config.h.top', to define templates for `config.h.in'
WARNING: is deprecated and discouraged.

WARNING: Using the third argument of `AC_DEFINE' and
WARNING: `AC_DEFINE_UNQUOTED' allows to define a template without
WARNING: `acconfig.h':

WARNING:   AC_DEFINE([NEED_MAIN], 1,
WARNING:             [Define if a function `main' is needed.])

WARNING: More sophisticated templates can also be produced, see the
WARNING: documentation.
configure.in:18: warning: AC_ARG_PROGRAM invoked multiple times
autoheader: `include/config.h.in' is unchanged
Running automake --gnu --include-deps ...
src/tds/Makefile.am:3: `lib_LTLIBRARIES' contains configure substitution,
but `EXTRA_LTLIBRARIES' not defined


with the last line seeming to be the significant problem, and then
when I make I get the error:

make[3]: Entering directory `/usr/users/cborn/src/ODBC/freetds_c7/src/tds'
make[3]: *** No rule to make target `-liconv', needed by `all-am'.  Stop.


This is on alpha Tru64. When I build on x86 Linux I get the same warning in
the autogen,
but the make builds correctly and passes checks (except for ctlib t0003,
which failed
before anyway). I've been pulling CVS head changes and clean building at
least daily, with
good success rates until now :)

David


> -----Original Message-----
> From: Frediano Ziglio [mailto:freddyz77 AT tin.it]
> Sent: Saturday, 17 August 2002 1:39 AM
> To: TDS Development Group
> Subject: [freetds] Re: [ freetds-Patches-596008 ] trusted tds7 login
> 
> 
> Il ven, 2002-08-16 alle 17:19, Brian Bruns ha scritto:
> > Freddy,
> > 
> > This looks awesome!  How much testing have you done with it 
> and in what
> > kind of environments?
> > 
> > Brian
> > 
> 
> I have tested with a server on my firm
> - tested 2 login (a test login and personal one)
> - NT domain (not w2k)
> - one server (sql 7, with tds7)
> 
> Applying this patch work either with normal login and domain 
> login (not
> together... I not test this, nor I know if this is possible).
> 
> I note also in src/tds/Makefile.am this line:
>    LIBS            = @LIBICONV@
> doing so do not include libraries added in configure stuff.
> Also we should add challenge.lo to libraries makefiles...
> Perhaps @LIBICONV@ should be added to lib_LTLIBRARIES...
> 
> I'm not very happy for this patch (as you note I have lowered 
> priority)
> for previous reason and because I changed code using const pointer and
> calling tds_put_XXX (I changed this to const but I think this 
> patch can
> wait future release).
> 
> I think before applying this patch is best to fix other things...
> Should I put this on a higher priority?
> I not so a long work but need some test (see also my previous 
> mail "Some
> API change")...
> 
> freddy77
> 
> 
> 
> ---
> You are currently subscribed to freetds as: 
> [David.Varley AT alcoa.com.au]
> To unsubscribe, forward this message to 
> $subst('Email.Unsub')
>