Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 352484 - [pkgAdmin] refreshPackages incorrectly spans regions
Summary: [pkgAdmin] refreshPackages incorrectly spans regions
Status: RESOLVED FIXED
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: Framework (show other bugs)
Version: 3.7   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.7.1   Edit
Assignee: Thomas Watson CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 351519
Blocks:
  Show dependency tree
 
Reported: 2011-07-19 12:05 EDT by Thomas Watson CLA
Modified: 2011-07-19 13:17 EDT (History)
5 users (show)

See Also:


Attachments
3.7.1 patch + test (10.66 KB, patch)
2011-07-19 12:11 EDT, Thomas Watson CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Watson CLA 2011-07-19 12:05:26 EDT
Bug to backport fix from bug351519

+++ This bug was initially created as a clone of Bug #351519 +++

Background:
Eclipse Virgo currently runs in a multi-region environment implemented with the help of the recently introduced framework hooks. In both regions it includes SpringDM and Virgo's kernel depends on Spring DM.

When refreshing PackageAdmin gathers all bundles with equal symbolic name and refreshes them all because of https://bugs.eclipse.org/bugs/show_bug.cgi?id=169593. 
This behaviour is fine in a single region framework.
Also PackageAdmin has a global view in a multi-region enviroment as its provided by the framework which too has global view of the system.

Problem:
In Virgo refreshing Spring DM in the USER region, actually refreshes Spring DM also in the KERNEL region and with this a whole lot more of kernel bundles.

I would expect that refreshPackages doesn't span across regions, but work by regarding each region as a separate "refresh area" as was with the old implementation of RFC 138 with two frameworks and two package admins respectively.
Comment 1 Thomas Watson CLA 2011-07-19 12:11:46 EDT
Created attachment 199916 [details]
3.7.1 patch + test
Comment 2 Thomas Watson CLA 2011-07-19 12:19:35 EDT
patch released for 3.7.1