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

[Seyeong Kim]

Please take a look this if you have time.

Thanks

On Wed, 04 Apr 2018 17:47:00 +0000 Niels Thykier <niels at thykier.net> wrote:
> 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
> 
>