Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Dist-Zilla-Plugin-CoalescePod CPAN distribution.

Report information
The Basics
Id: 99440
Status: patched
Priority: 0/
Queue: Dist-Zilla-Plugin-CoalescePod
People
Owner: Nobody in particular
Requestors: tsibley [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.2.1
Fixed in: (no value)

History Show all quoted text
  Sat Oct 11 18:43:12 2014 tsibley [...] cpan.org - Ticket created
Subject: Inject before __END__ markers too
The split pattern should be /(^__(?:DATA|END)__$)/m since __END__ markers act the same as __DATA__.
  Sun Oct 12 11:08:18 2014 yanick+cpan [...] babyl.dyndns.org - Correspondence added
On Sat Oct 11 18:43:12 2014, TSIBLEY wrote: Show quoted text
> The split pattern should be /(^__(?:DATA|END)__$)/m since __END__ > markers act the same as __DATA__.
I'll be darned; I didn't know about __END__ and __DATA__ being interchangeable. I'll be more specific in the POD about the plugin's behavior, but I think I'll keep the current behavior. In most cases when peeps put __END__, it's really to indicate the end of the Perl code, and in those cases putting the POD after it is the right thing to do (and it seems that __END__ only has that special magic for the original calling script -- http://perldoc.perl.org/perldata.html#Special-Literals). In all cases, thanks for the heads up! Joy, `/anick
  Sun Oct 12 11:08:18 2014 The RT System itself - Status changed from 'new' to 'open'
  Sun Oct 12 11:08:19 2014 yanick+cpan [...] babyl.dyndns.org - Status changed from 'open' to 'patched'