That’s phase 2, mostly because I’m still thinking about whether we could have 
attributes on MsiProperty kinda like {Install|Uninstall|Repair}Argument. Adding 
just Condition is still a win, though, so I’ll start with that. That comes 
after ExePackage/CommandLine, however.

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

From: Sean Hall [mailto:r.sean.h...@gmail.com]
Sent: Monday, 30 March, 2015 23:27
To: WiX toolset developer mailing list
Subject: Re: [WiX-devs] Optional command line arguments WIP updated

I would like to see Condition added to MsiProperty as well.  When can we start 
using this? :)

On Fri, Mar 27, 2015 at 1:53 PM, Bob Arnson 
<b...@firegiant.com<mailto:b...@firegiant.com>> wrote:
You’re both wrong. ☺ I wanted to use WixBundleExecutePackageAction instead of 
On* attributes so each CommandLine instance was just argument and condition. 
That said, Rob’s proposed authoring looks awesome. And I think it’s not much 
more complex than what I was thinking of.

Unless someone objects, I’m going with that (and will update the WIP to match).

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

From: Hoover, Jacob 
[mailto:jacob.hoo...@greenheck.com<mailto:jacob.hoo...@greenheck.com>]
Sent: Friday, 27 March, 2015 14:42
To: WiX toolset developer mailing list
Subject: Re: [WiX-devs] Optional command line arguments WIP updated

The way I read it was

    <ExePackage RepairCommand=”” UninstallCommand=”/uninstall”>
       <CommandLine OnInstall=”yes” OnRepair=”yes” Conditon=”VarX=1”>/foo 
/bar</CommandLine>
       <CommandLine OnUninstall=”yes” Conditon=”VarY=9”>/notfoo</CommandLine>
    </ExePackage>


From: Rob Mensching [mailto:r...@firegiant.com]
Sent: Friday, March 27, 2015 1:28 PM
To: WiX toolset developer mailing list
Subject: Re: [WiX-devs] Optional command line arguments WIP updated

I like this. I could really use a couple examples (sorry, “visualizing” changes 
is easiest way for me to understand)

So you are suggesting the following:

    <ExePackage RepairCommand=”” UnisntallCommand=”/uninstall”>
       <CommandLine InstallArgument=”/foo /bar”  RepairArgument=”/foo /bar”  
Conditon=”VarX=1” />
       <CommandLine UninstallArgument=”/notfoo” Conditon=”VarY=9” />
    </ExePackage>

That would match our use of InstallCommand, RepairCommand, UninstallCommand and 
probably avoid the use of the WixBundleExecutePackageAction 90+% of the time.

If so, I *really* like this.

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

From: Bob Arnson [mailto:b...@firegiant.com]
Sent: Wednesday, March 25, 2015 9:46 AM
To: wix-devs@lists.sourceforge.net<mailto:wix-devs@lists.sourceforge.net>
Subject: [WiX-devs] Optional command line arguments WIP updated

You can see the diff via pull request at 
https://github.com/wixtoolset/site/pull/50.

I’m strongly leaning toward the “package action variable.” It’s simple to 
implement and basically solves the problem adding this functionality to 
MsiProperty.

Feedback sought and welcomed.

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

------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net<mailto:WiX-devs@lists.sourceforge.net>
https://lists.sourceforge.net/lists/listinfo/wix-devs

------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the 
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs

Reply via email to