Skip to content

Commit

Permalink
add and use a login.defs.test with CREATE_HOME set
Browse files Browse the repository at this point in the history
I suspect this is not a big deal, and most distributions just ship their own
version verbatim like debian/login.defs. But if there is a distro - or even a
person - using this as is from upstream, then we dont' want to break them. So
let's undo this and use an etc/login.defs.test for the testing if needed.

Changelog: 01/13: move etc/login.defs.test to tests/system/etc/login.defs per
                  suggestion.

Signed-off-by: Serge Hallyn <[email protected]>
  • Loading branch information
hallyn authored and alejandro-colomar committed Jan 14, 2025
1 parent c91e3f9 commit 65723c3
Show file tree
Hide file tree
Showing 4 changed files with 476 additions and 3 deletions.
2 changes: 1 addition & 1 deletion share/ansible/roles/ci_run/tasks/alpine.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,6 @@

- name: Copy configuration file for testing
ansible.builtin.copy:
src: /usr/local/src/shadow/etc/login.defs
src: /usr/local/src/shadow/tests/system/etc/login.defs
dest: /etc/login.defs
remote_src: yes
2 changes: 1 addition & 1 deletion share/ansible/roles/ci_run/tasks/debian.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,6 @@

- name: Copy configuration file for testing
ansible.builtin.copy:
src: /usr/local/src/shadow/etc/login.defs
src: /usr/local/src/shadow/tests/system/etc/login.defs
dest: /etc/login.defs
remote_src: yes
2 changes: 1 addition & 1 deletion share/ansible/roles/ci_run/tasks/fedora.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,6 @@

- name: Copy configuration file for testing
ansible.builtin.copy:
src: /usr/local/src/shadow/etc/login.defs
src: /usr/local/src/shadow/tests/system/etc/login.defs
dest: /etc/login.defs
remote_src: yes
Loading

0 comments on commit 65723c3

Please sign in to comment.