Bug#905500: sreview-common: unowned directory after purge: /var/lib/sreview/

2018-08-07 Thread Andreas Beckmann
Control: reassign -1 piuparts

On 2018-08-07 12:29, Wouter Verhelst wrote:
> On Sun, Aug 05, 2018 at 03:32:08PM +0200, Andreas Beckmann wrote:
>>   /var/lib/sreview/   not owned
> 
> That is the home directory of the "sreview" user, which is created
> dynamically by adduser with the correct UID and GID of the sreview user,
> a requirement for SReview.
> 
> Shipping it with the wrong UID/GID might work, but could create
> problems.
> 
> Removing the SReview user seems wrong.
> 
> What do you suggest I do?

In that case we will add it to a whitelist.


Andreas



Bug#905500: sreview-common: unowned directory after purge: /var/lib/sreview/

2018-08-07 Thread Wouter Verhelst
On Sun, Aug 05, 2018 at 03:32:08PM +0200, Andreas Beckmann wrote:
>   /var/lib/sreview/not owned

That is the home directory of the "sreview" user, which is created
dynamically by adduser with the correct UID and GID of the sreview user,
a requirement for SReview.

Shipping it with the wrong UID/GID might work, but could create
problems.

Removing the SReview user seems wrong.

What do you suggest I do?

-- 
Could you people please use IRC like normal people?!?

  -- Amaya Rodrigo Sastre, trying to quiet down the buzz in the DebConf 2008
 Hacklab



Bug#905500: sreview-common: unowned directory after purge: /var/lib/sreview/

2018-08-05 Thread Andreas Beckmann
Package: sreview-common
Version: 0.3.1-2
Severity: important
User: debian...@lists.debian.org
Usertags: piuparts

Hi,

during a test with piuparts I noticed your package left unowned
directories on the system after purge, which is a violation of
policy 6.8:

https://www.debian.org/doc/debian-policy/#details-of-removal-and-or-configuration-purging

Filing this as important as having a piuparts clean archive is a release
goal since lenny.

The maintainer scripts create (and later remove) a file in that
directory. Manual directory removal may be not appropriate as this
directory is shared between several packages.

If the package would ship this as an empty directory, dpkg would take
care of the creation and removal (if it's empty).

>From the attached log (scroll to the bottom...):

1m9.1s ERROR: FAIL: Package purging left files on system:
  /var/lib/sreview/  not owned


cheers,

Andreas


sreview-common_0.3.1-2.log.gz
Description: application/gzip