[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [p2-dev] Removing Operand and ProvisioningPlan from API

IProvisioningPlan has methods named getAdditions() and getRemovals() returning an IQueryable.
I'm also planning on adding a method that allows one to get the "future" state (just in terms of IUs), which would be equivalent to: profile - removals + additions

I think this should suit your needs.


Inactive hide details for Thomas Hallgren ---17/12/2009 06:19:57 PM---Hi Pascal,Thomas Hallgren ---17/12/2009 06:19:57 PM---Hi Pascal,


From:

Thomas Hallgren <thomas@xxxxxxx>

To:

p2-dev@xxxxxxxxxxx

Date:

17/12/2009 06:19 PM

Subject:

Re: [p2-dev] Removing Operand and ProvisioningPlan from API




Hi Pascal,
In the Buckminster Aggregator (and the Galileo Builder) we use the planner to verify the consistency of the resulting repository, and also to tell us exactly what we need to copy. We obtain that information by examining the operands in the resulting plan.

Will we still be able to extract that information? I.e. what exactly will the engine install given the plan.

- thomas


On 2009-12-17 17:59, Pascal Rapicault wrote:
      Hi,

      I just committed changes toward getting rid of Operand as part of the API. This should also allow to get rid of ProvisioningPlan (the class would no longer be API, but IProvisioningPlan would of course stay).
      Rather than letting ppl create operands manually as they use to do, they can now create a ProfileChangeRequest in absolute mode (setAbsoluteMode()) and use the various adders and setters on the PCR.
      I have changed all the known places in the p2 code, and adapted the SimplePlanner to handle this properly. So far so good. I still have to change the PDE code, but this should be straight forward.

      Now comes the case of the tests where there is a handful of tests that are creating plan for testing the engine, and I feel compelled to just use the ProvisioningPlan constructor.

      If you have any comment about this work, please comment on bug
      https://bugs.eclipse.org/bugs/show_bug.cgi?id=298097

      PaScaL



      _______________________________________________
      p2-dev mailing list
      p2-dev@xxxxxxxxxxx
      https://dev.eclipse.org/mailman/listinfo/p2-dev
       

_______________________________________________
p2-dev mailing list
p2-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/p2-dev


GIF image

GIF image