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 <[email protected]>

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
<[email protected]>
http://NxNW.org/~steve/

Attachment: signature.asc
Description: Digital signature

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to