Hi Liane, * Liane Praza ([email protected]) wrote: > On 07/13/10 03:50 PM, Glenn Lagasse wrote: > >[I apologize for anyone that recieves this in duplicate, it was pointed > >out to me that my mailer had munged the CC list and so people that were > >meant to be copied, weren't] > > > >The ON SNAP Integration project is ready for a code review. Since we're > >moving existing code in slim_source into ON I've tried to make things a > >little easier to review. > > > >I have two webrevs/workspaces for review. The first workspace is based > >on slim_source and is sync'ed with Jack's latest changes that he's > >already got a review out for. This workspace has the file differences > >for the existing files that are moving from slim_source into ON. This > >workspace should be used to review only files that aren't "new" > >as part of the integration into ON*. That list of files is: > > >The webrev for this is: > > > >http://cr.opensolaris.org/~glagasse/hg_beadm_1_noinstallgrub/ > > > > >http://cr.opensolaris.org/~glagasse/onnv-clone > > I only took a quick glance through the packaging changes. > > install-beadm.mf > l30 should be deleted, per > http://onnv.sfbay.sun.com/flagdays/pages/20100614015030.html
> Please run pkgfmt over this manifest before integrating it. Will do. > SUNWbeadm.mf > l28 same as above Will do. > You also seem to be moving libbe.so from system/library/install to > install/beadm and changing it at the same time as doing a > cross-consolidation move. That's a bit exciting since both packages > will own the file for developers for a little while. I would have > tried to avoid moving and changing files at the same time. So, > please make sure your testing of an ON update without a > corresponding Install update goes well. The way this works in my testing is that the 'new' copy of libbe.so (where new=the one delivered from ON) does get installed to the system. But yes, there will be a period of time (until the WOS build comes out that has the full ON changes and the full slim_source changes to remove beadm/libbe from slim_source) where libbe.so will be owned by two different packages. Thanks for having a look Liane. I really appreciate it. Glenn _______________________________________________ caiman-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
