Re: sudo: please include an explanation of #include syntax in /etc/sudoers
Jonathan Nieder wrote:
> Justin B Rye wrote:
>> I suspect a lot of users are going to try to activate the feature by
>> removing the leading hash (cf. #609741).
>
> Nice find. Ideas for wording?
If it hadn't already gone into Squeeze, I'd say "don't document it as
broken, just change it to use a sane syntax in the first place". But
if we're stuck with it, the least painful approach is something like:
--- sudoers 2011-02-09 18:47:23.000000000 +0000
+++ sudoers.new 2011-02-16 11:59:00.157254456 +0000
@@ -20,4 +20,6 @@
# Allow members of group sudo to execute any command
%sudo ALL=(ALL:ALL) ALL
+# See sudoers(5) on "#include" directives:
+
#includedir /etc/sudoers.d
The blank line is intentional. A manpage reference should be enough
as long as people don't try searching for "hash"...
--
JBR with qualifications in linguistics, experience as a Debian
sysadmin, and probably no clue about this particular package
Reply to: