I think that you should put this back. However, before you do that, I'd appreciate it if you would shorten the download_and_preexecute() method name to something a bit more concise, like preexecute(). It's a stylistic nit, sure. However, I find typing really long variable names is a pain (literally).
Thanks, -j On Fri, Jul 11, 2008 at 01:44:54PM -0700, Brock Pytlik wrote: > A pretty simple and straightforward change I think. > It splits the execute function in imageplan into two steps and moves the > be/snapshot creation to be after the packages have been downloaded to > disk, but before any actual actions have been performed. > > Here's the webrev: > http://cr.opensolaris.org/~bpytlik/ips-1249/ > > Let me know what you think. > 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
