Michal Pryc wrote: > Brock, > I've seen new version ips-api-v7 which looks fine for me. > > Is it possible to push those changes to the gate Today? > I'm waiting on a final code review from the people over here. I'll make the rounds and see what kind of commitment I can get.
Brock > best > Michal > > > Brock Pytlik wrote: >> v6 fixes a few typos and fixes a problem where canceling an operation >> resulted in a deadlock. >> http://cr.opensolaris.org/~bpytlik/ips-api-v6/ >> >> This version passes all of the test suite. When put back, it will >> temporarily break removing packages using packagemanager. That will be >> resolved will the GUI gets updated to use the API, something I >> understand is being targeted for put back late this week. For this, and >> many other reasons, I'm targeting the beginning of 100 for this putback, >> and not the tail end of 99. >> >> Brock >> >> Brock Pytlik wrote: >>> v5 changes 6 lines, 147-148, 200-201, 310-311. This simply makes the >>> no-execute flag/do nothing argument make sure that prepare and >>> execute_plan cannot be called on the api_obj if the -n option is given. >>> Shawn's webrev inspired me to make sure this functionality was correct. >>> >>> http://cr.opensolaris.org/~bpytlik/ips-api-v5 >>> >>> Brock >>> >>> Brock Pytlik wrote: >>> >>>> v4 is simply a resync with the gate: >>>> http://cr.opensolaris.org/~bpytlik/ips-api-v4/ >>>> >>>> Brock Pytlik wrote: >>>> >>>> >>>>> Another new version is out. For the CLI and backend it's essentially >>>>> unchanged. It introduces changes to the GUI code to help it not break >>>>> when the API is put back. >>>>> >>>>> http://cr.opensolaris.org/~bpytlik/ips-api-v3/ >>>>> >>>>> John, Michal, could you please take a look at the changes to the GUI >>>>> code in general, and the changes to remove.py in particular. >>>>> Somehow my >>>>> changes seem to make the GUI never realize it's done evaluating the >>>>> removal plan, and I'm lost as to why remove would be showing this >>>>> behavior when install and image-update work. From looking at the >>>>> truss >>>>> output, it seems like it spends time reading files, then finishes, >>>>> and >>>>> just spins, updating progress bar. >>>>> >>>>> Thanks, >>>>> Brock >>>>> >>>>> Brock Pytlik wrote: >>>>> >>>>> >>>>> >>>>>> New webrev reflecting the comments I've gotten so far, and resyncing >>>>>> with the gate to gather in the new changes. >>>>>> >>>>>> http://cr.opensolaris.org/~bpytlik/ips-api-v2/ >>>>>> >>>>>> Thanks, >>>>>> Brock >>>>>> >>>>>> Brock Pytlik wrote: >>>>>> >>>>>> >>>>>> >>>>>> >>>>>>> Here's the link: >>>>>>> http://cr.opensolaris.org/~bpytlik/ips-api-v1/ >>>>>>> >>>>>>> I haven't done as much cleanup as I usually try to do, but since >>>>>>> I'm >>>>>>> ooto tomorrow, I thought it would be better to get this out now, >>>>>>> rather >>>>>>> than next Monday. >>>>>>> >>>>>>> Other things that I know need to be improved, but that I think >>>>>>> can wait >>>>>>> for another round of revisions: >>>>>>> API Exception organization >>>>>>> More complete functionality >>>>>>> Automated testing of cancellation (I hope to work with the GUI >>>>>>> team to >>>>>>> test this by hand in the interim) >>>>>>> Versioning of the Progress tracker (perhaps) >>>>>>> More Documentation >>>>>>> Figure out the future approach to testing we'll take >>>>>>> >>>>>>> Thanks, >>>>>>> Brock >>>>>>> _______________________________________________ >>>>>>> pkg-discuss mailing list >>>>>>> [email protected] >>>>>>> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss >>>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>>>> >>>>>> _______________________________________________ >>>>>> pkg-discuss mailing list >>>>>> [email protected] >>>>>> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss >>>>>> >>>>>> >>>>>> >>>>>> >>>>> _______________________________________________ >>>>> pkg-discuss mailing list >>>>> [email protected] >>>>> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss >>>>> >>>>> >>>>> >>>> _______________________________________________ >>>> pkg-discuss mailing list >>>> [email protected] >>>> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss >>>> >>>> >>> _______________________________________________ >>> pkg-discuss mailing list >>> [email protected] >>> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss >>> >> >> _______________________________________________ >> pkg-discuss mailing list >> [email protected] >> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss > _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
