Is there a meeting tomorrow? Maybe we can discuss this in the meeting.
I looked at 4.0 and didn't see very much difference in this corner of the
toolset (except all the Row classes being in the Data namepace now). Also,
this is purely internal right? I think if you want to rip them out you can do
it in 3.x, too.
From: r...@firegiant.com
To: wix-devs@lists.sourceforge.net
Date: Wed, 19 Feb 2014 17:05:33 +0000
Subject: Re: [WiX-devs] wixbug:3914 - bind variables in Variable fall through
to the manifest
The Info classes were a bad idea. Unfortunately, I didn’t realize how bad an
idea until there were a bunch of them and they were intertwined.
In WiX v4.0, I’m planning to rip them all out and replace them with simple Rows
(like everything else is). Not sure if that’s a good thing to do in v3.x so
if you want to try to fix this in WiX v3.x, you’ll probably need to be more
surgical.
_______________________________________________________________
FireGiant | Dedicated support for the WiX toolset |
http://www.firegiant.com/
From: Sean [mailto:rhal...@hotmail.com]
Sent: Tuesday, February 18, 2014 7:44 PM
To: WiX toolset developer mailing list
Subject: Re: [WiX-devs] wixbug:3914 - bind variables in Variable fall through
to the manifest
Sorry, looks like my email wasn't clear on what happens when. Here's how it
works now (my code didn't change this):
1. First pass at resolving variables.
2. Create Info classes from Rows.
3. Do stuff with Info classes, including setting bind variables.
3. Final pass at resolving variables.
4. Do more stuff with Info classes.
5. Generate manifest using mostly Info classes.
From:
os...@live.com
To: wix-devs@lists.sourceforge.net
Date: Wed, 19 Feb 2014 02:43:04 +0000
Subject: Re: [WiX-devs] wixbug:3914 - bind variables in Variable fall through
to the manifest
I haven’t looked at your code yet, but that sounds like a right approach to me.
The other possibility that comes to mind would be delaying generation of the
manifest until early in the final
pass after final variable resolution instead of when it is now. I’m not sure
what impact that would have on the code, however.
Blair
From: Sean
Sent: Sunday, February 16, 2014 5:03 PM
To: wix-devs@lists.sourceforge.net
I submitted a pull request for this bug:
http://wixtoolset.org/issues/3914/
https://wix.codeplex.com/SourceControl/network/forks/seanhall/wixbug3914/contribution/6236
The root cause of this bug is that the Binder uses different classes for
resolving delayed variables and creating the manifest. When resolving
variables, it uses Tables and Rows just like when binding an MSI. But between
the first pass at resolving variables
and the final pass, it creates Info classes which grab the intermediate output
from the Rows (there are a few exceptions where it doesn't create an Info
class, like the bundle update row). It then uses these Info classes to create
the manifest. This means
that if a row had a delayed field, then the field would be resolved but the
value wouldn't get into the manifest because the Info class had cached the
unresolved value.
I assumed that we want to continue to use the same code to resolve variables
when binding MSI's and bundles. So I created a VariableRow class, and made the
VariableInfo class store the VariableRow, using the row as the backing store.
There are many of these Info classes. I went ahead and did the same thing for
CatalogInfo. If you agree with this approach and accept the pull request, I
will work on the rest of the Info classes.
Thanks,
Sean
------------------------------------------------------------------------------
Android apps run on BlackBerry 10 Introducing the new BlackBerry 10.2.1 Runtime
for Android apps. Now with support for Jelly Bean, Bluetooth, Mapview and more.
Get your Android app
in front of a whole new audience. Start now.
http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk
_______________________________________________ WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs
------------------------------------------------------------------------------
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
------------------------------------------------------------------------------
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
------------------------------------------------------------------------------
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
