[apparmor] 2.8 Nominations

[Steve Beattie]

On Wed, Jan 02, 2013 at 04:27:33PM -0800, Steve Beattie wrote:
> On Tue, Dec 18, 2012 at 11:21:20PM +0100, Christian Boltz wrote:
> > I propose to backport the following patches to the 2.8 tree:
> > 
> > - [3/9] nvidia abstractions cleanups
> > - [5/9] skype profile - even if it's a quite big patch. OTOH there are 
> >   only additions, so it should be safe ;-)
> > - [6/9] add XCompose to abstractions/X
> > - [9/9] dnsmasq
> > 
> > The other patches (/proc -> /@{PROC} and @{pid} are too noisy for a 
> > minor release IMHO and don't bring a real benefit, so I don't think we 
> > should backport them.
> 
> Yes, I agree (and thus Acked-by: Steve Beattie <sbeattie at ubuntu.com>).
> For the record these are the following trunk commits:
> 
>   2090 - [3/9] nvidia abstractions cleanups
>   2092 - [5/9] skype profile
>   2093 - [6/9] add XCompose to abstractions/X
>   2096 - [9/9] dnsmasq

> I'd also like to nominate the following trunk commits for 2.8.1:

Doh, I missed Christian's other email nominating some of these in

  Subject: [apparmor] [patch] backport profile updates to 2.8 branch,
  Message-ID: <3878392.li1OWtZfu9 at tux.boltz.de.vu>

Sorry about that, I was both on holiday and ill for most of that week of
December.

For clarity, the ones he nominated that I also nominated
are:

   2065.1.1 - update fonts abstraction for new fontconfig paths
   2065.1.3 - update gnome abstraction to allow reading
              /usr/share/poppler/cMap/**
   2057 - update ubuntu-browsers.d/java for IcedTea 7
   2058 - let sanitized-helper also allow access to /usr/local
   2059 - ubuntu-integration does not work properly with exo-open

as well as just the updated README portion of:
   2069 - profiles - install extras into /usr/share/apparmor/extra-profiles/

and one that I overlooked:
   2062 - support alternate ping install location in /usr

Given that, Acked-by: Steve Beattie <sbeattie at ubuntu.com> for
all the above.

That leaves the following that I nominated for 2.8.1:

>   2066 - parser - *just* the updated caching test message from this commit [1]
>   2074 - tests - fix clone test on arm
>   2076 - parser tests - fix test driver for exec() failure
>   2079 - libapparmor - add pkgconfig support
>   2083 - parser tests - fix fine grained timestamp detection in caching tests
>   2084 - tests - paper over possible race in onexec test
> 
> [1] It looks like this leaked in to the directory walking commit by
>     mistake. It's a fine change on it's own right.
> 
> I'm more ambivalent about the following trunk commits. I'm nominating
> them for 2.8, but could be convinced to that they should not be
> included:
> 
>   2050 - parser - network rules debugging statements
>   2064 - parser - update apparmor_parser man page
>   2065 - parser - correct apparmor_parser -N command privilege
>   2069 - profiles - install extras into /usr/share/apparmor/extra-profiles/
>   2075 - tests - support alternate parser locations
>   2078 - profiles - add winbindd profile
> 
> Christian, if you'd like to do the merge for the ones you nominated
> yourself, that's fine by me, but if you'd rather I do them while I'm
> merging my nominations that are acceptable to others, that's cool, too.

This offer still stands.

-- 
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/20130102/3a8cad37/attachment.pgp>