[debhelper-devel] Bug#894510: debhelper: Because it is listing tmpfiles in systemd's only, conf overriding is not working

[Niels Thykier]

Hi,

@Michael/@Felipe: Could you have a look at this bug ?

On Sat, 31 Mar 2018 08:58:21 -0700 Seyeong Kim
<seyeong.kim at canonical.com> wrote:
> Package: debhelper
> Version: 9.20160115ubuntu3
> Severity: normal
> Tags: d-i
> 
> In autoscripts/postinst-init-tmpfiles, There is TMPFILE containing conf in systemd pkg only.
> Then if there is 00rsyslog.conf from rsyslog pkg. and installing or upgrading systemd
> 
> /var/log's permission is 755(which is default) not 775(which is in 00rsyslog.conf)
> overriding doesn't work when upgrading.
> 
> Please refer to below LP
> ubuntu lp bug : https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1748147
> 
> removing TMPFILE from autoscripts/postinst-init-tmpfiles solves this issue.
> e.g. change like below
> systemd-tmpfiles --create #TMPFILES# >/dev/null || true
> to
> systemd-tmpfiles --create >/dev/null || true
> 
> and removing related code from dh_installinit maybe needed
> e.g. below kind of part
>         if (compat(10) && !$dh{NOSCRIPTS}) {
>                 # Include postinst-init-tmpfiles if the package ships any files
>                 # in /usr/lib/tmpfiles.d or /etc/tmpfiles.d
>                 my @tmpfiles;
>                 find({
>                         wanted => sub {
>                                 my $name = $File::Find::name;
>                                 return unless -f $name;
>                                 $name =~ s/^\Q$tmp\E//g;
>                                 if ($name =~ m,^/usr/lib/tmpfiles\.d/, ||
>                                         $name =~ m,^/etc/tmpfiles\.d/,) {
>                                         push @tmpfiles, $name;
>                                 }
>                         },
>                         no_chdir => 1,
>                 }, $tmp);
>                 if (@tmpfiles > 0) {
>                         autoscript($package,"postinst", "postinst-init-tmpfiles",
>                                 "s,#TMPFILES#," . join(" ", sort @tmpfiles).",g");
>                 }
>         }
> 
> Is there any reason that TMPFILE list is there? 
> 
> [...]

Thanks,
~Niels