Skip Menu |

This queue is for tickets about the Net-Graylog-Client CPAN distribution.

Report information
The Basics
Id: 99977
Status: resolved
Priority: 0/
Queue: Net-Graylog-Client

People
Owner: JALAVOY [...] cpan.org
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 0.3
Fixed in: 0.7



Subject: Net::Graylog::* also for older perls
It seems that Net::Graylog may work also for older perls < 5.16 (see attached patch). At least the test suite does not fail (checked with 5.12.3). (See also the other ticket about making App::Basis usable for older perls, which is a prerequisite here)
Subject: 0001-module-works-also-for-older-perls.patch
From 4e4bd9d60b2d2b045bf01fff01feae4583118244 Mon Sep 17 00:00:00 2001 From: Slaven Rezic <slaven.rezic@idealo.de> Date: Mon, 3 Nov 2014 11:55:54 +0100 Subject: [PATCH] module works also for older perls --- Makefile.PL | 2 -- lib/Net/Graylog/API.pm | 1 - 2 files changed, 3 deletions(-) diff --git a/Makefile.PL b/Makefile.PL index 41e59e4..5eefa46 100644 --- a/Makefile.PL +++ b/Makefile.PL @@ -2,8 +2,6 @@ use strict; use warnings; -use 5.016000; - use ExtUtils::MakeMaker 6.30; diff --git a/lib/Net/Graylog/API.pm b/lib/Net/Graylog/API.pm index 4ca97ea..21e7811 100644 --- a/lib/Net/Graylog/API.pm +++ b/lib/Net/Graylog/API.pm @@ -10,7 +10,6 @@ package Net::Graylog::API; $Net::Graylog::API::VERSION = '0.3'; } -use 5.16.0; use strict; use warnings; use Furl; -- 1.7.9.5
So this should be fixed. I removed the restrictions, and don't see anything incompatible with older perls (outside of a say() call in Net::Graylog::API), as well as the App::Basis problem you mentioned. I'm not maintainer on App::Basis so I can't do anything for that unfortunately, but I believe everything in this particular module should work fine on older perls now. If you find any problems let me know. On Mon Nov 03 05:57:41 2014, SREZIC wrote: Show quoted text
> It seems that Net::Graylog may work also for older perls < 5.16 (see > attached patch). At least the test suite does not fail (checked with > 5.12.3). (See also the other ticket about making App::Basis usable for > older perls, which is a prerequisite here)