[wix-devs] Testing 4x

Wed Jul 15 10:29:04 PDT 2020

This sounds to me like an argument for a tool-enforced standard like Bob is
suggesting.

I would welcome a distinct fmt command separate from lint and convert. I
could see enterprises requiring a standard format but not wanting to
require lint's suggestions. Converting just to get a standard format seems
weird UX.

--
Edwin G. Castro

On Wed, Jul 15, 2020 at 7:42 AM Hoover, Jacob via wix-devs <
wix-devs at lists.wixtoolset.org> wrote:

> Until your favorite XML editor wants tabs and lint wants spaces, and your
> version control takes a hit every time you build a new install. Heaven
> forbid Bob likes notepad++ and Billy likes VSCode, and they happen to have
> different defaults. Just imagine 30 different repos with 30 different
> standards...
>
> From: wix-devs [mailto:wix-devs-bounces at lists.wixtoolset.org] On Behalf
> Of Bob Arnson via wix-devs
> Sent: Tuesday, July 14, 2020 4:57 PM
> To: Rob Mensching <rob at firegiant.com>; WiX Toolset Developer Mailing List
> <wix-devs at lists.wixtoolset.org>
> Cc: Bob Arnson <bob at firegiant.com>
> Subject: Re: [wix-devs] Testing 4x
>
> In my fevered dreams, "linting" is about making suggestions, not
> automating fixes. ("Do you really need File/@DiskId?") So "format" should
> be always safe to -f.
>
> I also have fevered dreams that -f is the default and --dry-run becomes an
> option, but that's another dream.
>
> -----Original Message-----
> From: Rob Mensching <rob at firegiant.com<mailto:rob at firegiant.com>>
> Sent: Tuesday, 14 July, 2020 17:26
> To: Bob Arnson <bob at firegiant.com<mailto:bob at firegiant.com>>; WiX Toolset
> Developer Mailing List <wix-devs at lists.wixtoolset.org<mailto:
> wix-devs at lists.wixtoolset.org>>
> Subject: RE: [wix-devs] Testing 4x
>
> So, can we combine "fmt" and "lint"? Or do we need:
>
> wix convert
> wix fmt
> wix lint
>
> I'd rather not have "wix fmt" and keep the "-f" (fix it) switch on "lint"
> and, of course, "convert".
>
> -----Original Message-----
> From: Bob Arnson <bob at firegiant.com<mailto:bob at firegiant.com>>
> Sent: Tuesday, July 14, 2020 2:05 PM
> To: Rob Mensching <rob at firegiant.com<mailto:rob at firegiant.com>>; WiX
> Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org<mailto:
> wix-devs at lists.wixtoolset.org>>
> Subject: RE: [wix-devs] Testing 4x
>
> I'm on record as saying that WixCop has three jobs: StyleCop, linter, and
> converter. Newer languages like Go and Rust solve the problem of style
> arguments by having one style that's enforced by a "fmt" tool and hey,
> look, we were doing that for ages before them. I'd be +1 splitting the
> rules (or enabling/disabling appropriately) to focus on each task.
>
> -----Original Message-----
> From: Rob Mensching <rob at firegiant.com<mailto:rob at firegiant.com>>
> Sent: Tuesday, 14 July, 2020 15:43
> To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org
> <mailto:wix-devs at lists.wixtoolset.org>>
> Cc: Bob Arnson <bob at firegiant.com<mailto:bob at firegiant.com>>
> Subject: RE: [wix-devs] Testing 4x
>
> I think the question is whether we should suppress "style-cop" while doing
> conversion? Enforcing whitespace while converting from v3 to v4 does add a
> lot noise.
>
> -----Original Message-----
> From: wix-devs <wix-devs-bounces at lists.wixtoolset.org<mailto:
> wix-devs-bounces at lists.wixtoolset.org>> On Behalf Of Bob Arnson via
> wix-devs
> Sent: Thursday, July 9, 2020 4:32 PM
> To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org
> <mailto:wix-devs at lists.wixtoolset.org>>
> Cc: Bob Arnson <bob at firegiant.com<mailto:bob at firegiant.com>>
> Subject: Re: [wix-devs] Testing 4x
>
> http://www.joyofsetup.com/2008/03/26/wix-goodies-wixcop/<
> http://www.joyofsetup.com/2008/03/26/wix-goodies-wixcop>
>
> -----Original Message-----
> From: wix-devs <wix-devs-bounces at lists.wixtoolset.org<mailto:
> wix-devs-bounces at lists.wixtoolset.org>> On Behalf Of Sean Hall via
> wix-devs
> Sent: Thursday, 9 July, 2020 18:18
> To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org
> <mailto:wix-devs at lists.wixtoolset.org>>
> Cc: Sean Hall <r.sean.hall at gmail.com<mailto:r.sean.hall at gmail.com>>
> Subject: Re: [wix-devs] Testing 4x
>
>
> https://github.com/wixtoolset/Core/blob/220bea1948c19132d0e5277021b967993293c5a3/src/WixToolset.Core/CommandLine/BuildCommand.cs#L548
> <
> https://github.com/wixtoolset/Core/blob/220bea1948c19132d0e5277021b967993293c5a3/src/WixToolset.Core/CommandLine/BuildCommand.cs#L548
> >
>
> On Fri, Jul 10, 2020 at 8:05 AM Hoover, Jacob via wix-devs <
> wix-devs at lists.wixtoolset.org<mailto:wix-devs at lists.wixtoolset.org>>
> wrote:
>
> > Convert throws a whole lot of these...
> >
> > error WIX0006: The whitespace preceding this node is incorrect.
> >
> > Why is white space in a XML document important?
> >
> > Also I assume this means that
> > VisualStudioExtension\src\Schemas\v4\schemas\wix4.xsd is out of
> > date/not done?
> >
> > What switch is used w/ build to pass in a wxl?
> >
> > From: Rob Mensching [mailto:rob at firegiant.com]
> > Sent: Thursday, July 9, 2020 4:48 PM
> > To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org
> <mailto:wix-devs at lists.wixtoolset.org>>
> > Cc: Hoover, Jacob <Jacob.Hoover at greenheck.com<mailto:
> Jacob.Hoover at greenheck.com>>
> > Subject: RE: Testing 4x
> >
> > 1. Always start with "wix convert" of your old code to new code. It
> > isn't just namespaces that change.
> >
> > 2. Updated documentation that shows the v4 shape of things will
> > obviously be important.
> >
> >
> > -----Original Message-----
> > From: wix-devs <wix-devs-bounces at lists.wixtoolset.org<mailto:
> <mailto:wix-devs-bounces at lists.wixtoolset.org%3cmailto:%0b>>
> wix-devs-bounces at lists.wixtoolset.org<mailto:
> wix-devs-bounces at lists.wixtoolset.org>>> On Behalf Of Hoover, Jacob via
> > wix-devs
> > Sent: Thursday, July 9, 2020 2:42 PM
> > To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org
> <mailto:wix-devs at lists.wixtoolset.org%20%0b>> <mailto:
> wix-devs at lists.wixtoolset.org>>
> > Cc: Hoover, Jacob <Jacob.Hoover at greenheck.com<mailto:
> <mailto:Jacob.Hoover at greenheck.com%3cmailto:%0b>>
> Jacob.Hoover at greenheck.com<mailto:Jacob.Hoover at greenheck.com>>>
> > Subject: Re: [wix-devs] Testing 4x
> >
> > If the CLI matters...
> >
> > Tools\build\Release\net461\win-x86>wix build Z:\Temp\Product.wxs
> > Z:\Temp\Components.wxs -o test
> >
> > From: wix-devs [mailto:wix-devs-bounces at lists.wixtoolset.org] On
> > Behalf Of Hoover, Jacob via wix-devs
> > Sent: Thursday, July 9, 2020 4:30 PM
> > To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org
> <mailto:wix-devs at lists.wixtoolset.org%20%0b>> <mailto:
> wix-devs at lists.wixtoolset.org>>
> > Cc: Hoover, Jacob <Jacob.Hoover at greenheck.com<mailto:
> <mailto:Jacob.Hoover at greenheck.com%3cmailto:%0b>>
> Jacob.Hoover at greenheck.com<mailto:Jacob.Hoover at greenheck.com>>>
> > Subject: Re: [wix-devs] Testing 4x
> >
> > And the authoring in question is:
> >
> > <DirectoryRef Id="Xxx">
> > <Component Id="Xxx.Directory" Guid="{...}"> <CreateFolder />
> > <RemoveFolder Id='Xxx.Directory.Remove' On='uninstall'/>
> > <Condition>NOT UPGRADINGPRODUCTCODE</Condition> </Component>
> > </DirectoryRef>
> >
> > From: wix-devs [mailto:wix-devs-bounces at lists.wixtoolset.org] On
> > Behalf Of Hoover, Jacob via wix-devs
> > Sent: Thursday, July 9, 2020 4:20 PM
> > To: WiX Toolset Developer Mailing List <wix-devs at lists.wixtoolset.org
> <mailto:wix-devs at lists.wixtoolset.org%20%0b>> <mailto:
> wix-devs at lists.wixtoolset.org<mailto:wix-devs at lists.wixtoolset
> <mailto:wix-devs at lists.wixtoolset.org%3cmailto:wix-devs at lists.wixtoolset%0b>>
> .org %3cmailto:wix-devs at lists.wixtoolset.org>>>
> > Cc: Hoover, Jacob <Jacob.Hoover at greenheck.com<mailto:
> <mailto:Jacob.Hoover at greenheck.com%3cmailto:%0b>>
> Jacob.Hoover at greenheck.com<mailto:Jacob.Hoover at greenheck.com%<mailto:
> Jacob.Hoover at greenheck.com%3cmailto:Jacob.Hoover at greenheck.com%25>
> > 3cmailto:Jacob.Hoover at greenheck.com>>>
> > Subject: [wix-devs] Testing 4x
> >
> > Trying to kick the tires on the now built tools, but some things lack
> > a lot of detail, or any detail, for that matter. From the meetings I
> > could remember wix build source files..., but what are the other
> > options? Tried running against an existing installer source, and after
> > doing namespace upgrades I am now running into a strange error:
> >
> > error WIX0005: The Component element contains an unexpected child
> > element 'Condition'
> >
> >
> >
> > Thanks,
> > Jacob
> >
> > ____________________________________________________________________
> > WiX Toolset Developer Mailing List provided by FireGiant
> > http://www.firegiant.com/<http://www.firegiant.com><
> http://www.firegiant.com<http://www.firegiant.com>><
> > http://www.firegiant.com<http://www.firegiant.com><
> http://www.firegiant.com<http://www.firegiant.com>>><
> > http://www.firegiant.com<http://www.firegiant.com><
> http://www.firegiant.com<http://www.firegiant.com>><
> > http://www.firegiant.com<http://www.firegiant.com><
> http://www.firegiant.com<http://www.firegiant.com>>>>
> > NOTE: This email was received from an external source. Please use
> > caution when opening links or attachments in the message.
> > ____________________________________________________________________
> > WiX Toolset Developer Mailing List provided by FireGiant
> > http://www.firegiant.com/<http://www.firegiant.com/><
> http://www.firegiant.com/<http://www.firegiant.com/>><
> > http://www.firegiant.com/<http://www.firegiant.com/><
> http://www.firegiant.com/<http://www.firegiant.com/>>>
> > NOTE: This email was received from an external source. Please use
> > caution when opening links or attachments in the message.
> > ____________________________________________________________________
> > WiX Toolset Developer Mailing List provided by FireGiant
> > http://www.firegiant.com/<http://www.firegiant.com/><
> http://www.firegiant.com/<http://www.firegiant.com/>>
> > NOTE: This email was received from an external source. Please use
> > caution when opening links or attachments in the message.
> > ____________________________________________________________________
> > WiX Toolset Developer Mailing List provided by FireGiant
> > http://www.firegiant.com/<http://www.firegiant.com/>
> >
> ____________________________________________________________________
> WiX Toolset Developer Mailing List provided by FireGiant
> http://www.firegiant.com/<http://www.firegiant.com/>
> ____________________________________________________________________
> WiX Toolset Developer Mailing List provided by FireGiant
> http://www.firegiant.com/<http://www.firegiant.com/>
> ____________________________________________________________________
> WiX Toolset Developer Mailing List provided by FireGiant
> http://www.firegiant.com/<http://www.firegiant.com/>
> NOTE: This email was received from an external source. Please use caution
> when opening links or attachments in the message.
> ____________________________________________________________________
> WiX Toolset Developer Mailing List provided by FireGiant
> http://www.firegiant.com/
>