CFA: recognize more fields from login.defs

yast / yast-yast2

YaST module yast2

http://en.opensuse.org/Portal:YaST

GNU General Public License v2.0

54 stars 44 forks source link

CFA: recognize more fields from login.defs #1166

Closed ancorgs closed 3 years ago

ancorgs commented 3 years ago

This is a pre-requisite for https://github.com/yast/yast-users/pull/284

This repository contains a CFA class to deal with login.defs. But it only handles a limited set of configuration parameters. This adds umask to that set, since it's needed in order to configure useradd (at some point, the umask configuration was part of /etc/default/useradd but now it's only present in login.defs).

Taking the opportunity, this PR also adds another parameters that influence the behavior of useradd.

coveralls commented 3 years ago

Coverage remained the same at 40.923% when pulling a228d8b12ccf327eaaf699bf72c77059c066147f on ancorgs:umask into b9658bbf18781953476333a219b1540b976a20c2 on yast:master.

yast-bot commented 3 years ago

:heavy_check_mark: Public Jenkins job #307 successfully finished :heavy_check_mark: Created OBS submit request #896680