Re: [PATCH] * config.sub: Deduplicate `basic_machine` by always allowing the vendor

[Ben Elliston]

On Mon, Aug 20, 2018 at 02:00:32AM -0400, John Ericson wrote:

> Yes the * is probably just a cut and paste artifact. I suppose it is
> harmless and could hint to the human what the expected behavior, but
> for simplicity I just removed it. The crx- * test I deleted from the
> first patch too. That leaves the 1st patch working properly.

Thanks. Applied.

> The 2nd patch is exactly as before. It still breaks things, but that
> is intentional, with the 3rd patch fixing the breakage and the crx
> test instead. I could have combined them, but I suspect they'll be
> easier to read separately so I kept them separate. If you want to
> apply them together that is fine with me.

I am still seeing Shellcheck problems and testsuite failures with the
2nd _and_ 3rd patches applied.  Please don't break the testsuite, even
temporarily.

Thanks,
Ben

signature.asc
Description: PGP signature