Bug 6602 – bootstrap.pm throws error for Net::UPnP version 1.2.1

Bug 6602 - bootstrap.pm throws error for Net::UPnP version 1.2.1

Summary:

bootstrap.pm throws error for Net::UPnP version 1.2.1

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Misc
Version:	7.0
Platform:	Sun Other

Importance:	P2 normal (vote)
Target Milestone:	7.x
Assigned To:	Andy Grundman

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-01-13 12:16 UTC by Carson Gaspar
Modified:	2011-03-16 04:34 UTC (History)
CC List:	1 user (show)

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Carson Gaspar 2008-01-13 12:16:23 UTC

With Net::UPnP 1.2.1 installed, slimserver.pl throws the following error at startup:

root:gandalf 0 # /tools/squeezecenter/slimserver.pl --logdir /var/log/slimserver --daemon --noupnp --prefsdir /etc/slimserver
Argument "1.2.1" isn't numeric in subroutine entry at /export/Tools/SunOS_5.10_i86pc_amd64/squeezecenter-7.0-16207-noCPAN/Slim/bootstrap.pm line 341.

carson:gandalf 0 PERL$ perl -e 'use Net::UPnP; print Net::UPnP->VERSION, "\n";'
1.2.1

Comment 1 Blackketter Dean 2008-01-14 10:11:21 UTC

Consider updating to 1.2.1 for 7.0

Comment 2 KDF 2008-03-11 00:37:19 UTC

I dont think the version of Net::UPNP is the problem. Its the way we handle it.  The problem line is:
eval { $mod->VERSION( $ver || 0 ); 1; };

All of the modules in modules.conf are of the form:
n.mmm

These are numeric due to having only ONE decimal point.  Net::UPnP instead is 1.2.1, which even when ||'d with 0 would probably not qualify as numeric.  hence, the error.

Comment 3 Andy Grundman 2008-04-01 14:06:31 UTC

Fixed in 7.0.1 change 18269.

Comment 4 James Richardson 2008-05-07 10:41:54 UTC

Carson:  Can you verify that this issue has been properly address for you?  If not,please reopen the bug

Comment 5 Chris Owens 2009-07-31 10:15:48 UTC

Reduce number of active targets for SC