Skip Menu |

This queue is for tickets about the Poet CPAN distribution.

Report information
The Basics
Id: 77676
Status: open
Priority: 0/
Queue: Poet

People
Owner: Nobody in particular
Requestors: fschlich [...] zedat.fu-berlin.de
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 0.11
Fixed in: (no value)



Subject: typos in pod
Hi, while working on packaging Poet for Debian, our QA tools called my attention to few typos in the pod, and I'd like to share the patch for inclusion at your convenience: --- a/lib/Poet/Conf.pm +++ b/lib/Poet/Conf.pm @@ -502,7 +502,7 @@ cache.defaults.depth: 2 -then only C<depth> will be overriden; the C<driver> and C<root_dir> will remain +then only C<depth> will be overridden; the C<driver> and C<root_dir> will remain as they were set in C<global/cache.cfg>. If instead local.cfg had cache: --- a/lib/Poet/Environment.pm +++ b/lib/Poet/Environment.pm @@ -249,7 +249,7 @@ =head1 CONFIGURING ENVIRONMENT SUBDIRECTORIES -Any subdirectories other than conf_dir can be overriden in configuration. e.g. +Any subdirectories other than conf_dir can be overridden in configuration. e.g. # Override bin_dir env.bin_dir: /some/other/bin/dir --- a/lib/Poet/Mason.pm +++ b/lib/Poet/Mason.pm @@ -206,7 +206,7 @@ =item clear_and_abort (status) -These methods are overriden to set the response status before aborting, if +These methods are overridden to set the response status before aborting, if I<status> is provided. e.g. to send back a FORBIDDEN result: $m->clear_and_abort(403); On a side note, I was forced to install File::Copy::Recursive in addition to the modules mentioned in META.yml or test_dot_files_in_share_dir() from lib/Poet/t/Environment.pm wouldn't pass. I wonder if that method isn't really an author test and shouldn't be run *unless* (not *if*) $ENV{AUTHOR_TESTING} is set? Florian
Subject: Re: [rt.cpan.org #77676] typos in pod
Date: Thu, 7 Jun 2012 12:29:54 -0700
To: bug-Poet [...] rt.cpan.org
From: Jonathan Swartz <swartz [...] pobox.com>
Man, I cannot spell overridden right! :) Fixed in next release. Thanks Jon On Jun 6, 2012, at 2:48 PM, Florian via RT wrote: Show quoted text
> Wed Jun 06 17:48:21 2012: Request 77676 was acted upon. > Transaction: Ticket created by fschlich > Queue: Poet > Subject: typos in pod > Broken in: 0.11 > Severity: Wishlist > Owner: Nobody > Requestors: fschlich@zedat.fu-berlin.de > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=77676 > > > > Hi, > > while working on packaging Poet for Debian, our QA tools called my > attention to few typos in the pod, and I'd like to share the patch for > inclusion at your convenience: > > > --- a/lib/Poet/Conf.pm > +++ b/lib/Poet/Conf.pm > @@ -502,7 +502,7 @@ > > cache.defaults.depth: 2 > > -then only C<depth> will be overriden; the C<driver> and C<root_dir> > will remain > +then only C<depth> will be overridden; the C<driver> and C<root_dir> > will remain > as they were set in C<global/cache.cfg>. If instead local.cfg had > > cache: > --- a/lib/Poet/Environment.pm > +++ b/lib/Poet/Environment.pm > @@ -249,7 +249,7 @@ > > =head1 CONFIGURING ENVIRONMENT SUBDIRECTORIES > > -Any subdirectories other than conf_dir can be overriden in > configuration. e.g. > +Any subdirectories other than conf_dir can be overridden in > configuration. e.g. > > # Override bin_dir > env.bin_dir: /some/other/bin/dir > --- a/lib/Poet/Mason.pm > +++ b/lib/Poet/Mason.pm > @@ -206,7 +206,7 @@ > > =item clear_and_abort (status) > > -These methods are overriden to set the response status before aborting, > if > +These methods are overridden to set the response status before > aborting, if > I<status> is provided. e.g. to send back a FORBIDDEN result: > > $m->clear_and_abort(403); > > > On a side note, I was forced to install File::Copy::Recursive in > addition to the modules mentioned in META.yml or > test_dot_files_in_share_dir() from lib/Poet/t/Environment.pm wouldn't > pass. I wonder if that method isn't really an author test and shouldn't > be run *unless* (not *if*) $ENV{AUTHOR_TESTING} is set? > > Florian