Control: tag -1 + pending

Dear Laurent,

Laurent Bigonville wrote:
> Looking in the postinst script I realized it was created by the package
> iself.

Correct.

> If I'm not wrong, the package is not preserving the changes to that file
> on upgrade.

I think that's not true. From 4.6.2-3's screen.postinst (comments are just
for highlighting lines in this mail):

  override=/etc/tmpfiles.d/screen-cleanup.conf
  if [ $perms -eq 4755 ]; then
    chmod 0755 /run/screen
    if [ ! -f $override ]; then # <---- check!
       echo 'd /run/screen 0755 root utmp' > $override
    fi
  elif [ $perms -eq 755 ]; then
    chmod 1777 /run/screen
    if [ ! -f $override ]; then # <---- check!
       echo 'd /run/screen 1777 root utmp' > $override
    fi
  fi

So it only writes this file if it doesn't yet exist. And according to
"git blame" the highlighted lines are in there since at least
2015-06-16.

> Also, some comments on the top of the file should mention that the
> file has been created by the package

Good idea! Will be done with the next upload!

                Regards, Axel
-- 
 ,''`.  |  Axel Beckert <a...@debian.org>, https://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5
  `-    |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE

Reply via email to