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 177945 - Do not require impl deps on api.progress
Summary: Do not require impl deps on api.progress
Status: RESOLVED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: Progress (show other bugs)
Version: 6.x
Hardware: PC Linux
: P3 normal (vote)
Assignee: Jesse Glick
URL:
Keywords: API
Depends on:
Blocks: 179289
  Show dependency tree
 
Reported: 2009-12-01 12:41 UTC by Jesse Glick
Modified: 2010-02-03 16:47 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:

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 Jesse Glick 2009-12-01 12:41:12 UTC 
Currently two modules have implementation dependencies on api.progress:

core.execution - uses module.Controller and spi.InternalHandle from Install

progress.ui - uses various spi.* and module.ProgressListAction from various classes

We are trying to remove impl deps from the platform cluster and these seem pretty easily solvable. Would need to move Controller and ProgressListAction into the spi package, review what this package exports for basic API design principles, and make it public.
Comment 1 Jesse Glick 2010-01-07 15:56:19 UTC 
Implemented in prototypes #bb9ae35d671c, though SPI package probably needs cleanup and documentation before finalized.
Comment 2 Jesse Glick 2010-02-03 16:47:01 UTC 
Fix is in trunk, see master issue.