This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 200278

Summary: Update commons-net library to 3.0.1
Product: ide Reporter: Tomas Mysik <tmysik>
Component: libsAssignee: Tomas Mysik <tmysik>
Status: RESOLVED FIXED    
Severity: normal CC: apireviews, efbiaiinzinz, tmysik
Priority: P2 Keywords: API, API_REVIEW_FAST
Version: 7.1   
Hardware: All   
OS: All   
Issue Type: ENHANCEMENT Exception Reporter:
Bug Depends on:    
Bug Blocks: 195879, 198078    
Attachments: patch 1: library update
patch 2: library dependencies
patch 3: dependent module versions
patch 4: Jakarta ORO removal

Description Tomas Mysik 2011-07-21 08:19:41 UTC
In order to be able to fix issue #195879 and implement enhancement #198078, commons-net library bundled in NetBeans must be upgraded to version 3.0.1.

This will be incompatible change, since currently we have version 1.4.1. Also, bundled Jakarta ORO library can be removed (friend API, only the current version of commons-net depends on it).
Comment 1 Tomas Mysik 2011-07-21 08:31:22 UTC
Created attachment 109546 [details]
patch 1: library update
Comment 2 Tomas Mysik 2011-07-21 08:36:19 UTC
Created attachment 109547 [details]
patch 2: library dependencies
Comment 3 Tomas Mysik 2011-07-21 08:39:28 UTC
Created attachment 109548 [details]
patch 3: dependent module versions
Comment 4 Tomas Mysik 2011-07-21 08:39:46 UTC
Created attachment 109549 [details]
patch 4: Jakarta ORO removal
Comment 5 Tomas Mysik 2011-07-21 08:51:53 UTC
Please review, the motivation can be found in the initial description.

I have tested functionality of the commons-net library in PHP and Mobility area and haven't found any problem. If there is something more I can/must do, please let me know.

Thanks.
Comment 6 Tomas Mysik 2011-07-26 13:49:12 UTC
Just one note: it seems that package "org.apache.commons.net.telnet" can be removed from the public API. I will test it and remove it if it's true.
Comment 7 Tomas Mysik 2011-07-27 13:33:14 UTC
Thanks for the review, I will integrate tomorrow.