re: CVS commit: src/external/gpl3

2020-03-27 Thread matthew green
> Date:Sat, 28 Mar 2020 11:46:29 +1100 > From:matthew green > Message-ID: <15233.1585356...@splode.eterna.com.au> > > | can we just leave this as-is and let netbsd GCC people care? > > Only if the GCC people do care, and understand the issue, and > implement what w

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Robert Elz
Date:Sat, 28 Mar 2020 11:46:29 +1100 From:matthew green Message-ID: <15233.1585356...@splode.eterna.com.au> | can we just leave this as-is and let netbsd GCC people care? Only if the GCC people do care, and understand the issue, and implement what we want Base

re: CVS commit: src/external/gpl3

2020-03-27 Thread matthew green
i don't like this "don't care about netbsd-8" feeling i'm hearing when netbsd-9 is so freshly released. we only *just* dropped caring about netbsd-7. please care about netbsd-8 for now. .mrg.

re: CVS commit: src/external/gpl3

2020-03-27 Thread matthew green
can we just leave this as-is and let netbsd GCC people care?

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Martin Husemann
On Fri, Mar 27, 2020 at 02:11:36PM -0400, Greg Troxel wrote: > I don't see why we don't care about 8. Given our release policy, that > is supported until 10 is released. It didn't have the local patch that Kamil wants to remove, so no change there - but if we can fix it there too that would be a

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Greg Troxel
Great to hear of progress. I don't see why we don't care about 8. Given our release policy, that is supported until 10 is released.

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Kamil Rytarowski
On 27.03.2020 18:42, Martin Husemann wrote: > On Fri, Mar 27, 2020 at 06:36:29PM +0100, Kamil Rytarowski wrote: >> I tested the code without our local patch as mentioned in my commit, >> even in the snapshot 1 commit before jak@'s change. >> >> In all the cases I get P_tmpdir defined and pointing t

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Martin Husemann
On Fri, Mar 27, 2020 at 06:36:29PM +0100, Kamil Rytarowski wrote: > I tested the code without our local patch as mentioned in my commit, > even in the snapshot 1 commit before jak@'s change. > > In all the cases I get P_tmpdir defined and pointing to "/tmp/". > > I have not tested it on NetBSD-8

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Kamil Rytarowski
On 27.03.2020 18:24, Martin Husemann wrote: > On Fri, Mar 27, 2020 at 06:14:24PM +0100, Kamil Rytarowski wrote: >> On 27.03.2020 16:01, Martin Husemann wrote: >>> Which compiler version did you test? >>> >> >> I tested on NetBSD HEAD GCC style distribution as host. > > Not sure I understand. You ne

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Martin Husemann
On Fri, Mar 27, 2020 at 06:14:24PM +0100, Kamil Rytarowski wrote: > On 27.03.2020 16:01, Martin Husemann wrote: > > Which compiler version did you test? > > > > I tested on NetBSD HEAD GCC style distribution as host. Not sure I understand. You need to test the in-tree build with the change backed

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Kamil Rytarowski
On 27.03.2020 16:01, Martin Husemann wrote: > Which compiler version did you test? > I tested on NetBSD HEAD GCC style distribution as host. $ uname -rms NetBSD 9.99.51 amd64 > I see this on netbsd-8: > >> echo $TMPDIR > TMPDIR: Undefined variable. >> ktrace -i $TOOLDIR/bin/x86_64--netbsd-gcc b

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Martin Husemann
Which compiler version did you test? I see this on netbsd-8: > echo $TMPDIR TMPDIR: Undefined variable. > ktrace -i $TOOLDIR/bin/x86_64--netbsd-gcc base64.c [..] > kdump | fgrep tmp [..] 19418 1 collect2 NAMI "/var/tmp//ccs9D5nv.le" 19418 1 collect2 NAMI "/var/tmp//ccs9D5nv.le" 89

Re: CVS commit: src/external/gpl3

2020-03-27 Thread Kamil Rytarowski
I've found out that this patch in not needed, at least on NetBSD and likely Linux. The algorithm to pick tmpdir is as follows: - check for memoized tmpdir - try env variables - try P_tmpdir - try /var/tmp, then /usr/tmp, then /tmp - fallback to current dir ('.') - save memoized tmpdir P_tmp