On Thu, 22 Sept 2022 at 09:19, Jonathan Wakely wrote:
>
> On Thu, 22 Sept 2022 at 07:45, LIU Hao via Gcc-help
> <gcc-help@xxxxxxxxxxx> wrote:
> >
> > 在 2022/9/21 00:02, mizo 91 via Gcc-help 写道:
> > > Hello,
> > >
> > > I'm having trouble compiling simple test program on windows 10 with long
> > > list of includes provided via '@response_file' argument
> > >
> > >
> >
> > Greetings. mingw-w64 developer speaking.
> >
> > As far as I can see, there are at least two issues about your report:
> >
> >
> > The first, obvious issue is that the error message is incorrect. The reason for that is, if we take
> > a look at 'libiberty/pex-win32.c' we see the following:
> >
> > 853 /* Create the child process. */
> > 854 pid = win32_spawn (executable, (flags & PEX_SEARCH) != 0,
> > 855 argv, env, dwCreationFlags, &si, &pi);
> > 856 if (pid == (pid_t) -1)
> > 857 pid = spawn_script (executable, argv, env, dwCreationFlags,
> > 858 &si, &pi);
> > 859 if (pid == (pid_t) -1)
> > 860 {
> > 861 *err = ENOENT;
> > 862 *errmsg = "CreateProcess";
> > 863 }
> >
> > We also notice this is the only place where `"CreateProcess"` appears as a sole part of an error
> > message.
> >
> > The cause of this issue is apparent: libiberty tries `win32_spawn`, and if for whatever reason it
> > fails, it makes another attempt with `spawn_script`, and if it fails again, `*err` is always set to
> > `ENOENT` i.e. `No such file or directory`, no matter why.
>
> It seems to me that pex-win32.c should use GetLastError() (and maybe
> FormatMessage) to get a Windows error code, rather than assuming
> ENOENT.
>
> Using C++ that would look something like:
>
> std::string errmsg = "CreateProcess:" +
> std::system_category().message(GetLastError());
>
> Or maybe win32_spawn should call GetLastError() and convert that to an
> errno value and store it in errno:
>
> std::error_condition ec =
> std::system_category().default_error_condition(GetLastError());
> if (ec.category() == std::generic_category())
> errno = ec.value();
> else
> errno = ENOENT; // or EINVAL might be better?
I reported https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107007
