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 240019 - Removal of the Spring's public-packages
Summary: Removal of the Spring's public-packages
Status: RESOLVED FIXED
Alias: None
Product: javaee
Classification: Unclassified
Component: Spring (show other bugs)
Version: 8.0
Hardware: All All
: P2 normal (vote)
Assignee: Martin Fousek
URL:
Keywords: API_REVIEW_FAST
Depends on:
Blocks: 239612
  Show dependency tree
 
Reported: 2014-01-06 10:59 UTC by Martin Fousek
Modified: 2014-01-07 07:55 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
patch v1 (2.07 KB, patch)
2014-01-06 10:59 UTC, Martin Fousek
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Fousek 2014-01-06 10:59:55 UTC
Created attachment 143623 [details]
patch v1

I would like to remove exposed public-packages of the libs.springframework module. These packages look to be exposed without any usage within the repository. It shouldn't have any impact to standard modules and I don't know and I'm pretty sceptic about any usage by the community.

BTW, these public packages prevents removal of the obsolete Spring 2.5.6 JAR which we are going to replace with the latest versions of the Spring library.

Please review. I would need to integrate it tomorrow to get it into nb80 beta branch. Thanks a lot.
Comment 1 Jaroslav Tulach 2014-01-06 11:47:44 UTC
Bumping major version seems sufficient in this case to me. If nobody else objects, go on. Btw. here is the official howto: http://wiki.netbeans.org/VersioningPolicy#Incompatible_change
Comment 2 Martin Fousek 2014-01-07 07:55:05 UTC
(In reply to Jaroslav Tulach from comment #1)
> Bumping major version seems sufficient in this case to me. If nobody else
> objects, go on. Btw. here is the official howto:
> http://wiki.netbeans.org/VersioningPolicy#Incompatible_change

Thanks for information and the review!

Patch integrated in web-main #6129d412aa77.