FreeTDS Development Group

["Glenn, Chris" <Glenn AT addsys.com>]

The patch works great - thanks!  I assume this will be in tonight's
build?

While I have you - two other things;

1) Enhancement request;
        Could the 'bcp' function be enhanced to show the actual number
of rows copied in/out in each batch like Sybase's bcp? And don't show
blank lines between each batch?

2) Bug?;
        The "rows affected" value always seems to be returned as -1.
I've noticed this in my CT-Lib application which attempts to validate
this value after executing an insert statement and expects to get 1.
While attempting to hack in my own adjustment for the bcp enhancement
request above, I changed the bcp.c routine to print out this value.
        My changes looked like;
        ...
                   *rows_copied += tds->rows_affected;
/*      dbperror(dbproc, SYBEBBCI, 0);  -- batch copied to server --
*/
                   printf("%d rows copied to server.\n",
tds->rows_affected);
        ...

        The bcp output looked like;
        ...
                Starting copy...

                -1 rows copied to server.
                -1 rows copied to server.
                -3 rows copied.
        ...

Thanks again!

Chris

-----Original Message-----
From: freetds-bounces AT lists.ibiblio.org
[mailto:freetds-bounces AT lists.ibiblio.org] On Behalf Of Lowden, James K
Sent: Wednesday, November 29, 2006 3:50 PM
To: FreeTDS Development Group
Subject: Re: [freetds] snapshot dumps core on RHEL ES release 3

> From: Glenn, Chris
> Sent: Wednesday, November 29, 2006 3:09 PM
> 
> 
> Here is my 'tsql' command;
> -----------------------------------------------------
> /usr/local/bin/tsql -Usa -Psysadm
> locale is "C"
> locale charset is "ANSI_X3.4-1968"
> 1> select db_name()
> 2> go
> Segmentation fault (core dumped)
> -----------------------------------------------------

Bah!  tsql is broken.  Here's a patch, already committed.  

Index: tsql.c
===================================================================
RCS file: /cvsroot/freetds/freetds/src/apps/tsql.c,v
retrieving revision 1.96
diff -u -r1.96 tsql.c
--- tsql.c      28 Nov 2006 16:52:41 -0000      1.96
+++ tsql.c      29 Nov 2006 20:41:05 -0000
@@ -298,7 +298,7 @@
                s = NULL;

        first_arg = argv;
-       if (strcasecmp(first_arg[0], "go") == 0) {
+       if (argc > 0 && strcasecmp(first_arg[0], "go") == 0) {
                argc--;
                first_arg++;
        }

That should oughta do it.  

--jkl

-----------------------------------------
The information contained in this transmission may be privileged and
confidential and is intended only for the use of the person(s) named
above. If you are not the intended recipient, or an employee or agent
responsible
for delivering this message to the intended recipient, any review,
dissemination,
distribution or duplication of this communication is strictly
prohibited. If you are
not the intended recipient, please contact the sender immediately by
reply e-mail
and destroy all copies of the original message. Please note that we do
not accept
account orders and/or instructions by e-mail, and therefore will not be
responsible
for carrying out such orders and/or instructions.  If you, as the
intended recipient
of this message, the purpose of which is to inform and update our
clients, prospects
and consultants of developments relating to our services and products,
would not
like to receive further e-mail correspondence from the sender, please
"reply" to the
sender indicating your wishes.  In the U.S.: 1345 Avenue of the
Americas, New York,
NY 10105.
_______________________________________________
FreeTDS mailing list
FreeTDS AT lists.ibiblio.org
http://lists.ibiblio.org/mailman/listinfo/freetds