Hi Marc,
thanks for your comments.

> > Pardon me for jumping in. I am the adduser maintainer. slurm-wlm is 
> > actually one of a low one digit number of packages that have adduser in 
> > their pre inst. May I ask for the reason you have to create your users 
> > in pre-inst and therefore need a pre-depends on adduser?

The slurm user is not actually used in the package where it is created,
but only in three other packages that depend on it. It is included there
to simplify the process of removing one of them while keeping the other.
It was placed in the preinst script out of an abundance of caution.

> > Would it be possible that you don't need that and just create your users 
> > in postinst and just need a regular dependency on adduser?

I wasn't aware that having adduser as a Pre-Depends could be such a
significant issue. I created a new version of the packagte with adduser
in postinst that looks fine in my testing environment.

> > Using one method or the other seems to be unusual and sounds like asking 
> > for trouble intentionally. I would like to know what led you to this 
> > solution

Regarding the coexistence of the two methods for adding the user, I
accepted the merge request as submitted because I assumed there might be
systems not using systemd.

Best regards,
-- 
Gennaro Oliva

Reply via email to