What about OnApplyPhaseCount() or something down that line? Often can riff 
words off Bob and get something that sounds better.

The filler word “Of” in a variable or method name is usually a hint something 
is awry. <smile/>

_______________________________________________________________
FireGiant  |  Dedicated support for the WiX toolset  |  
http://www.firegiant.com/

From: Sean Hall [mailto:r.sean.h...@gmail.com]
Sent: Tuesday, May 27, 2014 7:21 PM
To: WiX toolset developer mailing list
Subject: Re: [WiX-devs] WIXFEAT:4413 Add WixBundleNumberOfApplyPhases

Rob, I think you made a comment in the last meeting about not liking the name 
of OnApplyNumberOfPhases.  I don't think it's very good either, but I'm coming 
up blank on a better name.  Any ideas?

On Fri, May 16, 2014 at 6:42 PM, Rob Mensching 
<r...@firegiant.com<mailto:r...@firegiant.com>> wrote:
Note: there is a difference between 4413 and 3736. The 3736 makes sense as a 
variable because it’s intended to be used in paths on ExePackages and such like 
that. I don’t see a scenario where the number of apply phases would ever be 
used as a variable.  Maybe as an InstallCondition or something… but that’s too 
late (IIRC, conditions are evaluated during Plan). <smile/>

Anyway, that’s the difference.

Also, to be clear, the reason the bug sat untouched for so long is that no one 
is working on it. There are lot of bugs and feature requests. Waiting for 
someone to work on your bug/feature request is playing the lottery. <smile/>

_______________________________________________________________
FireGiant  |  Dedicated support for the WiX toolset  |  
http://www.firegiant.com/

From: Sean Hall [mailto:r.sean.h...@gmail.com<mailto:r.sean.h...@gmail.com>]
Sent: Wednesday, May 7, 2014 5:10 PM
To: WiX toolset developer mailing list
Subject: Re: [WiX-devs] WIXFEAT:4413 Add WixBundleNumberOfApplyPhases

Neil - Maybe that's why it has sat untouched for so long? :)

As for being deprecated right out of the gate, <shrug />.  Weird things happen 
when you want backwards compatibility :)
------------------------------------------------------------------------------
Time is money. Stop wasting it! Get your web API in 5 minutes.
www.restlet.com/download
http://p.sf.net/sfu/restlet
_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs

Reply via email to