Re: [PATCH] PR libstdc++/78870 support std::filesystem on Windows

[Jonathan Wakely <jwakely dot gcc at gmail dot com>]

Oops, that message got bounced from the lists and was the wrong
version of the patch anyway - this is the one I meant to attach.

On Sun, 17 Jun 2018 at 00:00, Jonathan Wakely <jwakely.gcc@gmail.com> wrote:
>
> Here's what I wanted to test, which removes the new AC_CHECK_FUNCS (replacing them with GLIBCXX_COMPILE_OR_LINK tests that shouldn't even get run on bare metal ).
>
>
>
> On Sat, 16 Jun 2018 at 23:39, Jonathan Wakely <jwakely.gcc@gmail.com> wrote:
>>
>>
>>
>> On Sat, 16 Jun 2018 at 14:06, Jonathan Wakely <jwakely.gcc@gmail.com> wrote:
>>>
>>>
>>>
>>> On Sat, 16 Jun 2018 at 02:04, Sandra Loosemore <sandra@codesourcery.com> wrote:
>>>>
>>>> On 05/31/2018 01:19 PM, Jonathan Wakely wrote:
>>>> > This adds incomplete but functional support for std::filesystem and
>>>> > std::experimental::filesystem on MinGW. In theory there should be no
>>>> > changes to the existing behaviour for POSIX targets from this patch,
>>>> > as all the various bugs I found while working on this have already
>>>> > been fixed in separate patches.
>>>> >
>>>> > Tested powerpc64le-linux, and x86_64-w64-mingw32 (with a few expected
>>>> > FAILures on mingw-w64). Committed to trunk.
>>>>
>>>> As noted in the issue (comment #16) 2 weeks ago already, this patch
>>>> breaks libstdc++ configure on bare-metal targets.
>>>>
>>>> configure:80055: checking for link
>>>> configure:80055: error: Link tests are not allowed after GCC_NO_EXECUTABLES.
>>>>
>>>> Please fix?  It's blocking other people from testing unrelated patches
>>>> on mainline.
>>>>
>>>
>>>
>>> I'm not the only person able to make changes to autoconf files (and already said I didn't know how to fix this).
>>>
>>> I have a patch to replace the AC_CHECK_FUNCS line with GLIBCXX_COMPILE_OR_LINK tests inside GLIBCXX_CHECK_FILESYSTEM_DEPS which should work, I'll finish testing it when I get home this evening.
>>>
>>
>> I can't build aarch64-none-elf even before my commit, so I also can't test a fix.
>>
>> checking for shl_load... configure: error: Link tests are not allowed after GCC_NO_EXECUTABLES.
>>
>>
>>

Attachment: patch.txt
Description: Text document