[wix-devs] 5911 - BA Configurable REINSTALLMODE

[Sean Hall]

If you remove the enum values with "a" from the WIP, those are the values
currently used by Burn: omus (implicitly by not specifying REINSTALLMODE
during Install), vomus (minor upgrade), cmuse (Repair),
cmuso (Modify/Mend). Burn doesn't already include "c" every time. It's
these kinds of things where I don't think it's worth it to restrict
anything, just let the BA do what it thinks is best and if they shoot
themselves in the foot then so be it.

On Mon, Aug 16, 2021 at 3:50 PM Bob Arnson <bob at firegiant.com> wrote:

> 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/
>