Re: [PATCH] gitignore: add *.bz2 and *.cpio to top-level; clean up usr/

From: Sam Ravnborg
Date: Fri Jun 26 2009 - 18:44:30 EST


On Sat, Jun 27, 2009 at 04:01:28AM +0530, Jaswinder Singh Rajput wrote:
> On Sat, 2009-06-27 at 00:21 +0200, Sam Ravnborg wrote:
> > On Sun, Jun 21, 2009 at 01:02:27PM +0530, Jaswinder Singh Rajput wrote:
> > >
> > > On Sat, 2009-06-20 at 13:52 -0700, H. Peter Anvin wrote:
> > > > Add *.bz2 and *.cpio to the top-level gitignore file, for consistency
> > > > with *.gz and *.lzma which are already there. Remove specific rules
> > > > in usr/gitignore, since they are now covered by the global rules.
> > > >
> > > > Signed-off-by: H. Peter Anvin <hpa@xxxxxxxxx>
> > > > Cc: Sam Ravnborg <sam@xxxxxxxxxxxx>
> > >
> > > It seems this is based on my patch :
> > >
> > > [PATCH] gitignore usr/initramfs_data.cpio.bz2 and usr/initramfs_data.cpio.lzma
> > >
> > > And you forget to CC me ;-)
> > >
> > > Are you trying to do this, fill signed-off-by and enjoy :
> > >
> > > [PATCH] Add *.bz2 and *.cpio to the top-level gitignore file, for consistency
> > > with *.gz and *.lzma which are already there. Remove specific rules
> > > in other gitignores, since they are now covered by the global rules.
> >
> > I would prefer to have *.gz and *.lzma pushed out to the
> > directories where it matters.
> > There global ignore rules will bite us one day.
> >
> > So I did not apply neither this nor Peter's original patch.
> >
>
> Ok and what about :
>
> [PATCH] gitignore usr/initramfs_data.cpio.bz2 and usr/initramfs_data.cpio.lzma

You are far to trigger happy with your patches.
This is not a quest about getting maximum number of patches
applied.
This is about doing stuff in the right way and Peter had not even
replied when you start to push your patch.

We have been there before and I'm getting tired of your
attitude towards your patch handling.

Sam
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/