[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 03/51] block: Unify the get_tmp_filename() implementation
From: |
Bin Meng |
Subject: |
Re: [PATCH 03/51] block: Unify the get_tmp_filename() implementation |
Date: |
Thu, 1 Sep 2022 14:41:28 +0800 |
Hi Marc-André,
On Wed, Aug 31, 2022 at 8:54 PM Marc-André Lureau
<marcandre.lureau@gmail.com> wrote:
>
> Hi Bin
>
> On Wed, Aug 24, 2022 at 1:42 PM Bin Meng <bmeng.cn@gmail.com> wrote:
>>
>> From: Bin Meng <bin.meng@windriver.com>
>>
>> At present get_tmp_filename() has platform specific implementations
>> to get the directory to use for temporary files. Switch over to use
>> g_get_tmp_dir() which works on all supported platforms.
>>
>
> It "works" quite differently though. Is this patch really necessary here?
Without this patch the qtest cases builds on Windows do not have any
problem. So it is optional. I put it in the same series as it has the
same context of using hardcoded /tmp directory name.
>
> If yes, please explain why.
>
> If not, I suggest you drop optional / rfc / "nice to have" patches from the
> series. It will help to get it merged faster.
I can drop this single patch and send another single patch if this is
the desired practice.
>
> thanks
Regards,
Bin
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH 03/51] block: Unify the get_tmp_filename() implementation,
Bin Meng <=