Hi Iain,
>> I'm omitting the changes (also mentioned in the PR) to
>> compilable/ddoc9676a.d and compilable/depsOutput9948.d which contain
>> absolute path names in EXTRA_SOURCES because those need to go upstream
>> first.
>>
>
> This has been committed upstream
> (https://github.com/dlang/dmd/pull/9051) - I just need to trickle it
> down, among other patches that are in my queue.
great, thanks.
> I have no problem however if you go ahead commit the changes with this patch.
Good: that's what I did.
>> Bootstrapped without regressions on i386-pc-solaris2.11 and
>> x86_64-pc-linux-gnu. Ok for mainline?
>
> OK on my side, just one mental note.
>
>> @@ -360,6 +361,9 @@ proc gdc-do-test { } {
>> # Initialize `dg'.
>> dg-init
>>
>> + # Create gdc.test link so test names include that subdir.
>> + catch { file link $subdir . }
>> +
>
> I guess the proper fix would be to split this into three files, right?
>
> - compilable/compilable.exp
> - fail_compilation/fail_compilation.exp
> - runnable/runnable.exp
>
> Then move the common bits into lib/gdc-dmd.exp
While that's possible, we usually only have per-subdir .exp files when
e.g. there are massive/systematic differences in the set of compilation
flags used. In the case of those three subdirs, you could indeed split
into per-subdir files and a common one, but it's also fine as is.
The $subdir link above (where subdir is gdc.test, the directory below
.../testsuite) only achieves that both (say)
gdc.test/compilable/99bottles.d as passed to dg-runtest and
imports.jsonimport1 (without the prefixe and referenced in the sources
and -I flags) work just the same. So this will remain necessary even if
you decide to split gdc-test.exp into three .exp files.
Rainer
--
-----------------------------------------------------------------------------
Rainer Orth, Center for Biotechnology, Bielefeld University
