Skip to content

Declare a dependency on the "mount" package (DBTS: #926262)

deadjoe requested to merge deadjoe/adduser:DBTS-926262-mount into master

DBTS: #926262

Yet another PR that plucks low-hanging fruit!

I went back and forth here with regards to adding mount as a dependency.

On the one hand, the mount stuff is not really core to the functionality of the adduser package. You can still get most of the functionality out of the tools with only minor breakage when mount is not installed.

On the other hand, mount is a required package, so adding a dependency is not likely to break anything for anyone except those working in very strange environments. Also, if we don't depend on mount and it's not there, any maintainer script that uses deluser --remove-home will be broken.

In the end, I leaned toward adding the dependency. After all, the package does call the mount command.

Let me know if there are problems with any of this reasoning. I think it is mostly sound.

Thanks!

Edited by deadjoe

Merge request reports

Loading