| Message ID | 20090403014922.GA28891@orion (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | kyle mcmartin |
| Headers | show |
On Fri, Apr 03, 2009 at 05:49:22AM +0400, Alexander Beregalov wrote: > > Signed-off-by: Alexander Beregalov <a.beregalov@gmail.com> I don't feel comfortable removing these for hysterical raisins, unless people on the list ack this. (Just like I wouldn't remove the "sorry about the wall" comment... :) regards, Kyle -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Apr 03, 2009 at 01:58:49PM -0400, Kyle McMartin wrote: > I don't feel comfortable removing these for hysterical raisins, unless > people on the list ack this. (Just like I wouldn't remove the "sorry > about the wall" comment... :) I appreciate that ;-)
On Fri, Apr 03, 2009 at 01:58:49PM -0400, Kyle McMartin wrote: > On Fri, Apr 03, 2009 at 05:49:22AM +0400, Alexander Beregalov wrote: > > > > Signed-off-by: Alexander Beregalov <a.beregalov@gmail.com> > > I don't feel comfortable removing these for hysterical raisins, unless > people on the list ack this. I don't mind them being removed. The history is preserved else where for whoever wants to look. Please add my Ack-by: Grant Grundler <grundler@parisc-linux.org> to whatever parts of this patch you care to accept. > (Just like I wouldn't remove the "sorry > about the wall" comment... :) Sorry, I've no clue about this one...what's the story? :) thanks, grant -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Apr 3, 2009 at 7:58 PM, Kyle McMartin <kyle@mcmartin.ca> wrote: > On Fri, Apr 03, 2009 at 05:49:22AM +0400, Alexander Beregalov wrote: >> >> Signed-off-by: Alexander Beregalov <a.beregalov@gmail.com> > > I don't feel comfortable removing these for hysterical raisins, unless > people on the list ack this. Well I clearly have no say regarding those specific files, but it seems to me like these CVS Id tags are completely meaningless, anyway you want to look at them. They barely refer to the last person having touched these files in our *CVS* tree, which hasn't been touched in ice ages... Copyright information is stored elsewhere, and historical trivia can still be found in said CVS tree. Plus, I may be wrong but ISTR back in the days when we were still using CVS that these very tags were frowned upon as they were the source of merge conflicts (I'm pretty sure there were some emails about that years ago, though I can't seem to find them online). HTH T-Bone
On Sat, Apr 04, 2009 at 12:09:58AM -0600, Grant Grundler wrote: > On Fri, Apr 03, 2009 at 01:58:49PM -0400, Kyle McMartin wrote: > > On Fri, Apr 03, 2009 at 05:49:22AM +0400, Alexander Beregalov wrote: > > > > > > Signed-off-by: Alexander Beregalov <a.beregalov@gmail.com> > > > > I don't feel comfortable removing these for hysterical raisins, unless > > people on the list ack this. > > I don't mind them being removed. The history is preserved else > where for whoever wants to look. Please add my > Ack-by: Grant Grundler <grundler@parisc-linux.org> > > to whatever parts of this patch you care to accept. I'm comfortable with removing the $Id$ tags too. Generally these have been in copy-and-pasted files; we weren't relying on them for anything. Acked-by: Matthew Wilcox <willy@linux.intel.com> > > (Just like I wouldn't remove the "sorry > > about the wall" comment... :) > > Sorry, I've no clue about this one...what's the story? :) Find the file it's in ... look at the history ... you'll get it eventually ;-) You'll probably have to go back to the 2.2 tree though.
On Sat, Apr 04, 2009 at 01:14:27PM -0600, Matthew Wilcox wrote: > I'm comfortable with removing the $Id$ tags too. Generally these have > been in copy-and-pasted files; we weren't relying on them for anything. > > Acked-by: Matthew Wilcox <willy@linux.intel.com> > Yeah, looks like cvs.parisc-linux.org had -ko set for *ages* anyway, so they're even stale wrt cvs. nuked. cheers, Kyle -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/parisc/include/asm/dma.h b/arch/parisc/include/asm/dma.h index 31ad0f0..f7a18f9 100644 --- a/arch/parisc/include/asm/dma.h +++ b/arch/parisc/include/asm/dma.h @@ -1,5 +1,4 @@ -/* $Id: dma.h,v 1.2 1999/04/27 00:46:18 deller Exp $ - * linux/include/asm/dma.h: Defines for using and allocating dma channels. +/* asm/dma.h: Defines for using and allocating dma channels. * Written by Hennus Bergman, 1992. * High DMA channel support & info by Hannu Savolainen * and John Boyd, Nov. 1992. diff --git a/arch/parisc/kernel/cache.c b/arch/parisc/kernel/cache.c index 837530e..f34082d 100644 --- a/arch/parisc/kernel/cache.c +++ b/arch/parisc/kernel/cache.c @@ -1,5 +1,4 @@ -/* $Id: cache.c,v 1.4 2000/01/25 00:11:38 prumpf Exp $ - * +/* * This file is subject to the terms and conditions of the GNU General Public * License. See the file "COPYING" in the main directory of this archive * for more details. diff --git a/arch/parisc/kernel/pci.c b/arch/parisc/kernel/pci.c index 6936386..f7064ab 100644 --- a/arch/parisc/kernel/pci.c +++ b/arch/parisc/kernel/pci.c @@ -1,5 +1,4 @@ -/* $Id: pci.c,v 1.6 2000/01/29 00:12:05 grundler Exp $ - * +/* * This file is subject to the terms and conditions of the GNU General Public * License. See the file "COPYING" in the main directory of this archive * for more details. diff --git a/arch/parisc/kernel/processor.c b/arch/parisc/kernel/processor.c index ecb6093..6ce3b52 100644 --- a/arch/parisc/kernel/processor.c +++ b/arch/parisc/kernel/processor.c @@ -1,5 +1,4 @@ -/* $Id: processor.c,v 1.1 2002/07/20 16:27:06 rhirst Exp $ - * +/* * Initial setup-routines for HP 9000 based hardware. * * Copyright (C) 1991, 1992, 1995 Linus Torvalds diff --git a/arch/parisc/kernel/setup.c b/arch/parisc/kernel/setup.c index 82131ca..cb71f3d 100644 --- a/arch/parisc/kernel/setup.c +++ b/arch/parisc/kernel/setup.c @@ -1,5 +1,4 @@ -/* $Id: setup.c,v 1.8 2000/02/02 04:42:38 prumpf Exp $ - * +/* * Initial setup-routines for HP 9000 based hardware. * * Copyright (C) 1991, 1992, 1995 Linus Torvalds diff --git a/arch/parisc/lib/checksum.c b/arch/parisc/lib/checksum.c index 462696d..ae66d31 100644 --- a/arch/parisc/lib/checksum.c +++ b/arch/parisc/lib/checksum.c @@ -13,8 +13,6 @@ * modify it under the terms of the GNU General Public License * as published by the Free Software Foundation; either version * 2 of the License, or (at your option) any later version. - * - * $Id: checksum.c,v 1.3 1997/12/01 17:57:34 ralf Exp $ */ #include <linux/module.h> #include <linux/types.h> diff --git a/arch/parisc/mm/fault.c b/arch/parisc/mm/fault.c index 92c7fa4..d617e79 100644 --- a/arch/parisc/mm/fault.c +++ b/arch/parisc/mm/fault.c @@ -1,5 +1,4 @@ -/* $Id: fault.c,v 1.5 2000/01/26 16:20:29 jsm Exp $ - * +/* * This file is subject to the terms and conditions of the GNU General Public * License. See the file "COPYING" in the main directory of this archive * for more details.
Signed-off-by: Alexander Beregalov <a.beregalov@gmail.com> --- arch/parisc/include/asm/dma.h | 3 +-- arch/parisc/kernel/cache.c | 3 +-- arch/parisc/kernel/pci.c | 3 +-- arch/parisc/kernel/processor.c | 3 +-- arch/parisc/kernel/setup.c | 3 +-- arch/parisc/lib/checksum.c | 2 -- arch/parisc/mm/fault.c | 3 +-- 7 files changed, 6 insertions(+), 14 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html