Re: [PATCH] Rearrange detection of temporary directory for NetBSD

2021-06-30 Thread Gerald Pfeifer
On Tue, 29 Jun 2021, Jeff Law wrote: >> 2021-06-29 Gerald Pfeifer >> >> * make-temp-file.c (usrtmp): Remove. >> (choose_tmpdir): Remove use of usrtmp. > Yup. This is fine. You might consider updating the comment which > references /usr/tmp in choose_tmpdir

Re: [PATCH] Rearrange detection of temporary directory for NetBSD

2021-06-29 Thread Jeff Law via Gcc-patches
On 6/28/2021 4:45 PM, Gerald Pfeifer wrote: On Thu, 26 Mar 2020, Kamil Rytarowski wrote: On 25.03.2020 23:36, Jeff Law wrote: I wouldn't mind dropping /usr/tmp. That so antiquated that it'd be non- controversial. Can you send that as a separate patch. Behavior for !__NetBSD__ is out of in

Re: [PATCH] Rearrange detection of temporary directory for NetBSD

2021-06-28 Thread Gerald Pfeifer
On Thu, 26 Mar 2020, Kamil Rytarowski wrote: > On 25.03.2020 23:36, Jeff Law wrote: >> I wouldn't mind dropping /usr/tmp. That so antiquated that it'd be >> non- controversial. Can you send that as a separate patch. > Behavior for !__NetBSD__ is out of interest. This is not a very useful approa

Re: [PATCH] Rearrange detection of temporary directory for NetBSD

2020-03-25 Thread Kamil Rytarowski
On 25.03.2020 23:36, Jeff Law wrote: > On Wed, 2020-03-18 at 20:29 +0100, Kamil Rytarowski wrote: >> Set /tmp first, then /var/tmp. /tmp is volatile on NetBSD and >> /var/tmp not. This improves performance in the common use. >> The downstream copy of GCC was patched for this preference >> since 201

Re: [PATCH] Rearrange detection of temporary directory for NetBSD

2020-03-25 Thread Jeff Law via Gcc-patches
On Wed, 2020-03-18 at 20:29 +0100, Kamil Rytarowski wrote: > Set /tmp first, then /var/tmp. /tmp is volatile on NetBSD and > /var/tmp not. This improves performance in the common use. > The downstream copy of GCC was patched for this preference > since 2015. > > Remove occurence of /usr/tmp as it

[PATCH] Rearrange detection of temporary directory for NetBSD

2020-03-18 Thread Kamil Rytarowski
Set /tmp first, then /var/tmp. /tmp is volatile on NetBSD and /var/tmp not. This improves performance in the common use. The downstream copy of GCC was patched for this preference since 2015. Remove occurence of /usr/tmp as it was never valid for NetBSD. It was already activey disabled in the GCC