[wix-devs] 5911 - BA Configurable REINSTALLMODE

[Bob Arnson]

Right but I was thinking flag enum values for the bits we want to give control over. So, for example, I'd say:

1. Don't let the BA control "v": It should be omitted for first-time installs and is the correct choice for minor upgrades.
2. Don't let the BA control "c": Burn already includes it and I can't picture a reason to *not* include it.
3. Let the BA choose one of the file-versioning switches or whichever ones we want to support:

P = BOOTSTRAPPER_REINSTALLMODE_MISSING
O = BOOTSTRAPPER_REINSTALLMODE_MISSING_OR_OLDER (mend)
E = BOOTSTRAPPER_REINSTALLMODE_MISSING_OR_OLDER_OR_EQUAL (repair)
A = BOOTSTRAPPER_REINSTALLMODE_ALL_FILES (repair/reinstall)
D = BOOTSTRAPPER_REINSTALLMODE_MISSING_OR_DIFFERENT

You had O/E/A in the enum and I'd be fine with dropping P (O would be fine) and D (just do A at that point).

-----Original Message-----
From: wix-devs <wix-devs-bounces at lists.wixtoolset.org> On Behalf Of Sean Hall via wix-devs
Sent: Monday, 16 August, 2021 10:04
To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org>
Cc: Sean Hall <r.sean.hall at gmail.com>
Subject: Re: [wix-devs] 5911 - BA Configurable REINSTALLMODE

I thought we decided to restrict the set of possible values during the meeting discussion. Which is why the WIP has "Providing an enum instead of letting the BA pick an arbitrary string is a compromise between giving the BA full control over REINSTALLMODE and giving it no control."

On Sun, Aug 15, 2021 at 9:55 PM Bob Arnson <bob at firegiant.com> wrote:

> It should be a bit mask, no? Or are you avoiding that to restrict the 
> set of possible values?
>
> -----Original Message-----
> From: wix-devs <wix-devs-bounces at lists.wixtoolset.org> On Behalf Of 
> Sean Hall via wix-devs
> Sent: Wednesday, 4 August, 2021 17:51
> To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org>
> Cc: Sean Hall <r.sean.hall at gmail.com>
> Subject: [wix-devs] 5911 - BA Configurable REINSTALLMODE
>
> I created a WIP for 5911 at
> https://wixtoolset.org/development/wips/5911-ba-configurable-reinstall
> mode
> .
> Any feedback?
> ____________________________________________________________________
> WiX Toolset Developer Mailing List provided by FireGiant 
> http://www.firegiant.com/
>
____________________________________________________________________
WiX Toolset Developer Mailing List provided by FireGiant http://www.firegiant.com/