Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323340 - [planner] Resolution is not stable
Summary: [planner] Resolution is not stable
Status: RESOLVED FIXED
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: p2 (show other bugs)
Version: 3.6   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: 3.7 RC1   Edit
Assignee: Daniel Le Berre CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 323333
Blocks: 363965
  Show dependency tree
 
Reported: 2010-08-22 19:12 EDT by Pascal Rapicault CLA
Modified: 2011-11-16 14:28 EST (History)
2 users (show)

See Also:

Attachments
New patch (1.59 KB, application/octet-stream)
2010-08-22 19:17 EDT, Pascal Rapicault CLA 		no flags Details
Previous patch updated to be applied on HEAD. (1.77 KB, patch)
2011-05-02 15:26 EDT, Daniel Le Berre CLA 		pascal: review+
Details | Diff
Show Obsolete (1) View All 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 Pascal Rapicault CLA 2010-08-22 19:12:59 EDT 
+++ This bug was initially created as a clone of Bug #323322 +++

While investigating what is going wrong in shared installs, I came across the following issue that is probably the root cause of the problem for shared installs. 

The resolution is not stable when being re-run. To be more precise, when I try to install an IU in the context of some repositories, then cause a re-resolution of the resulting profile, IUs are being dropped.
Even though this looks not important, because internally p2 re-resolves the current profile to generate the attachment of IUs, this ends up being a key issue.

To show the problem, I have released a test case called EPPPackageInstallStability in the 3.6 branch. At this point I have not found a way to solve this problem.
Comment 1 Pascal Rapicault CLA 2010-08-22 19:17:18 EDT 
Created attachment 177191 [details]
New patch

This new patch simply removes the if block and cleans up the code. It will need the new version of SAT4J. Daniel, could you please review. I'll release once SAT4J 2.2.1 has been approved.
Comment 2 Daniel Le Berre CLA 2010-08-23 03:11:28 EDT 
The new code is fine.
Comment 3 Daniel Le Berre CLA 2011-05-02 15:26:04 EDT 
Created attachment 194534 [details]
Previous patch updated to be applied on HEAD.

I simply updated the previous patch with the conditional in case of singleton.
Comment 4 Pascal Rapicault CLA 2011-05-02 15:38:43 EDT 
I've released the new patch.