[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [p2-dev] Feature patch and GC

It is true that the UI could probably show things a bit differently. Maybe simply showing the feature patch with a specific icon could already help. Please open an enhancement request about that.
As for the GC behavior, this is a controversial issue. Typically in the product scenarios that I have seen, the patches are here to stay and keeping the old plug-ins would probably result in comments that are just at the opposite of yours :). Also, p2 patches can actually patch any dependency and as a result the size of the thing being "patched out" can be non-trivial such that you would not want to keep it.
To get out of the business of tweaking the GC for every possible use, the GC is extensible and one could implement the behaviour you are after.

HTH

PaScaL

Inactive hide details for James D Miles ---04/07/2009 11:36:18 AM---It seems to me that the garbage collector is too aggressiveJames D Miles ---04/07/2009 11:36:18 AM---It seems to me that the garbage collector is too aggressive in certain situations. Here is the scena


From:

James D Miles <jdmiles@xxxxxxxxxx>

To:

p2-dev@xxxxxxxxxxx

Date:

04/07/2009 11:36 AM

Subject:

[p2-dev] Feature patch and GC




It seems to me that the garbage collector is too aggressive in certain situations. Here is the scenario.
fA100 + pA100 installed
feature patch fp100 patches fA100 with pA101
Install fp100 now fA and platform are running with pA101

Both fA and fp are shown in p2 UI
Both fA100, fp100, pA100, and pA101 are on file system.
pA101 is loaded into OSGi

Now
- remove preferences for repositories used above.
- run garbage collector
- Both fA and fp are shown in p2 UI
- Both fA, fp,
pA100, and pA101 are on file system. (pA100 is removed from file system)
- pA101 is loaded into OSGi

The garbage collector has removed pA100.
Everything needed to run the system is available.

If a user now attempts to uninstall the patch or revert we will get an error
So the message is correct but I think from the users standpoint it is strange. The system clear shows in the p2 UI that fA100 is available and installed but it is also telling him it is not fully installed.

To me it is a bug to show the feature/IU installed but not have the artifacts available. Comments?

It likely would be easier to fix the aggressive GC than to somehow help the user locate the missing repository.
_______________________________________________
p2-dev mailing list
p2-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/p2-dev

GIF image

GIF image