Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Requirements fixup #28

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion MANIFEST
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,6 @@ lib/POE/Wheel/SocketFactory.pm
mylib/Devel/Null.pm
mylib/ForkingDaemon.pm
mylib/MyOtherFreezer.pm
mylib/PoeBuildInfo.pm
mylib/coverage.perl
mylib/cpan-test.perl
mylib/events_per_second.pl
Expand Down
123 changes: 104 additions & 19 deletions Makefile.PL
Original file line number Diff line number Diff line change
Expand Up @@ -82,17 +82,102 @@ unless ($ret =~ /^Y$/i) {

print "\n";

use lib qw(./mylib);
use PoeBuildInfo qw(
TEST_FILES
CLEAN_FILES
CORE_REQUIREMENTS
DIST_ABSTRACT
DIST_AUTHOR
CONFIG_REQUIREMENTS
HOMEPAGE
REPOSITORY
);
sub CONFIG_REQUIREMENTS () {
(
"POE::Test::Loops" => '1.360',
);
}

sub CORE_REQUIREMENTS () {
my @core_requirements = (
"Carp" => 0,
"Errno" => '1.09',
"Exporter" => 0,
"File::Spec" => '0.87',
"IO" => '1.24', # MSWin32 blocking(0)
"IO::Handle" => '1.27',
"IO::Pipely" => '0.005',
"POSIX" => '1.02',
"Socket" => '1.7',
"Storable" => '2.16',
"Test::Harness" => '2.26',
"Time::HiRes" => '1.59',
CONFIG_REQUIREMENTS,
);

if (-f 'META.yml') { # only include dynamic deps when installing
if ($^O eq "MSWin32") {
push @core_requirements, (
"Win32::Console" => '0.031',
"Win32API::File" => '0.05',
"Win32::Job" => '0.03',
"Win32::Process" => 0,
"Win32" => 0,
);
}
elsif ($^O eq 'cygwin') {
# Skip IO::Tty. It has trouble building as of this writing.
}
else {
push @core_requirements, (
"IO::Tty" => '1.08', # avoids crashes on fbsd
);
}
}

return @core_requirements;
}

sub DIST_AUTHOR () {
( 'Rocco Caputo <[email protected]>' )
}

sub DIST_ABSTRACT () {
( 'Portable, event-loop agnostic eventy networking and multitasking.' )
}

sub CLEAN_FILES () {
my @clean_files = qw(
*/*/*/*/*~
*/*/*/*~
*/*/*/*~
*/*/*~
*/*~
*~
META.yml
Makefile.old
bingos-followtail
coverage.report
poe_report.xml
run_network_tests
t/20_resources/10_perl
t/20_resources/10_perl/*
t/20_resources/20_xs
t/20_resources/20_xs/*
t/30_loops
t/30_loops/*
t/30_loops/*/*
test-output.err
);
"@clean_files";
}

sub TEST_FILES () {
my @test_files = qw(
t/*.t
t/*/*.t
t/*/*/*.t
);
"@test_files";
}

sub REPOSITORY () {
'https://github.com/rcaputo/poe'
}

sub HOMEPAGE () {
'http://poe.perl.org/'
}

### Touch files that will be generated at "make dist" time.
### ExtUtils::MakeMaker will complain about them if
Expand Down Expand Up @@ -142,14 +227,14 @@ sub check_for_modules {
check_for_modules("required", CORE_REQUIREMENTS);
check_for_modules(
"optional",
"Compress::Zlib" => 1.33,
"Curses" => 1.08,
"IO::Poll" => 0.01,
"IO::Pty" => 1.02,
"LWP" => 5.79,
"Term::Cap" => 1.10,
"Term::ReadKey" => 2.21,
"URI" => 1.30,
"Compress::Zlib" => '1.33',
"Curses" => '1.08',
"IO::Poll" => '0.01',
"IO::Pty" => '1.02',
"LWP" => '5.79',
"Term::Cap" => '1.10',
"Term::ReadKey" => '2.21',
"URI" => '1.30',
);

# check for optional IPv6 stuff
Expand Down
121 changes: 0 additions & 121 deletions mylib/PoeBuildInfo.pm

This file was deleted.