Skip Menu |
 

This queue is for tickets about the DBD-ODBC CPAN distribution.

Report information
The Basics
Id: 57957
Status: resolved
Priority: 0/
Queue: DBD-ODBC

People
Owner: Nobody in particular
Requestors: MPREWITT [...] cpan.org
Cc: prewittm [...] ms.com
AdminCc:

Bug Information
Severity: (no value)
Broken in:
  • 1.23
  • 1.24
Fixed in: 1.24_2



CC: prewittm [...] ms.com
Subject: dbd_preparse not ignoring placeholders in comments
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 1744
Download (untitled) / with headers
text/plain 1.7k
We encountered the 'DBD::ODBC does not yet support binding a named parameter more than once' error when something that looked like a placeholder was found in the SQL comments part of a query like this: /* $Date: 2010/05/01 12:00:00 */ select * from tb The :00 and :00 were interpreted as placeholders. I looked at dbd_preparse in dbdimp.c and see why it is happening. It looks like there is no detection as to whether you are in or outside of a comment. If you look at the recent versions of dbdimp.c in DBD::Sybase, you'll see they've added code to detect this this situation and ignore anything inside line or block comments: while (*src) { next_state = state; /* default situation */ while (*src) { next_state = state; /* default situation */ switch (state) { case DEFAULT: if (*src == '\'' || *src == '"') { last_literal = *src; next_state = LITERAL; } else if (*src == '/' && *(src+1) == '*') { next_state = COMMENT; } else if (*src == '-' && *(src+1) == '-') { next_state = LINE_COMMENT; } else if (*src == '@') { varname[0] = '@'; pos = 1; next_state = VARIABLE; } break; case LITERAL: if (*src == last_literal) { next_state = DEFAULT; } break; case COMMENT: if (*(src-1) == '*' && *src == '/') { next_state = DEFAULT; } break; case LINE_COMMENT: if (*src == '\n') { next_state = DEFAULT; } break; case VARIABLE: if (!isalnum(*src) && *src != '_') { varname[pos] = 0; next_state = DEFAULT; } else if (pos < VARNAME_LEN) { varname[pos++] = *src; } } It would be great to add this to the ODBC version of dbdimp.c (looks like it was originally copied from the DBD::Sybase implementation (or a common base)).
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10878-1275398276-1775.57957-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 2043
Download (untitled) / with headers
text/plain 1.9k
On Fri May 28 15:52:51 2010, MPREWITT wrote: Show quoted text
> We encountered the 'DBD::ODBC does not yet support binding a named > parameter more than once' error when something that looked like a > placeholder was found in the SQL comments part of a query like this: > > /* $Date: 2010/05/01 12:00:00 */ > select * from tb > > The :00 and :00 were interpreted as placeholders. I looked at > dbd_preparse in dbdimp.c and see why it is happening. It looks like > there is no detection as to whether you are in or outside of a comment. > If you look at the recent versions of dbdimp.c in DBD::Sybase, you'll > see they've added code to detect this this situation and ignore anything > inside line or block comments: > > while (*src) { > next_state = state; /* default situation */ > while (*src) { > next_state = state; /* default situation */ > switch (state) { > case DEFAULT: > if (*src == '\'' || *src == '"') { > last_literal = *src; > next_state = LITERAL; > } else if (*src == '/' && *(src+1) == '*') { > next_state = COMMENT; > } else if (*src == '-' && *(src+1) == '-') { > next_state = LINE_COMMENT; > } else if (*src == '@') { > varname[0] = '@'; > pos = 1; > next_state = VARIABLE; > } > break; > case LITERAL: > if (*src == last_literal) { > next_state = DEFAULT; > } > break; > case COMMENT: > if (*(src-1) == '*' && *src == '/') { > next_state = DEFAULT; > } > break; > case LINE_COMMENT: > if (*src == '\n') { > next_state = DEFAULT; > } > break; > case VARIABLE: > if (!isalnum(*src) && *src != '_') { > varname[pos] = 0; > next_state = DEFAULT; > } else if (pos < VARNAME_LEN) { > varname[pos++] = *src; > } > } > > It would be great to add this to the ODBC version of dbdimp.c (looks > like it was originally copied from the DBD::Sybase implementation (or a > common base)).
I've no idea where that code came from - before my time. I will try and add this in 1.24_2. Martin -- Martin J. Evans Wetherby, UK
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10893-1275937441-263.57957-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 2340
Download (untitled) / with headers
text/plain 2.2k
On Fri May 28 15:52:51 2010, MPREWITT wrote: Show quoted text
> We encountered the 'DBD::ODBC does not yet support binding a named > parameter more than once' error when something that looked like a > placeholder was found in the SQL comments part of a query like this: > > /* $Date: 2010/05/01 12:00:00 */ > select * from tb > > The :00 and :00 were interpreted as placeholders. I looked at > dbd_preparse in dbdimp.c and see why it is happening. It looks like > there is no detection as to whether you are in or outside of a comment. > If you look at the recent versions of dbdimp.c in DBD::Sybase, you'll > see they've added code to detect this this situation and ignore anything > inside line or block comments: > > while (*src) { > next_state = state; /* default situation */ > while (*src) { > next_state = state; /* default situation */ > switch (state) { > case DEFAULT: > if (*src == '\'' || *src == '"') { > last_literal = *src; > next_state = LITERAL; > } else if (*src == '/' && *(src+1) == '*') { > next_state = COMMENT; > } else if (*src == '-' && *(src+1) == '-') { > next_state = LINE_COMMENT; > } else if (*src == '@') { > varname[0] = '@'; > pos = 1; > next_state = VARIABLE; > } > break; > case LITERAL: > if (*src == last_literal) { > next_state = DEFAULT; > } > break; > case COMMENT: > if (*(src-1) == '*' && *src == '/') { > next_state = DEFAULT; > } > break; > case LINE_COMMENT: > if (*src == '\n') { > next_state = DEFAULT; > } > break; > case VARIABLE: > if (!isalnum(*src) && *src != '_') { > varname[pos] = 0; > next_state = DEFAULT; > } else if (pos < VARNAME_LEN) { > varname[pos++] = *src; > } > } > > It would be great to add this to the ODBC version of dbdimp.c (looks > like it was originally copied from the DBD::Sybase implementation (or a > common base)).
Should be fixed in the subversion trunk now. I added support for inline comments (/* comment */) and line comments (-- comment ends on newline). I've also added some tests for it. If you want to give it a go send me you Perl distribution/platform (e.g., if ActiveState, Strawberry or UNIX) and I'll mail you instructions. I will probably release 1.24_2 with this change to CPAN shortly. Martin -- Martin J. Evans Wetherby, UK
X-Anti-Virus: Kaspersky Anti-Virus for MailServers 5.5.35/RELEASE, bases: 07062010 #3962598, status: clean
CC: "Banai, Zoltan" <Zoltan.Banai [...] morganstanley.com>
MIME-Version: 1.0
Content-Class: urn:content-classes:message
X-Spam-Flag: NO
Acceptlanguage: en-US
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Content-Type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -10.003
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 4D430240847 for <cpan-bug+DBD-ODBC [...] hipster.bestpractical.com>; Mon, 7 Jun 2010 15:33:23 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id lzWv67PgxoMr for <cpan-bug+DBD-ODBC [...] hipster.bestpractical.com>; Mon, 7 Jun 2010 15:33:21 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 3C7B2240833 for <bug-DBD-ODBC [...] rt.cpan.org>; Mon, 7 Jun 2010 15:33:21 -0400 (EDT)
Received: (qmail 31442 invoked by uid 103); 7 Jun 2010 19:34:16 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 7 Jun 2010 19:34:16 -0000
Received: from hqmtaint02.ms.com (HELO hqmtaint02.ms.com) (205.228.53.69) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Mon, 07 Jun 2010 12:34:14 -0700
Received: from hqmtaint02 (localhost.ms.com [127.0.0.1]) by hqmtaint02.ms.com (output Postfix) with ESMTP id 05D2AE38EAD for <bug-DBD-ODBC [...] rt.cpan.org>; Mon, 7 Jun 2010 15:34:11 -0400 (EDT)
Received: from ny0030as02 (unknown [170.74.93.68]) by hqmtaint02.ms.com (internal Postfix) with ESMTP id C0FAD110034 for <bug-DBD-ODBC [...] rt.cpan.org>; Mon, 7 Jun 2010 15:34:10 -0400 (EDT)
Received: from ny0030as02 (localhost [127.0.0.1]) by ny0030as02 (msa-out Postfix) with ESMTP id A2BB8AB822A for <bug-DBD-ODBC [...] rt.cpan.org>; Mon, 7 Jun 2010 15:34:10 -0400 (EDT)
Received: from NPWEXGOB02.msad.ms.com (np212c1n1 [10.184.90.163]) by ny0030as02 (mta-in Postfix) with ESMTP id 9F9E95CC035 for <bug-DBD-ODBC [...] rt.cpan.org>; Mon, 7 Jun 2010 15:34:10 -0400 (EDT)
Received: from NPWEXGIB01.msad.ms.com (10.184.26.184) by NPWEXGOB02.msad.ms.com (10.184.90.163) with Microsoft SMTP Server (TLS) id 8.2.176.0; Mon, 7 Jun 2010 15:34:02 -0400
Received: from npwexhub02.msad.ms.com (10.164.54.4) by NPWEXGIB01.msad.ms.com (10.184.26.184) with Microsoft SMTP Server (TLS) id 8.2.176.0; Mon, 7 Jun 2010 15:33:22 -0400
Received: from NYWEXMBX2128.msad.ms.com ([10.184.95.6]) by npwexhub02.msad.ms.com ([10.164.54.4]) with mapi; Mon, 7 Jun 2010 15:33:22 -0400
Delivered-To: cpan-bug+DBD-ODBC [...] hipster.bestpractical.com
Subject: RE: [rt.cpan.org #57957] dbd_preparse not ignoring placeholders in comments
Thread-Index: AcsGdDIV/kIlmMK6Tdah/UkIAfSCNAAA7BDA
X-Spam-Check-BY: 16.mx.develooper.com
Date: Mon, 7 Jun 2010 15:33:20 -0400
X-Spam-Level:
X-Mimeole: Produced By Microsoft MimeOLE V6.00.3790.4657
To: <bug-DBD-ODBC [...] rt.cpan.org>
Priority: normal
Content-Transfer-Encoding: quoted-printable
From Marc.Prewitt [...] morganstanley.com Mon Jun 7 15: 33:23 2010
In-Reply-To: <rt-3.8.HEAD-10893-1275937441-1549.57957-6-0 [...] rt.cpan.org>
X-Spam-Status: No, score=-10.003 tagged_above=-99.9 required=10 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_HI=-8, SPF_SOFTFAIL=0.596] autolearn=ham
Content-Language: en-US
Importance: normal
References: <RT-Ticket-57957 [...] rt.cpan.org> <rt-3.8.HEAD-10893-1275937441-1549.57957-6-0 [...] rt.cpan.org>
Message-ID: <689096550A3B584EB205F0DB241FDB8E04907D2B9D [...] NYWEXMBX2128.msad.ms.com>
X-MS-Tnef-Correlator:
Return-Path: <Marc.Prewitt [...] morganstanley.com>
X-RT-Mail-Extension: dbd-odbc
X-Original-To: cpan-bug+DBD-ODBC [...] hipster.bestpractical.com
X-MS-Has-Attach:
Thread-Topic: [rt.cpan.org #57957] dbd_preparse not ignoring placeholders in comments
Accept-Language: en-US
From: "Prewitt, Marc" <Marc.Prewitt [...] morganstanley.com>
RT-Message-ID: <rt-3.8.HEAD-10892-1275939259-1965.57957-0-0 [...] rt.cpan.org>
Content-Length: 3137
Thanks Martin. We're using perl 5.10/unix. We'll give it a try. Zoltan, If we can try this out in /ms/dev I can test it out there. Thanks, Marc Show quoted text
-----Original Message----- From: Martin J Evans via RT [mailto:bug-DBD-ODBC@rt.cpan.org] Sent: Monday, June 07, 2010 3:04 PM To: MPREWITT@cpan.org Cc: Prewitt, Marc (IT) Subject: [rt.cpan.org #57957] dbd_preparse not ignoring placeholders in comments <URL: https://rt.cpan.org/Ticket/Display.html?id=57957 > On Fri May 28 15:52:51 2010, MPREWITT wrote:
> We encountered the 'DBD::ODBC does not yet support binding a named > parameter more than once' error when something that looked like a > placeholder was found in the SQL comments part of a query like this: > > /* $Date: 2010/05/01 12:00:00 */ > select * from tb > > The :00 and :00 were interpreted as placeholders. I looked at > dbd_preparse in dbdimp.c and see why it is happening. It looks like > there is no detection as to whether you are in or outside of a comment. > If you look at the recent versions of dbdimp.c in DBD::Sybase, you'll > see they've added code to detect this this situation and ignore anything > inside line or block comments: > > while (*src) { > next_state = state; /* default situation */ > while (*src) { > next_state = state; /* default situation */ > switch (state) { > case DEFAULT: > if (*src == '\'' || *src == '"') { > last_literal = *src; > next_state = LITERAL; > } else if (*src == '/' && *(src+1) == '*') { > next_state = COMMENT; > } else if (*src == '-' && *(src+1) == '-') { > next_state = LINE_COMMENT; > } else if (*src == '@') { > varname[0] = '@'; > pos = 1; > next_state = VARIABLE; > } > break; > case LITERAL: > if (*src == last_literal) { > next_state = DEFAULT; > } > break; > case COMMENT: > if (*(src-1) == '*' && *src == '/') { > next_state = DEFAULT; > } > break; > case LINE_COMMENT: > if (*src == '\n') { > next_state = DEFAULT; > } > break; > case VARIABLE: > if (!isalnum(*src) && *src != '_') { > varname[pos] = 0; > next_state = DEFAULT; > } else if (pos < VARNAME_LEN) { > varname[pos++] = *src; > } > } > > It would be great to add this to the ODBC version of dbdimp.c (looks > like it was originally copied from the DBD::Sybase implementation (or a > common base)).
Should be fixed in the subversion trunk now. I added support for inline comments (/* comment */) and line comments (-- comment ends on newline). I've also added some tests for it. If you want to give it a go send me you Perl distribution/platform (e.g., if ActiveState, Strawberry or UNIX) and I'll mail you instructions. I will probably release 1.24_2 with this change to CPAN shortly. Martin -- Martin J. Evans Wetherby, UK -------------------------------------------------------------------------- NOTICE: If received in error, please destroy, and notify sender. Sender does not intend to waive confidentiality or privilege. Use of this email is prohibited when received in error. We may monitor and store emails to the extent permitted by applicable law.
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-10892-1275939259-1965.57957-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <RT-Ticket-57957 [...] rt.cpan.org> <rt-3.8.HEAD-10893-1275937441-1549.57957-6-0 [...] rt.cpan.org> <689096550A3B584EB205F0DB241FDB8E04907D2B9D [...] NYWEXMBX2128.msad.ms.com> <rt-3.8.HEAD-10892-1275939259-1965.57957-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-10879-1275984010-963.57957-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 997
Download (untitled) / with headers
text/plain 997b
On Mon Jun 07 15:34:19 2010, marc.prewitt@morganstanley.com wrote: Show quoted text
> Thanks Martin. We're using perl 5.10/unix. We'll give it a try. > > Zoltan, > > If we can try this out in /ms/dev I can test it out there. > > Thanks, > Marc
If you have a subversion client then do: svn checkout http://svn.perl.org/modules/dbd-odbc/trunk <your directory name here> If you use an HTTP proxy you may need your http_proxy env var setting. Alternatively, mail me (mjevans at cpan dot org) and I'll send you a copy of the trunk. cd <your directory name here> export DBI_DSN=dbi:ODBC:yourdsn export DBI_USER=db_username export DBI_PASS=db_password perl Makefile.PL make make test The test t/rt_57957.t has a few minor tests for placeholders in comments and literals. To run your own perl with this DBD::ODBC either: perl -I<your directory name here>/blib/lib -I<your_directory_name_here>/blib/arch myperl.pl or make install it. Let me know how you get on. Martin -- Martin J. Evans Wetherby, UK
X-Anti-Virus: Kaspersky Anti-Virus for MailServers 5.5.35/RELEASE, bases: 13062010 #4023142, status: clean
MIME-Version: 1.0
Content-Class: urn:content-classes:message
X-Spam-Flag: NO
Acceptlanguage: en-US
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
Content-Type: text/plain; charset="utf-8"
X-RT-Original-Encoding: utf-8
X-Spam-Score: -10.003
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id 308F22406EF for <cpan-bug+DBD-ODBC [...] hipster.bestpractical.com>; Sun, 13 Jun 2010 14:56:33 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ay7vbDwMPIGx for <cpan-bug+DBD-ODBC [...] hipster.bestpractical.com>; Sun, 13 Jun 2010 14:56:30 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id 95D9C24063B for <bug-DBD-ODBC [...] rt.cpan.org>; Sun, 13 Jun 2010 14:56:30 -0400 (EDT)
Received: (qmail 3975 invoked by uid 103); 13 Jun 2010 18:57:32 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 13 Jun 2010 18:57:32 -0000
Received: from hqmtaint03.ms.com (HELO hqmtaint03.ms.com) (205.228.53.73) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Sun, 13 Jun 2010 11:57:30 -0700
Received: from hqmtaint03 (localhost.ms.com [127.0.0.1]) by hqmtaint03.ms.com (output Postfix) with ESMTP id BDC1CB6C4EF; Sun, 13 Jun 2010 14:57:26 -0400 (EDT)
Received: from ny0030as02 (unknown [170.74.93.68]) by hqmtaint03.ms.com (internal Postfix) with ESMTP id 96FDBA3005E; Sun, 13 Jun 2010 14:57:26 -0400 (EDT)
Received: from ny0030as02 (localhost [127.0.0.1]) by ny0030as02 (msa-out Postfix) with ESMTP id 7D762AB821F; Sun, 13 Jun 2010 14:57:26 -0400 (EDT)
Received: from HNWEXGOB02.msad.ms.com (hn212c1n1 [10.184.121.167]) by ny0030as02 (mta-in Postfix) with ESMTP id 788A45CC035; Sun, 13 Jun 2010 14:57:26 -0400 (EDT)
Received: from HNWEXGIB02.msad.ms.com (10.184.57.209) by HNWEXGOB02.msad.ms.com (10.184.121.167) with Microsoft SMTP Server (TLS) id 8.2.176.0; Sun, 13 Jun 2010 14:57:25 -0400
Received: from npwexhub04.msad.ms.com (10.184.26.156) by HNWEXGIB02.msad.ms.com (10.184.57.209) with Microsoft SMTP Server (TLS) id 8.2.176.0; Sun, 13 Jun 2010 14:57:25 -0400
Received: from NYWEXMBX2128.msad.ms.com ([10.184.95.6]) by npwexhub04.msad.ms.com ([10.184.26.156]) with mapi; Sun, 13 Jun 2010 14:57:24 -0400
Delivered-To: cpan-bug+DBD-ODBC [...] hipster.bestpractical.com
Subject: Re: [rt.cpan.org #57957] dbd_preparse not ignoring placeholders in comments
Thread-Index: AcsG4K4DEp0Bn/ctS062fBvFY+tqkwESZUfu
X-Spam-Check-BY: 16.mx.develooper.com
Date: Sun, 13 Jun 2010 14:57:25 -0400
X-Spam-Level:
X-Mimeole: Produced By Microsoft MimeOLE V6.00.3790.4657
To: <bug-DBD-ODBC [...] rt.cpan.org>, <MPREWITT [...] cpan.org>
Priority: normal
Content-Transfer-Encoding: base64
From Marc.Prewitt [...] morganstanley.com Sun Jun 13 14: 56:33 2010
X-Spam-Status: No, score=-10.003 tagged_above=-99.9 required=10 tests=[BAYES_00=-2.599, RCVD_IN_DNSWL_HI=-8, SPF_SOFTFAIL=0.596] autolearn=ham
Importance: normal
Content-Language: en-US
Message-ID: <689096550A3B584EB205F0DB241FDB8E0490588D7B [...] NYWEXMBX2128.msad.ms.com>
X-MS-Tnef-Correlator:
Return-Path: <Marc.Prewitt [...] morganstanley.com>
X-RT-Mail-Extension: dbd-odbc
X-Original-To: cpan-bug+DBD-ODBC [...] hipster.bestpractical.com
X-MS-Has-Attach:
Thread-Topic: [rt.cpan.org #57957] dbd_preparse not ignoring placeholders in comments
Accept-Language: en-US
From: "Prewitt, Marc" <Marc.Prewitt [...] morganstanley.com>
RT-Message-ID: <rt-3.8.HEAD-10893-1276455456-1660.57957-0-0 [...] rt.cpan.org>
Content-Length: 1852
Download (untitled) / with headers
text/plain 1.8k
Initial testing looks good. I'm going to run this version on our devel system for a while and see if anything comes up. Thanks, Marc Show quoted text
----- Original Message ----- From: Martin J Evans via RT <bug-DBD-ODBC@rt.cpan.org> To: MPREWITT@cpan.org <MPREWITT@cpan.org> Cc: Prewitt, Marc (IT) Sent: Tue Jun 08 04:00:12 2010 Subject: [rt.cpan.org #57957] dbd_preparse not ignoring placeholders in comments <URL: https://rt.cpan.org/Ticket/Display.html?id=57957 > On Mon Jun 07 15:34:19 2010, marc.prewitt@morganstanley.com wrote:
> Thanks Martin. We're using perl 5.10/unix. We'll give it a try. > > Zoltan, > > If we can try this out in /ms/dev I can test it out there. > > Thanks, > Marc
If you have a subversion client then do: svn checkout http://svn.perl.org/modules/dbd-odbc/trunk <your directory name here> If you use an HTTP proxy you may need your http_proxy env var setting. Alternatively, mail me (mjevans at cpan dot org) and I'll send you a copy of the trunk. cd <your directory name here> export DBI_DSN=dbi:ODBC:yourdsn export DBI_USER=db_username export DBI_PASS=db_password perl Makefile.PL make make test The test t/rt_57957.t has a few minor tests for placeholders in comments and literals. To run your own perl with this DBD::ODBC either: perl -I<your directory name here>/blib/lib -I<your_directory_name_here>/blib/arch myperl.pl or make install it. Let me know how you get on. Martin -- Martin J. Evans Wetherby, UK -------------------------------------------------------------------------- NOTICE: If received in error, please destroy, and notify sender. Sender does not intend to waive confidentiality or privilege. Use of this email is prohibited when received in error. We may monitor and store emails to the extent permitted by applicable law.
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-10893-1276455456-1660.57957-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <689096550A3B584EB205F0DB241FDB8E0490588D7B [...] NYWEXMBX2128.msad.ms.com> <rt-3.8.HEAD-10893-1276455456-1660.57957-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-2912-1276806481-1129.57957-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 422
Download (untitled) / with headers
text/plain 422b
On Sun Jun 13 14:57:36 2010, marc.prewitt@morganstanley.com wrote: Show quoted text
> Initial testing looks good. I'm going to run this version on our devel > system for a while and see if anything comes up. > > Thanks, > Marc
How are you getting on with this. I am close to doing a new development release and would like some reassurance that this change has worked and not broken something. Martin -- Martin J. Evans Wetherby, UK
MIME-Version: 1.0
In-Reply-To: <rt-3.8.HEAD-2912-1276806481-1129.57957-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
References: <689096550A3B584EB205F0DB241FDB8E0490588D7B [...] NYWEXMBX2128.msad.ms.com> <rt-3.8.HEAD-10893-1276455456-1660.57957-0-0 [...] rt.cpan.org> <rt-3.8.HEAD-2912-1276806481-1129.57957-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-2366-1279909444-459.57957-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 608
Download (untitled) / with headers
text/plain 608b
On Thu Jun 17 16:28:01 2010, MJEVANS wrote: Show quoted text
> On Sun Jun 13 14:57:36 2010, marc.prewitt@morganstanley.com wrote:
> > Initial testing looks good. I'm going to run this version on our devel > > system for a while and see if anything comes up. > > > > Thanks, > > Marc
> > How are you getting on with this. I am close to doing a new development > release and would like some reassurance that this change has worked and > not broken something. > > Martin
This is fixed in DBD::ODBC 1.24_2 which is on its way to CPAN and I will close it as soon as rt sees 1.24_2. Martin -- Martin J. Evans Wetherby, UK


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.