Well, this is why I sent the email.  <smile/>  It seemed to me that my approach 
could only go so far, there were classes like ChainPackageInfo where it would 
have taken way more work than I put into the VariableInfo class.  I didn't want 
to put in all that work just for it to fundamentally change in v4.x.
I thought that getting rid of the Info classes was already going to be a whole 
lot of work, and it sounds like you were thinking that it would be a part of an 
even bigger task.  Weren't you thinking about releasing 3.9 and 4.0 sometime in 
the summer?  If Bob is working on 3.9 and you're working on 4.0 (I assume you 
have plenty of other things you want to do in 4.0), who else would do this?
Side note: apparently if I kept the whole message chain in the email, it 
started to get too big.
From: r...@firegiant.com
To: wix-devs@lists.sourceforge.net
Date: Thu, 20 Feb 2014 04:58:42 +0000
Subject: Re: [WiX-devs] wixbug:3914 - bind variables in Variable fall through 
to the manifest









I got far enough to understand that it is a very large undertaking. <smile/>
 
Seriously, the Binder needs a massive scrubbing and refactoring that is greater 
than the work I’ve done to the linker thus far. If you really want to take it
 on, you’ll want to really kinda’ grok the whole thing. Which means we should 
probably talk for a while about it.
 
This isn’t a thing I’d typically suggest taking on as your third or fourth item 
in the WiX toolset. <smile/>
 
In WiX v3.x I’d highly recommend doing the targeted fix over the huge re-write. 
 I haven’t looked at your pull request but it sounds like that was the direction
 you were heading…
 

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

                                          
------------------------------------------------------------------------------
Managing the Performance of Cloud-Based Applications
Take advantage of what the Cloud has to offer - Avoid Common Pitfalls.
Read the Whitepaper.
http://pubads.g.doubleclick.net/gampad/clk?id=121054471&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