Re: [rt.cpan.org #61168] Specifying seperation character per table does not work

2010-09-16 Thread H.Merijn Brand
On Wed, 15 Sep 2010 23:51:40 +0200, Jens Rehsack rehs...@googlemail.com wrote: Ok, I understood your desire - but your patch doesn't catch all the backward compatibility you typically want - see (and apply) 0001-add-test-case-for-rt61168.patch first. As we have no reason this time to run

Re: [rt.cpan.org #61168] Specifying seperation character per table does not work

2010-09-16 Thread Jens Rehsack
2010/9/16 H.Merijn Brand h.m.br...@xs4all.nl: On Wed, 15 Sep 2010 23:51:40 +0200, Jens Rehsack rehs...@googlemail.com wrote: Ok, I understood your desire - but your patch doesn't catch all the backward compatibility you typically want - see (and apply) 0001-add-test-case-for-rt61168.patch

Re: [svn:dbi] r14407 - in dbi/trunk: . lib/DBD lib/DBD/File

2010-09-16 Thread Tim Bunce
No tests? Tim. On Wed, Sep 15, 2010 at 10:48:36PM -0700, rehs...@cvs.perl.org wrote: Author: REHSACK Date: Wed Sep 15 22:48:35 2010 New Revision: 14407 Modified: dbi/trunk/Changes dbi/trunk/lib/DBD/DBM.pm dbi/trunk/lib/DBD/File.pm dbi/trunk/lib/DBD/File/Developers.pod

Re: [svn:dbi] r14407 - in dbi/trunk: . lib/DBD lib/DBD/File

2010-09-16 Thread Jens Rehsack
2010/9/16 Tim Bunce tim.bu...@pobox.com: No tests? The tests are already in there - t/5?dbm* test for these features, especially t/52dbm_complex.t. This is more a refactoring to allow better implementation in DBD::CSV - which has a separate test for it's special case. Tim. /Jens On Wed, Sep

Re: [rt.cpan.org #61168] Specifying seperation character per table does not work

2010-09-16 Thread H.Merijn Brand
On Thu, 16 Sep 2010 09:43:05 +0200, Jens Rehsack rehs...@googlemail.com wrote: 2010/9/16 H.Merijn Brand h.m.br...@xs4all.nl: On Wed, 15 Sep 2010 23:51:40 +0200, Jens Rehsack rehs...@googlemail.com wrote: Ok, I understood your desire - but your patch doesn't catch all the backward