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
