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 wrote: > 2010/9/16 H.Merijn Brand : > > On Wed, 15 Sep 2010 23:51:40 +0200, Jens Rehsack > > wrote: > > > >> Ok, I understood your desire - but your patch doesn't catch all the > >> backward > >> compatibility you typically want - see (and apply)

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

2010-09-16 Thread Jens Rehsack
2010/9/16 Tim Bunce : > 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 15, 2010 at 10:4

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.p

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 : > On Wed, 15 Sep 2010 23:51:40 +0200, Jens Rehsack > 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 reaso