All of lore.kernel.org
 help / color / mirror / Atom feed
From: guido@trentalancia.com (Guido Trentalancia)
To: refpolicy@oss.tresys.com
Subject: [refpolicy] [PATCH/RFC 0/19]: patch set to update the git reference policy
Date: Thu, 03 Feb 2011 01:04:30 +0100	[thread overview]
Message-ID: <1296691470.5782.8.camel@tesla.lan> (raw)
In-Reply-To: <20110202235258.96745g5m1222lvwo@webmail.tuffmail.net>

Hello Martin !

Thanks very much for your interest in this attempt to feed back some
possible improvements to the reference policy based upon testing on a
generic recent installation.

On Wed, 02/02/2011 at 23.52 +0000, Martin Orr wrote:
> On Mon 31 Jan 21:18:57 2011, Guido Trentalancia wrote:
> > On Mon, 31/01/2011 at 14.52 -0500, Christopher J. PeBenito wrote:
> >> One thing I want to clarify for each of the actual patches you need:
> >>
> >> * a better subject: "patch set to update the git reference policy" isn't
> >> very informative.
> >
> > Then, it would probably be impossible to submit a patch set at all. We
> > will just have many individual, separate patches. Because the whole
> > patch set aims to tackle very different issues in many different places
> > that it would probably be impossible to summarize everything in the
> > subject.
> 
> I think this is the point: because you deal with many different  
> issues, you do not really have a "set".  Chris can decide  
> independently for each of the patches whether to apply it or not, and  
> that will (usually) not break and will cause a measurable improvement  
> in refpolicy.

I will do my best. However, I am not entirely sure (and cannot
guarantee) that applying only a subset of the patches will lead to
desirable results. In general, they will be made as much independent
from each other as technically possible.

> The subject of each patch should be a short summary of what that  
> individual patch does, for example "dbus file labelling" for patch 1  
> and "Allow dbus messages" for patch 2.  If you can't give such a label  
> to a particular patch, that might mean that you have divided up your  
> patches badly.

Yes, I will re-submit individual, disjoint patches so that each message
has a different subject. A short textual description at the beginning of
each message will summarize the aims of the patch that follows.

> >> * a detailed description of what the patch does.
> >
> > Sure. It will be done.
> >
> >> This will help facilitate review of the patches, and will help us
> >> understand the details.
> >
> > In general, the set of patches is the result of testing refpolicy on a
> > very recent generic Linux installation. It aims to fix generic issues
> > with a few essential modules while trying to use the latest refpolicy on
> > a recent unbranded Linux installation.

In general, it's just several tiny adjustments to some permissions in a
bunch of modules that I have tested. Something else might come at a
later time if I manage to test other modules or write new modules.

In the meanwhile, should you have other comments or questions to raise,
please do not hesitate to contact me.

Kind regards,

Guido

      reply	other threads:[~2011-02-03  0:04 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-19  0:40 [refpolicy] RFC: patch to update git reference policy Guido Trentalancia
2011-01-20 13:18 ` Christopher J. PeBenito
2011-01-20 17:32   ` Guido Trentalancia
2011-01-21 12:37     ` Christopher J. PeBenito
2011-01-24  0:43       ` [refpolicy] [PATCH/RFC 0/19]: patch set to update the " Guido Trentalancia
2011-01-24 15:01         ` Dominick Grift
2011-01-24 15:56           ` Guido Trentalancia
2011-01-24 15:59             ` Dominick Grift
2011-01-24 21:01               ` Guido Trentalancia
2011-01-24 21:22                 ` Dominick Grift
     [not found]         ` <4D471319.2000907@tresys.com>
2011-01-31 21:18           ` Guido Trentalancia
2011-02-02 23:52             ` Martin Orr
2011-02-03  0:04               ` Guido Trentalancia [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1296691470.5782.8.camel@tesla.lan \
    --to=guido@trentalancia.com \
    --cc=refpolicy@oss.tresys.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.