Since Apply already requires an HWnd, I actually don't think it's a big deal to 
change Detect to require one, too.
 
Sean
 
From: r...@robmensching.com
To: wix-devs@lists.sourceforge.net
Date: Thu, 23 Jan 2014 16:41:21 +0000
Subject: [WiX-devs] Self update bundles, the detect issue.









>From this pull request: 
https://wix.codeplex.com/SourceControl/network/forks/jchoover/Wix/contribution/5911
 I’m looking for more input on the issue surround the 
IBootstrapperEngine::Detect() signature changing. I see a couple ways around 
this:
 
1.      
Move the pull request to WiX v4.0 where we can “break” (in this case: enhance) 
the Detect API as we see fit.
2.      
Add another method to the IBootstrapperEngine called “DetectWIthUpdates()” or 
something (Jacob already proposed something like “Detect2()” which is not high 
on my list of good names <smile/>).
 
Personally, I’m leaning towards option #1. Option #2 isn’t completely horrible 
but there are already ways to skip update detection (if it is slow, for 
example) and adding a top level method seems overkill.
 
Comments? Suggestions?




------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today. 
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs                           
          
------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today. 
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs

Reply via email to