Bug #57342 for DBIx-VersionedDDL: Text::CSV configuration within does not parse double quotes.

Mon May 10 07:28:13 2010 ben [...] opusvl.com - Ticket created

Subject:

Text::CSV configuration within does not parse double quotes.

Hello, Thanks for the module, very handy. :) I have some table names that must be wrapped in double quotes (eg. user in postgres must be referenced "user") This caused the Text::CSV not to parse my .sql files once your module had stripped \n and comments etc. other than that works great.

Mon May 10 17:22:20 2010 dan.horne [...] redbone.co.nz - Correspondence added

Subject:	Re: [rt.cpan.org #57342] Text::CSV configuration within does not parse double quotes.
Date:	Tue, 11 May 2010 09:22:07 +1200
To:	bug-DBIx-VersionedDDL [...] rt.cpan.org
From:	Dan Horne <dan.horne [...] redbone.co.nz>

Hi, I haven't used postgress and aren't familiar with it. Can you please post a simple test case that exemplifies the issue Thanks Dan On 10 May 2010 23:28, Ben via RT <bug-DBIx-VersionedDDL@rt.cpan.org> wrote: Show quoted text

> Mon May 10 07:28:13 2010: Request 57342 was acted upon. > Transaction: Ticket created by bugthing > Queue: DBIx-VersionedDDL > Subject: Text::CSV configuration within does not parse double quotes. > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: ben@opusvl.com > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=57342 > > > > Hello, > > Thanks for the module, very handy. :) > > I have some table names that must be wrapped in double quotes (eg. user > in postgres must be referenced "user") > > This caused the Text::CSV not to parse my .sql files once your module > had stripped \n and comments etc. > > other than that works great. >

Mon May 10 17:22:22 2010 The RT System itself - Status changed from 'new' to 'open'

Tue May 11 04:24:29 2010 ben [...] opusvl.com - Correspondence added

From:

ben [...] opusvl.com

Hello, I have created a .tgz containing a test that fails, but 'should' pass. Instructions on how to use it are in README. You will have to setup a postgres server to test against. Not too tricky but let me know if you want pointers. Thanks again, -Ben On Mon May 10 17:22:20 2010, dan.horne@redbone.co.nz wrote: Show quoted text

> Hi, > > I haven't used postgress and aren't familiar with it. Can you please

post a Show quoted text

> simple test case that exemplifies the issue > > Thanks > > Dan > > On 10 May 2010 23:28, Ben via RT <bug-DBIx-VersionedDDL@rt.cpan.org>

wrote: Show quoted text

>

> > Mon May 10 07:28:13 2010: Request 57342 was acted upon. > > Transaction: Ticket created by bugthing > > Queue: DBIx-VersionedDDL > > Subject: Text::CSV configuration within does not parse double

quotes. Show quoted text

> > Broken in: (no value) > > Severity: (no value) > > Owner: Nobody > > Requestors: ben@opusvl.com > > Status: new > > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=57342 > > > > > > > Hello, > > > > Thanks for the module, very handy. :) > > > > I have some table names that must be wrapped in double quotes (eg. user > > in postgres must be referenced "user") > > > > This caused the Text::CSV not to parse my .sql files once your module > > had stripped \n and comments etc. > > > > other than that works great. > >

Subject:

VersionedDDL.tgz

Download VersionedDDL.tgz
application/x-gtar 1.4k

Message body not shown because it is not plain text.

Mon May 24 17:26:45 2010 dhorne [...] cpan.org - Correspondence added

On Tue May 11 04:24:29 2010, bugthing wrote: Show quoted text

> Hello, > > I have created a .tgz containing a test that fails, but 'should' pass. > > Instructions on how to use it are in README. > > You will have to setup a postgres server to test against. Not too tricky > but let me know if you want pointers. >

Hi Apologies for the delay. I haven't had much joy in getting DBD::Pg to compile on my macbook, so I've simply modified the distribution's sql to include quoted table names to test the fix. Can you please try the attached mod against your Postgres DB? Thanks Dan

Subject:

DBIx-VersionedDDL-0.10_01.tar.gz

Download DBIx-VersionedDDL-0.10_01.tar.gz
application/x-gzip 8.8k

Message body not shown because it is not plain text.

Thu May 27 04:58:23 2010 ben [...] opusvl.com - Correspondence added

From:

ben [...] opusvl.com

Hello, I have run the tests I did before, but using the new modules you sent over... all works fine :) thanks alot. On Mon May 24 17:26:45 2010, DHORNE wrote: Show quoted text

> On Tue May 11 04:24:29 2010, bugthing wrote:

> > Hello, > > > > I have created a .tgz containing a test that fails, but 'should' pass. > > > > Instructions on how to use it are in README. > > > > You will have to setup a postgres server to test against. Not too tricky > > but let me know if you want pointers. > >

> > Hi > > Apologies for the delay. I haven't had much joy in getting DBD::Pg to > compile on my macbook, so I've simply modified the distribution's sql to > include quoted table names to test the fix. Can you please try the > attached mod against your Postgres DB? > > Thanks > > Dan

Thu May 27 18:10:06 2010 dhorne [...] cpan.org - Correspondence added

Thanks. V 0.11 winging its way to CPAN On Thu May 27 04:58:23 2010, bugthing wrote: Show quoted text

> Hello, > > I have run the tests I did before, but using the new modules you sent > over... all works fine :) > > thanks alot. > > > > On Mon May 24 17:26:45 2010, DHORNE wrote:

> > On Tue May 11 04:24:29 2010, bugthing wrote:

> > > Hello, > > > > > > I have created a .tgz containing a test that fails, but 'should' pass. > > > > > > Instructions on how to use it are in README. > > > > > > You will have to setup a postgres server to test against. Not too

tricky Show quoted text

> > > but let me know if you want pointers. > > >

> > > > Hi > > > > Apologies for the delay. I haven't had much joy in getting DBD::Pg to > > compile on my macbook, so I've simply modified the distribution's sql to > > include quoted table names to test the fix. Can you please try the > > attached mod against your Postgres DB? > > > > Thanks > > > > Dan

>

Thu May 27 18:10:09 2010 dhorne [...] cpan.org - Status changed from 'open' to 'resolved'