Re: [PATCH v2] genmultilib: Add sanity check

2022-11-21 Thread Christophe Lyon via Gcc-patches
On 11/21/22 13:32, Mark Wielaard wrote: Hi Christophe, On Mon, 2022-11-21 at 13:12 +0100, Christophe Lyon wrote: On 11/21/22 11:16, Mark Wielaard wrote: On Fri, Nov 18, 2022 at 12:42:10PM -0700, Jeff Law wrote: * genmultilib: Add sanity check. OK. It should be interesting to see

Re: [PATCH v2] genmultilib: Add sanity check

2022-11-21 Thread Mark Wielaard
Hi Christophe, On Mon, 2022-11-21 at 13:12 +0100, Christophe Lyon wrote: > On 11/21/22 11:16, Mark Wielaard wrote: > > On Fri, Nov 18, 2022 at 12:42:10PM -0700, Jeff Law wrote: > > > > * genmultilib: Add sanity check. > > > > > > OK. It should be interesting to see if it trips. > > > > I

Re: [PATCH v2] genmultilib: Add sanity check

2022-11-21 Thread Christophe Lyon via Gcc-patches
On 11/21/22 11:16, Mark Wielaard wrote: Hi, On Fri, Nov 18, 2022 at 12:42:10PM -0700, Jeff Law wrote: * genmultilib: Add sanity check. OK.� It should be interesting to see if it trips. It trips up various buildbot setups: https://builder.sourceware.org/buildbot/#/changes/13720

Re: [PATCH v2] genmultilib: Add sanity check

2022-11-21 Thread Mark Wielaard
Hi, On Fri, Nov 18, 2022 at 12:42:10PM -0700, Jeff Law wrote: > > * genmultilib: Add sanity check. > > OK.  It should be interesting to see if it trips. It trips up various buildbot setups: https://builder.sourceware.org/buildbot/#/changes/13720 Error calling ../../gcc/gcc/genmultilib: Numb

Re: [PATCH v2] genmultilib: Add sanity check

2022-11-18 Thread Jeff Law via Gcc-patches
On 11/3/22 03:52, Christophe Lyon via Gcc-patches wrote: When a list of dirnames is provided to genmultilib, its length is expected to match the number of options. If this is not the case, the build fails later for reasons not obviously related to this mistake. This patch adds a sanity check t