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

IO::Socket::IP is required not only on test time #36

Merged
merged 1 commit into from
Apr 7, 2015

Conversation

bjakubski
Copy link
Contributor

EmptyPort.pm uses IO::Socket::IP so it is not a test-only dependency.

Unless one have IO::Socket::IP installed already this means that dist simply fails to compile despite installing all declared dependencies (when test-only deps were not installed, i.e. when "cpanm --notest" was used)

kazuho added a commit that referenced this pull request Apr 7, 2015
IO::Socket::IP is required not only on test time
@kazuho kazuho merged commit 65cd951 into tokuhirom:master Apr 7, 2015
kazuho added a commit that referenced this pull request Apr 7, 2015
Changelog diff is:

diff --git a/Changes b/Changes
index b31475c..d96882d 100644
--- a/Changes
+++ b/Changes
@@ -2,6 +2,9 @@ Revision history for Perl module Test::TCP

 {{$NEXT}}

+2.11 2015-04-07T00:07:25Z
+    - declare IO::Socket::IP as dependency #36
+
 2.10 2015-04-06T19:23:43Z
     - ensure the test object is DESTROYed when Net::EmptyPort::empty_port exits https://rt.cpan.org/Public/Bug/Display.html?id=103299
@kazuho
Copy link
Collaborator

kazuho commented Apr 7, 2015

Thank you for noticing the issue! Merged and shipped as 2.11.

tokuhirom added a commit that referenced this pull request May 18, 2015
Changelog diff is:

diff --git Changes Changes
index d96882d..2c5eb49 100644
--- Changes
+++ Changes
@@ -2,6 +2,11 @@ Revision history for Perl module Test::TCP

 {{$NEXT}}

+2.12 2015-05-18T08:14:30Z
+
+    - Fixed spelling mistake
+      (Reported by gregor herrmann)
+
 2.11 2015-04-07T00:07:25Z
     - declare IO::Socket::IP as dependency #36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants