Skip Menu |

This queue is for tickets about the RT-Extension-SMSNotify CPAN distribution.

Report information
The Basics
Id: 106650
Status: new
Priority: 0/
Queue: RT-Extension-SMSNotify

People
Owner: Nobody in particular
Requestors: dom [...] cpan.org
Cc:
AdminCc:

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



Subject: POD errors
The attached patch fixes invalid POD, which causes errors in pod2man output (for example). Cheers, Dominic.
Subject: 0002-Fix-invalid-POD.patch
From bcc86f7905c1e2885572359684b8495dde6c779e Mon Sep 17 00:00:00 2001 From: Dominic Hargreaves <dom@earth.li> Date: Tue, 25 Aug 2015 11:16:07 +0100 Subject: [PATCH 2/2] Fix invalid POD Patch-Name: pod_fixes.patch --- lib/RT/Extension/SMSNotify.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/RT/Extension/SMSNotify.pm b/lib/RT/Extension/SMSNotify.pm index 1140178..147d24b 100644 --- a/lib/RT/Extension/SMSNotify.pm +++ b/lib/RT/Extension/SMSNotify.pm @@ -186,7 +186,7 @@ or to send direct to a specified phone number: rt-crontool --search RT::Search::FromSQL --search-arg 'Id = 1033' --action RT::Action::SMSNotify --action-arg "p:+1234123132" --template 'SLA Alert SMS' -=head TEMPLATES +=head1 TEMPLATES Unlike email templates, your SMS templates don't need a header. RT will complain if you don't leave the first line blank in your template as it thinks @@ -213,7 +213,7 @@ For example, a template for due date alerting could be: Craig Ringer <craig@2ndquadrant.com> -=COPYRIGHT +=head1 COPYRIGHT Copyright 2013 Craig Ringer -- 2.1.4
On Tue Aug 25 11:25:34 2015, DOM wrote: Show quoted text
> The attached patch fixes invalid POD, which causes errors in pod2man > output (for example).
Additionally, we are applying another patch in Debian to fix the NAME entries for pod2man, attached.
Subject: fix_whatis.patch
From 3afadee2429be77a9bb6acb390abeb50f4b78812 Mon Sep 17 00:00:00 2001 From: Dominic Hargreaves <dom@earth.li> Date: Tue, 25 Aug 2015 11:30:27 +0100 Subject: Fix NAME entries This fixes the Lintian warning manpage-has-bad-whatis-entry Patch-Name: fix_whatis.patch Forwarded: https://rt.cpan.org/Ticket/Display.html?id=106650 --- lib/RT/Action/SMSNotify.pm | 2 +- lib/RT/Extension/SMSNotify/OnShift.pm | 2 +- lib/RT/Extension/SMSNotify/PagerForUser.pm | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/RT/Action/SMSNotify.pm b/lib/RT/Action/SMSNotify.pm index a61c11c..fd7cbdf 100644 --- a/lib/RT/Action/SMSNotify.pm +++ b/lib/RT/Action/SMSNotify.pm @@ -15,7 +15,7 @@ use base qw(RT::Action); =head1 NAME -RT::Action::SMSNotify +RT::Action::SMSNotify - an action to send SMS notifications =head1 DESCRIPTION diff --git a/lib/RT/Extension/SMSNotify/OnShift.pm b/lib/RT/Extension/SMSNotify/OnShift.pm index 35abdb3..7947cf5 100644 --- a/lib/RT/Extension/SMSNotify/OnShift.pm +++ b/lib/RT/Extension/SMSNotify/OnShift.pm @@ -11,7 +11,7 @@ use RT::Extension::SMSNotify::Shifts; =head1 NAME - RT::Extension::SMSNotify::IsOnShift +RT::Extension::SMSNotify::IsOnShift - enable notifications based on shifts =head1 DESCRIPTION diff --git a/lib/RT/Extension/SMSNotify/PagerForUser.pm b/lib/RT/Extension/SMSNotify/PagerForUser.pm index 4cc9dd1..a8dd687 100644 --- a/lib/RT/Extension/SMSNotify/PagerForUser.pm +++ b/lib/RT/Extension/SMSNotify/PagerForUser.pm @@ -9,7 +9,7 @@ use DateTime; =head1 NAME - RT::Extension::SMSNotify::PagerForUser +RT::Extension::SMSNotify::PagerForUser - pager number lookup handler =head1 DESCRIPTION