On Tue, Aug 14, 2018 at 01:53:37PM -0400, John Ericson wrote:

> Message-Id: 
> <7aa195016503d03125cdcbaf0a53788322ea7dc6.1534269204.git.John.Ericson@Obsidian.Systems>
> From: John Ericson <John.Ericson@Obsidian.Systems>
> Date: Tue, 14 Aug 2018 13:25:58 -0400
> Subject: [PATCH 1/2]     * testsuite/config-sub.data: Add legacy test cases
> To: config-patches@gnu.org

This is a bit weird:

> @@ -140,8 +142,13 @@ dec3100                                          
> mips-dec-ultrix4.2
>  decstation                                   mips-dec-ultrix4.2
>  decstation-3100                                      mips-dec-ultrix4.2
>  decstatn                                     mips-dec-ultrix4.2
> +decsystem10*                                 pdp10-dec-tops10
> +dec10*                                               pdp10-dec-tops10
> +decsystem20*                                 pdp10-dec-tops20
> +dec20*                                               pdp10-dec-tops20
>  dicos                                                i686-pc-dicos

Why are you adding test cases with asterisks in them?  Looks like too
much copying and pasting?

Also, the revised testsuite does not pass:

Invalid configuration `crx-unknown-elf': machine `crx-unknown' not recognized
FAIL: crx-unknown-elf -> , but crx-unknown-elf should map to itself

Please remove the failing tests until you submit a patch that fixes
the problem (and the test case can accompany that patch).

Thanks,
Ben

Attachment: signature.asc
Description: PGP signature

_______________________________________________
config-patches mailing list
config-patches@gnu.org
https://lists.gnu.org/mailman/listinfo/config-patches

Reply via email to