bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] warn on missing selinux files


From: Paolo Bonzini
Subject: Re: [PATCH] warn on missing selinux files
Date: Sat, 27 Jun 2009 14:37:07 +0200
User-agent: Thunderbird 2.0.0.17 (X11/20081009)


That all looks fine, except the comment for your new function,
gl_LIBSELINUX belongs in the .m4 file, and not just in the log.

Ok.

For the log, this would then be fine:

        (gl_LIBSELINUX): New function.  Extracted from...

Ok?  Should I test /selinux instead of /selinux/enforce?

That would be better, since a system for which $(getenforce) reports
"Permissive", that /selinux/enforce won't exist.
It might be better still simply to see if getenforce can be run.

getenforce is not installed on a Debian non-SELinux-enabled system, still such a system has /selinux and can use libselinux.

Paolo




reply via email to

[Prev in Thread] Current Thread [Next in Thread]