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> wrote:
> You’re both wrong. J 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]
> *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 <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 <b...@firegiant.com>]
> *Sent:* Wednesday, March 25, 2015 9:46 AM
> *To:* 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
> 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
