Re: [gentoo-portage-dev] [PATCH 2/2] sys-apps/portage: exclude /var/tmp/ccache from tmpfiles cleanup

[Zac Medico]

On 01/01/2018 05:00 PM, Mike Gilbert wrote:
> On Mon, Jan 1, 2018 at 6:23 PM, Zac Medico <zmedico@g.o> wrote:
>> On 12/31/2017 07:22 AM, Mike Gilbert wrote:
>>> By default, systemd-tmpfiles removes files older than 30 days from /var/tmp.
>>> The default portage config sets CCACHE_DIR=/var/tmp/ccache.
>>>
>>> Bug: https://bugs.gentoo.org/490676#c14
>>> Package-Manager: Portage-2.3.19_p3, Repoman-2.3.6_p37
>>> ---
>>>  sys-apps/portage/files/portage-ccache.conf | 2 ++
>>>  sys-apps/portage/portage-2.3.19.ebuild     | 4 +++-
>>>  sys-apps/portage/portage-9999.ebuild       | 4 +++-
>>>  3 files changed, 8 insertions(+), 2 deletions(-)
>>>  create mode 100644 sys-apps/portage/files/portage-ccache.conf
>>
>> Looks good. Maybe we should also call tmpfiles_process in pkg_postinst?
>
> Calling tmpfiles_process is only relevant for tmpfiles.d snippets that
> create files/directories, which normally only happens on system
> startup.
>
> This one prevents files/directories from being removed during the
> recurring "clean" job, which runs once every 24 hours on systemd.
> OpenRC systems have no such process.

Ok, thanks for the explanation. The patches look good. Please go ahead
and apply them.
-- 
Thanks,
Zac