Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:120940
MIME-Version: 1.0
References: <ZOquxkEdpVa8GfcE@castor> <d2ca7e4a-99f9-4813-cbde-c4e92faa7b69@bastelstu.be>
In-Reply-To: <d2ca7e4a-99f9-4813-cbde-c4e92faa7b69@bastelstu.be>
Reply-To: Levi Morrison <levi.morrison@datadoghq.com>
Date: Mon, 28 Aug 2023 19:57:13 -0600
Message-ID: <CAAKU0ukfy5wDcDnkEgc2o3PhCSczdDxPWG+NcHAKiVHMSDVKZg@mail.gmail.com>
To: =?UTF-8?Q?Tim_D=C3=BCsterhus?= <tim@bastelstu.be>
Cc: Athos Ribeiro <athoscribeiro@gmail.com>, internals@lists.php.net
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] [VOTE] Support optional suffix parameter in tempnam
From: internals@lists.php.net ("Levi Morrison via internals")

On Sun, Aug 27, 2023 at 4:20=E2=80=AFAM Tim D=C3=BCsterhus <tim@bastelstu.b=
e> wrote:
>
> Hi Athos
>
> On 8/27/23 04:02, Athos Ribeiro wrote:
> > I am moving this RFC [1] to the voting phase.  Voting will be open for =
the
> > next 2 weeks, until September 10th, as per https://wiki.php.net/rfc.
> >
> > [1] https://wiki.php.net/rfc/tempnam-suffix-v2
> >
>
> I find this a useful feature in general, but I believe it not working on
> Windows completely nullifies the "could even provide more context for
> software processing such files" argument in favor of this feature. It
> will be unexpected for users if their code completely fails to work on
> Windows, because the suffix is ignored.
>
> For that reason I voted "no".
>
> Best regards
> Tim D=C3=BCsterhus
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: https://www.php.net/unsub.php


I have voted no for a similar reason. It would be nice if there were
os-specific packages in core that handled these kinds of things,
because the functionality is definitely useful. But if you call an API
like `FileSystem\Os\Unix\tempnam` then at least the platform specific
behavior is obvious and understood.