[apparmor] [PATCH] clarifications for mount rules

Steve Beattie steve at nxnw.org
Wed Apr 11 21:16:13 UTC 2012


On Wed, Apr 11, 2012 at 02:36:30PM -0500, Jamie Strandboge wrote:
> Attached is a patch to clarify the intended behavior of mount rules. I
> added text to KNOWN BUGS for areas where current behavior differs from
> documented behavior. This is based on IRC conversations. I recommend
> applying the patch and then reading it since reading the patch diff
> alone might be confusing.

Acked-By: Steve Beattie <sbeattie at ubuntu.com>

It all looks good to me. The only thing I'd point out is that

> +mount options=ro,
> +mount options=atime,

when translated through pod2man gets rendered on a single line
(though not by pod2html, curiously). But splitting them into separate
lines takes away from the fact that they're meant to be considered in
conjunction with each other. Using C<> to indicate code just ends up
surrounding the options in quotes, which also doesn't quite convey the
right thing. So I think leaving it as is is okay.

(Reading man perlpod, I couldn't see a way to introduce a linebreak.)

-- 
Steve Beattie
<sbeattie at ubuntu.com>
http://NxNW.org/~steve/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20120411/973fe32f/attachment.pgp>


More information about the AppArmor mailing list