projg2 / eclean-kernel

Installed kernel cleanup tool
GNU General Public License v2.0
31 stars 10 forks source link

Fix --no-mount logic inversion #17

Closed elprans closed 3 years ago

elprans commented 3 years ago

The --no-mount argument uses store_false which implies True as the default, making --no-mount the default behavior when the option isn't specified and when it is specified, it actually means the inverse. This doesn't seem to be the actual intent (otherwise the option should be renamed to --mount).

mgorny commented 3 years ago

Good catch, thanks!