Thanks, I'll do that (I tend towards verbose names and then use 
completion to help me w/ the pain so definitely drag me back to 
normality when I venture too far down that path.)

Brock

[EMAIL PROTECTED] wrote:
> 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

Reply via email to