Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Parse-CSV CPAN distribution.

Report information
The Basics
Id: 27734
Status: open
Priority: 0/
Queue: Parse-CSV

People
Owner: Nobody in particular
Requestors: amain [...] wested.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.00
Fixed in: (no value)



Subject: Accepting patches for Parse::CSV?
Date: Mon, 25 Jun 2007 11:10:38 -0700
To: bug-Parse-CSV [...] rt.cpan.org
From: Andrew Main <amain [...] wested.org>
Hello Adam, I'm working with you lovely Parse::CSV module and have a couple features I'd like to add: * Support for a fields() method, so I don't have to do $parser->{fields} to get the field names (using fields => 'auto') before I start parsing the body of the file. * Support for embedded newlines--right now _getline uses: my $line = <$handle>; Rather than my $line = $csv->getline($handle) http://search.cpan.org/~hmbrand/Text-CSV_XS-0.29/CSV_XS.pm#Embedded_newlines Just wanted to confirm that you're accepting patches before I code this up with a some test cases. Thanks so much! -Andrew
From: MARKSTOS [...] cpan.org
On Mon Jun 25 14:11:11 2007, amain@wested.org wrote: Show quoted text
> Hello Adam, > > I'm working with you lovely Parse::CSV module and have a couple features > I'd like to add: > > * Support for a fields() method, so I don't have to do $parser->{fields} > to get the field names (using fields => 'auto') before I start parsing > the body of the file. > > * Support for embedded newlines--right now _getline uses: > > my $line = <$handle>; > > Rather than > > my $line = $csv->getline($handle) > >
http://search.cpan.org/~hmbrand/Text-CSV_XS-0.29/CSV_XS.pm#Embedded_newlines Show quoted text
> > Just wanted to confirm that you're accepting patches before I code this > up with a some test cases. > > Thanks so much! > > -Andrew
Andrew, Please proceed with a patch and post it here. Even if Adam doesn't accept it (and I suspect he would), the rest of us will benefit by having the patch in an easy-to-find public place, so we can patch our own copies if we'd like. I for one also am interested in embedded newline support and am a fan of Parse::CSV.
Subject: Re: [rt.cpan.org #27734] Accepting patches for Parse::CSV?
Date: Wed, 12 Dec 2007 14:58:50 +1100
To: bug-Parse-CSV [...] rt.cpan.org
From: "Adam Kennedy" <adamkennedybackup [...] gmail.com>
I don't typically accept patches as such (as I'm terrible with them), my normal preference is to provide contributors commit access to the repository and have them apply the improvements directly. That said, if Mark is willing to apply any patch, you are welcome to provide one. Adam K On 12/12/2007, MARKSTOS via RT <bug-Parse-CSV@rt.cpan.org> wrote: Show quoted text
> > > Queue: Parse-CSV > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=27734 > > > On Mon Jun 25 14:11:11 2007, amain@wested.org wrote:
> > Hello Adam, > > > > I'm working with you lovely Parse::CSV module and have a couple features > > I'd like to add: > > > > * Support for a fields() method, so I don't have to do $parser->{fields} > > to get the field names (using fields => 'auto') before I start parsing > > the body of the file. > > > > * Support for embedded newlines--right now _getline uses: > > > > my $line = <$handle>; > > > > Rather than > > > > my $line = $csv->getline($handle) > > > >
> > http://search.cpan.org/~hmbrand/Text-CSV_XS-0.29/CSV_XS.pm#Embedded_newlines
> > > > Just wanted to confirm that you're accepting patches before I code this > > up with a some test cases. > > > > Thanks so much! > > > > -Andrew
> > Andrew, > > Please proceed with a patch and post it here. Even if Adam doesn't > accept it (and I suspect he would), the rest of us will benefit by > having the patch in an easy-to-find public place, so we can patch our > own copies if we'd like. > > I for one also am interested in embedded newline support and am a fan of > Parse::CSV. > > > > >
Subject: Parse::CSV doesn't handle embeded newlines
From: MARKSTOS [...] cpan.org
I'm still interested in getting this fixed, but I'll need to know the repo address (it's not in the docs), as well as what username and password to use. I guessed that my PAUSE username/password might work, but they didn't. Because I forgot this bug existed in Parse::CSV, I tried to use it again, and will need to switch to raw Text::CSV_XS until this is fixed. I also noticed that Parse::CSV hardcodes within it the options to new() that Text::CSV_XS accepts. This means it needs to be maintained as Text::CSV_XS adds more options (and it already added one). It seems perhaps cleaner for Parse::CSV to keep track of its own options, and pass everything else to Text::CSV_XS. Mark On Tue Dec 11 22:59:15 2007, adamkennedybackup@gmail.com wrote: Show quoted text
> I don't typically accept patches as such (as I'm terrible with them), > my > normal preference is to provide contributors commit access to the > repository > and have them apply the improvements directly. > > That said, if Mark is willing to apply any patch, you are welcome to > provide > one. > > Adam K > > On 12/12/2007, MARKSTOS via RT <bug-Parse-CSV@rt.cpan.org> wrote:
> > > > > > Queue: Parse-CSV > > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=27734 > > > > > On Mon Jun 25 14:11:11 2007, amain@wested.org wrote:
> > > Hello Adam, > > > > > > I'm working with you lovely Parse::CSV module and have a couple
> features
> > > I'd like to add: > > > > > > * Support for a fields() method, so I don't have to do $parser-
> >{fields}
> > > to get the field names (using fields => 'auto') before I start
> parsing
> > > the body of the file. > > > > > > * Support for embedded newlines--right now _getline uses: > > > > > > my $line = <$handle>; > > > > > > Rather than > > > > > > my $line = $csv->getline($handle) > > > > > >
> > > > http://search.cpan.org/~hmbrand/Text-CSV_XS-
> 0.29/CSV_XS.pm#Embedded_newlines
> > > > > > Just wanted to confirm that you're accepting patches before I code
> this
> > > up with a some test cases. > > > > > > Thanks so much! > > > > > > -Andrew
> > > > Andrew, > > > > Please proceed with a patch and post it here. Even if Adam doesn't > > accept it (and I suspect he would), the rest of us will benefit by > > having the patch in an easy-to-find public place, so we can patch
> our
> > own copies if we'd like. > > > > I for one also am interested in embedded newline support and am a
> fan of
> > Parse::CSV. > > > > > > > > > >
Subject: Re: [rt.cpan.org #27734] Parse::CSV doesn't handle embeded newlines
Date: Thu, 27 Mar 2008 10:03:46 +1100
To: bug-Parse-CSV [...] rt.cpan.org
From: Adam Kennedy <adamkennedybackup [...] gmail.com>
My repository is temporarily down. I'll get back to you when it comes back up. Parse::CSV does provide the ability to provide options that get passed through to Text::CSV_XS... Adam K MARKSTOS via RT wrote: Show quoted text
> Queue: Parse-CSV > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=27734 > > > I'm still interested in getting this fixed, but I'll need to know the > repo address (it's not in the docs), as well as what username and > password to use. I guessed that my PAUSE username/password might work, > but they didn't. > > Because I forgot this bug existed in Parse::CSV, I tried to use it > again, and will need to switch to raw Text::CSV_XS until this is fixed. > > I also noticed that Parse::CSV hardcodes within it the options to new() > that Text::CSV_XS accepts. This means it needs to be maintained as > Text::CSV_XS adds more options (and it already added one). It seems > perhaps cleaner for Parse::CSV to keep track of its own options, and > pass everything else to Text::CSV_XS. > > Mark > > > On Tue Dec 11 22:59:15 2007, adamkennedybackup@gmail.com wrote: >
>> I don't typically accept patches as such (as I'm terrible with them), >> my >> normal preference is to provide contributors commit access to the >> repository >> and have them apply the improvements directly. >> >> That said, if Mark is willing to apply any patch, you are welcome to >> provide >> one. >> >> Adam K >> >> On 12/12/2007, MARKSTOS via RT <bug-Parse-CSV@rt.cpan.org> wrote: >>
>>> Queue: Parse-CSV >>> Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=27734 > >>> >>> On Mon Jun 25 14:11:11 2007, amain@wested.org wrote: >>>
>>>> Hello Adam, >>>> >>>> I'm working with you lovely Parse::CSV module and have a couple >>>>
>> features >>
>>>> I'd like to add: >>>> >>>> * Support for a fields() method, so I don't have to do $parser- >>>>
>>> {fields} >>>
>>>> to get the field names (using fields => 'auto') before I start >>>>
>> parsing >>
>>>> the body of the file. >>>> >>>> * Support for embedded newlines--right now _getline uses: >>>> >>>> my $line = <$handle>; >>>> >>>> Rather than >>>> >>>> my $line = $csv->getline($handle) >>>> >>>> >>>>
>>> http://search.cpan.org/~hmbrand/Text-CSV_XS- >>>
>> 0.29/CSV_XS.pm#Embedded_newlines >>
>>>> Just wanted to confirm that you're accepting patches before I code >>>>
>> this >>
>>>> up with a some test cases. >>>> >>>> Thanks so much! >>>> >>>> -Andrew >>>>
>>> Andrew, >>> >>> Please proceed with a patch and post it here. Even if Adam doesn't >>> accept it (and I suspect he would), the rest of us will benefit by >>> having the patch in an easy-to-find public place, so we can patch >>>
>> our >>
>>> own copies if we'd like. >>> >>> I for one also am interested in embedded newline support and am a >>>
>> fan of >>
>>> Parse::CSV. >>> >>> >>> >>> >>> >>>
> > > >
Andrew - if you still want to submit a patch, I've pulled the code into GitHub at https://github.com/kenahoo/Perl-Parse-CSV , you could submit a pull request there.