[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: minidlna

Michael Tautschnig wrote:
> > > - You ship a defaults file, which allows to customize $USER and $GROUP, but your
> > >   scripts don't take this into account.
> > 
> > Are you sure about that one? In minidlna.init, I only assign default
> > values to $USER and $GROUP if they are empty (which won't be the case if
> > they where set in the default file).
> > 
> I'm sorry, I was too imprecise here. I was referring to the {pre,post}inst,
> {pre,post}rm scripts here. These should simply read the defaults file (if
> already available) as well.

Oh I see, thanks for clarifying.

It seems like a clever thing to do in {pre,post}inst: there's no need to
create a default minidlna user:group if something else is already set in
the default file; I'm just unsure if it should create the user:group
found in the default file if they do not exist already (if they do not
exist, they were most likely removed by the user manually, and
re-creating them silently as system users seems like a bad idea).

And in postrm, I would not remove any user:group except
minidlna:minidlna, which was normally created by preinst.

Is this what you had in mind? I'll implement those changes, and upload a
new version soon.

Thanks again for your precious help.


Benoît Knecht

Reply to: