[wix-devs] Clean room defect with 3.11.2 - /layout [P]

[Hoover, Jacob]

apply.cpp 
 - LayoutBundle : should also check origional location?

plan.cpp
 - PlanLayoutBundle: PathForCurrentProcess should be changed to WixOriginalSource?

I think the plan part would be the quick fix, and the apply part is just for sanity (otherwise it will get a sharing violation).

-----Original Message-----
From: wix-devs [mailto:wix-devs-bounces at lists.wixtoolset.org] On Behalf Of Phill Hogland
Sent: Wednesday, January 27, 2016 3:29 PM
To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org>
Subject: Re: [wix-devs] Clean room defect with 3.11.2 - /layout [P]

Yes, -layout is broken.  Thanks for pointing this out.  I just finished doing -layout on 15 bundles and burned the final archive discs for our release (without actually looking at the folder to see as you reported that the files are not actually there.  This is a big problem for us.   Must go talk to the boss.

-----Original Message-----
From: wix-devs [mailto:wix-devs-bounces at lists.wixtoolset.org] On Behalf Of Hoover, Jacob
Sent: Wednesday, January 27, 2016 3:24 PM
To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org>
Subject: Re: [wix-devs] Clean room defect with 3.11.2 - /layout [P]

Right, but a layout is designed for a user to generate an offline install.  If they are doing a Layout, the old default was the CurrentDirectory, which now should be the original bundle location.  I have confirmed one can override this by providing an explicit path, but this is an unexpected and undesirable change in behavior.

-----Original Message-----
From: wix-devs [mailto:wix-devs-bounces at lists.wixtoolset.org] On Behalf Of Steven Ogilvie
Sent: Wednesday, January 27, 2016 3:21 PM
To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org>
Subject: Re: [wix-devs] Clean room defect with 3.11.2 - /layout [P]

Classification: Public
Yes

I ran it from here: 
C:\Temp\admin>TITUSIlluminateSetup.exe /layout

It copied the install package to: C:\Users\steven.ogilvie\AppData\Local\Temp\{633334F0-6A6E-47E4-950C-135018F23862}\.cr

>From the log file:
Burn v3.10.2.2522, Windows v6.1 (Build 7601: Service Pack 1), path: C:\Users\STEVEN~1.OGI\AppData\Local\Temp\{95591265-5AD4-41FD-A0B8-8C402CC0CBA7}\.cr\TITUSIlluminateSetup.exe

Steve




This message has been marked as Public by Steven Ogilvie on January-27-16 4:20:42 PM.

The above classification labels were added to the message by TITUS Message Classification. 
For more information visit www.titus.com.

-----Original Message-----
From: wix-devs [mailto:wix-devs-bounces at lists.wixtoolset.org] On Behalf Of Hoover, Jacob
Sent: January-27-16 4:08 PM
To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org>
Subject: [wix-devs] Clean room defect with 3.11.2 - /layout


It seems layout is broken with 3.10.2, as the payload goes to the .cr instead of the original exe location.  Can anyone else confirm this?

Thanks,
Jacob

____________________________________________________________________
WiX Toolset Developer Mailing List provided by FireGiant http://www.firegiant.com/ ____________________________________________________________________
WiX Toolset Developer Mailing List provided by FireGiant http://www.firegiant.com/ ____________________________________________________________________
WiX Toolset Developer Mailing List provided by FireGiant http://www.firegiant.com/ ____________________________________________________________________
WiX Toolset Developer Mailing List provided by FireGiant http://www.firegiant.com/