Hi Rémi and Karl,

On Wed, Jan 11, 2017 at 09:23:14PM +0000, Karl Kornel wrote:
> If I would play devil’s advocate, then I would say this: “An action
> like this seems inappropriate to place into a package that does not
> clearly indicate why it is needed.  Instead, you should create a
> slurm-common package, and have the user creation happen there.”

This is the reason because we didn't include this change into
Debian: I understand that repeating code is not so smart, but decoupling
the action of creating user from the packages that actually need it, in
my opinion is worst since it hides the reasons behind the user creation.

> I would be OK with that: I am fine putting together a patch to create
> a “slurm-common” package that does the user creation.  Some common
> documentation could also be moved to that package.  But, I did not
> know if that was appropriate, so I submitted the simpler change first!

I think that a package for the user creation is a bit excessive.
Best regards,
-- 
Gennaro Oliva

Reply via email to