#2 Remove trailing whitespace in default /etc/selinux/config
Merged 6 years ago by lvrabec. Opened 6 years ago by walters.
rpms/ walters/selinux-policy config-trailing-whitespace  into  master

file modified
+1 -1
@@ -465,7 +465,7 @@ 

  #     targeted - Targeted processes are protected,

  #     minimum - Modification of targeted policy. Only selected processes are protected. 

  #     mls - Multi Level Security protection.

- SELINUXTYPE=targeted 

+ SELINUXTYPE=targeted

  

  " > /etc/selinux/config

  

See https://pagure.io/atomic-wg/issue/341 - basically for libostree
(and hence rpm-ostree, and Fedora Editions that use it like Fedora Atomic Host),
the Anaconda selinux --enforcing verb will end up rewriting
/etc/selinux/config to the same value it had before.

But because of the trailing space character, this generates
a difference, and means the config file appears locally modified,
and hence deployed systems won't receive updates.

I think Anaconda should also be fixed to avoid touching the file at all
if it wouldn't result in a change, but let's remove the trailing space
here too, as it's better to fix in two places.

Pull-Request has been merged by lvrabec

6 years ago
Metadata