[pkg-apparmor] Review doc : Contribute to upstream

[intrigeri]

Hi,

u wrote (16 Jan 2015 12:50:19 GMT) :
> intrigeri:
>> u wrote (10 Jan 2015 21:02:57 GMT) :
>>> > It can be found on the Debian Wiki:
>>> > https://wiki.debian.org/AppArmor/ContributeUpstream
>> I'd rather use subpages (AppArmor/Contribute/Upstream) for namespacing
>> purposes, instead of mixing "/" and CamelCase. But I don't know
>> MoinMoin well, so maybe there's a good reason to do it the way
>> you did.

> I had no particular reason to do so, except that I copied what we agreed
> on for the PackageMaintainers page.

> I have no strong feelings about this, but I don't think it's useful to
> add another level of pages here. Your suggestion would require creating
> another "Contribute" page, right?

Well, we already have a "Contribute" page, so all that's needed would
be to rename ContributeUpstream to Contribute/Upstream. Not that
I care that much anyway :)

>>> > contribute to existing/upstream AppArmor profiles,
>> "existing/upstream AppArmor profiles" links to
>> https://code.launchpad.net/~apparmor-dev/apparmor-profiles/,
>> so maybe make it clear that this is only about contributing to
>> profiles that live in the upstream apparmor-profiles repo, as opposed
>> to profiles that live somewhere else upstream?

> I think i have made that clearer now.

Indeed!

>>> > Once this page has been reviewed, I could point to it on the bugs tagged
>>> > "new-profile" [1] - after verification if these profiles already exist
>>> > upstream or in the apparmor-profiles-extra package. What do
>>> > you think?
>> Looks good. I've not the big picture of how the reader is supposed to
>> travel through our many pages in mind, though. For now, let's go on
>> adding content, and at some point we'll want to take a step back and
>> review this all in terms of navigation flow. TODO++ ?

> I added that to the todo list.
> I completely agree that navigation needs to be worked on. I will make a
> proposal on how I suggest to do it soonish.

Great :)

Cheers,
-- 
intrigeri