Skip Menu |

This queue is for tickets about the NetworkInfo-Discovery CPAN distribution.

Report information
The Basics
Id: 107465
Status: new
Priority: 0/
Queue: NetworkInfo-Discovery

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

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



Subject: Fails to compile; ICMP_MASREQ undefined
Module fails to compile, seen during test run: $ make test PERL_DL_NONLAZY=1 "/home/robn/perl5/perlbrew/perls/perl-5.20.0/bin/perl" "-Iblib/lib" "-Iblib/arch" test.pl 1..1 # Running under perl version 5.020002 for linux # Current time local: Fri Oct 2 23:14:21 2015 # Current time GMT: Fri Oct 2 13:14:21 2015 # Using Test.pm version 1.26 Bareword "ICMP_MASREQ" not allowed while "strict subs" in use at lib/NetworkInfo/Discovery/Sniff.pm line 214. Compilation failed in require at test.pl line 10. BEGIN failed--compilation aborted at test.pl line 10. Makefile:861: recipe for target 'test_dynamic' failed make: *** [test_dynamic] Error 255 This looks like a typo; ICMP_MASKREQ exists in my headers, ICMP_MASREQ does not. Attached patch causes the compile to succeed and the tests to pass; however, I have not tested any functionality yet.
Subject: nid-compile-fix.diff
diff -ur NetworkInfo-Discovery-0.12.orig/lib/NetworkInfo/Discovery/Sniff.pm NetworkInfo-Discovery-0.12/lib/NetworkInfo/Discovery/Sniff.pm --- NetworkInfo-Discovery-0.12.orig/lib/NetworkInfo/Discovery/Sniff.pm 2015-10-02 23:11:43.631461887 +1000 +++ NetworkInfo-Discovery-0.12/lib/NetworkInfo/Discovery/Sniff.pm 2015-10-02 23:12:03.791612380 +1000 @@ -235,8 +235,8 @@ $type = "ICMP_TSTAMPREPLY"; } elsif ($icmp->{type} == ICMP_IREQ ) { $type = "ICMP_IREQ"; - } elsif ($icmp->{type} == ICMP_MASREQ ) { - $type = "ICMP_MASREQ"; + } elsif ($icmp->{type} == ICMP_MASKREQ ) { + $type = "ICMP_MASKREQ"; } elsif ($icmp->{type} == ICMP_IREQREPLY ) { $type = "ICMP_IREQREPLY"; } elsif ($icmp->{type} == ICMP_MASKREPLY ) {