All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Moore <paul@paul-moore.com>
To: Masahiro Yamada <yamada.m@jp.panasonic.com>
Cc: Michal Marek <mmarek@suse.cz>,
	selinux@tycho.nsa.gov, linux-kbuild@vger.kernel.org
Subject: Re: [PATCH] selinux, kbuild: remove unnecessary $(hostprogs-y) from clean-files
Date: Thu, 05 Jun 2014 16:30:20 -0400	[thread overview]
Message-ID: <1539785.Vu4DnvQCIK@sifl> (raw)
In-Reply-To: <20140605161823.9579.AA925319@jp.panasonic.com>

On Thursday, June 05, 2014 04:18:23 PM Masahiro Yamada wrote:
> Hi Paul, Michal,
> 
> On Tue, 03 Jun 2014 18:27:41 -0400
> 
> Paul Moore <paul@paul-moore.com> wrote:
> > On Tuesday, June 03, 2014 02:47:39 PM Masahiro Yamada wrote:
> > > Hi Paul Moore,
> > 
> > Hello.
> > 
> > > On Thu, 17 Apr 2014 21:41:53 -0400 Paul Moore wrote:
> > > > On Thursday, April 17, 2014 11:42:26 AM Masahiro Yamada wrote:
> > > > > Files added to hostprogs-y are cleaned. (See scripts/Makefile.clean)
> > > > > Adding them to clean-files is redundant.
> > > > > 
> > > > > Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
> > > > > ---
> > > > > 
> > > > >  scripts/selinux/genheaders/Makefile | 1 -
> > > > >  scripts/selinux/mdp/Makefile        | 2 +-
> > > > >  2 files changed, 1 insertion(+), 2 deletions(-)
> > > > 
> > > > Acked-by: Paul Moore <paul@paul-moore.com>
> > > 
> > > It looks like this patch was not included into
> > > GIT-PULL of "SELinux patches for 3.16".
> > > 
> > > Is it possible to apply this and send another pull request?
> > 
> > My apologies, I thought the kbuild people were going to merge this patch;
> > this is why I sent my Acked-by instead of a note that I've merged the
> > patch. Considering that this patch doesn't actually fix any runtime
> > problems and it hasn't been in any of the different "next" trees, I think
> > it is best to wait until 3.17 for this patch.
> > 
> > I've added this patch to my queue of patches to be added to the SELinux
> > #next branch after the 3.16 merge window closes.
> > 
> > Thanks again for the patch, I'm sorry that we didn't get it merged sooner.
> 
> No worries.
> 
> Michal, could you apply this patch to kbuild.git#kbuild, please?

As I said, it is not a problem for me to carry the patch in the SELinux tree.

-- 
paul moore
www.paul-moore.com


      reply	other threads:[~2014-06-05 20:30 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-17  2:42 [PATCH] selinux,kbuild: remove unnecessary $(hostprogs-y) from clean-files Masahiro Yamada
2014-04-17  2:42 ` [PATCH] selinux, kbuild: " Masahiro Yamada
2014-04-18  1:41 ` Paul Moore
2014-04-18  1:41   ` Paul Moore
2014-06-03  5:47   ` Masahiro Yamada
2014-06-03 22:27     ` Paul Moore
2014-06-05  7:18       ` Masahiro Yamada
2014-06-05 20:30         ` Paul Moore [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=1539785.Vu4DnvQCIK@sifl \
    --to=paul@paul-moore.com \
    --cc=linux-kbuild@vger.kernel.org \
    --cc=mmarek@suse.cz \
    --cc=selinux@tycho.nsa.gov \
    --cc=yamada.m@jp.panasonic.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.