[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Fix some temp file issues
From: |
Eli Zaretskii |
Subject: |
Re: [PATCH] Fix some temp file issues |
Date: |
Sun, 09 Oct 2022 07:19:43 +0300 |
> Date: Sat, 8 Oct 2022 15:22:50 -0700
> Cc: bug-make@gnu.org
> From: Paul Eggert <eggert@cs.ucla.edu>
>
> On 2022-10-08 00:14, Eli Zaretskii wrote:
> >> tmpfile uses "wb+" (POSIX requires this) and we should be consistent in
> >> all the paths that create temporary FILE *. The attached patch adds a
> >> comment about this.
> > I don't remember: where is this temporary file read?
>
> It's read by a different process (fork+exec), so it doesn't matter for
> 'make' now whether it uses "w+" or "w". It matters only for possible
> future changes to 'make' later, assuming we ever change other parts of
> 'make' to sometimes need read access via the same FILE *.
I meant the "b" part, not the "+" part. On systems where that changes
the bytestream written to the file, the change might require a
suitable change where we read that stuff.