Bug #27342 for Net-SIP: Minor issues with Net::SIP

Wed May 30 08:38:44 2007 luismunoz [...] cpan.org - Ticket created

Subject:

Minor issues with Net::SIP

Hi there, First of all, thanks for writing your module. I'm likely to become a user of this code for a large, production application so I'm very interested in coordinating efforts with you - How can I help? I noticed the following things, to which I can happily provide patches if you want: * The README says (quote): "This is a module for handling SIP, the IETF standard for VOIP (RFC3289)." Shouldn't this be RFC-3261? * make test (and real use) of the code produces lots of "Pseudo-hashes are deprecated" warnings, probably because of the use of the 'fields' module. I would simply add a 'deprecated' to the 'use warnings' as I believe the code will do the right thing under perl 5.10... I'll wait for your comments. Thanks again and best regards. -lem

Wed May 30 16:47:25 2007 Steffen_Ullrich [...] genua.de - Correspondence added

From:

Steffen_Ullrich [...] genua.de

Show quoted text

> * The README says (quote): > > "This is a module for handling SIP, the IETF standard for VOIP (RFC3289)." > > Shouldn't this be RFC-3261?

You are right. A fix will be included in the next release. Show quoted text

> > * make test (and real use) of the code produces lots of "Pseudo-hashes > are deprecated" warnings, probably because of the use of the 'fields' > module. I would simply add a 'deprecated' to the 'use warnings' as I > believe the code will do the right thing under perl 5.10...

strange. I did not know that fields::new was depreceated for 5.10 and I did not found anything about it in the man page. I'm curently running perl5.8(.7) and don't get any warnings, but I'll try with 5.9.4. Which version of perl are you using? Steffen

Wed May 30 16:47:25 2007 The RT System itself - Status changed from 'new' to 'open'

Wed May 30 17:52:22 2007 Steffen_Ullrich [...] genua.de - Correspondence added

On Mi. 30. Mai 2007, 16:47:25, SULLR wrote: Show quoted text

> strange. I did not know that fields::new was depreceated for 5.10 > and I did not found anything about it in the man page. > I'm curently running perl5.8(.7) and don't get any warnings, but I'll > try with 5.9.4.

I've tried it with 5.9.4. I get some errors related to the reblessing of objects created by fields::new but once I worked around it (putting Hash::Util::unlock_keys before reblessing) all tests run successfully. So I cannot see any of the warnings you described.

Thu May 31 10:36:38 2007 luismunoz [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #27342] Minor issues with Net::SIP
Date:	Thu, 31 May 2007 10:34:50 -0400
To:	bug-Net-SIP [...] rt.cpan.org
From:	"Luis E. Muñoz" <LUISMUNOZ [...] cpan.org>

On May 30, 2007, at 5:52 PM, Steffen Ullrich via RT wrote: Show quoted text

> So I cannot see any of the warnings you described.

I confirmed this was a problem on my end. An old fink-based Perl installation was fighting with a newer macport based Perl installation. After getting rid of the fink one, tests do not produce a single warning with Net::SIP 0.28. Thanks for the help and sorry for the false alarm :-) Regards. -lem

Thu May 31 17:06:09 2007 Steffen_Ullrich [...] genua.de - Correspondence added

Show quoted text

> Thanks for the help and sorry for the false alarm :-)

I'm glad to know it was no real bug. But nevertheless thanks for your input Regards, Steffen

Thu May 31 17:06:11 2007 Steffen_Ullrich [...] genua.de - Status changed from 'open' to 'rejected'