[PATCH] parisc: remove CVS keywords

[PATCH] parisc: remove CVS keywords

[Alexander Beregalov]

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(-)

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.

Re: [PATCH] parisc: remove CVS keywords

[Kyle McMartin]

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

Re: [PATCH] parisc: remove CVS keywords

[Matthew Wilcox]

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 ;-)

-- 
Matthew Wilcox				Intel Open Source Technology Centre
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step."

Re: [PATCH] parisc: remove CVS keywords

[Grant Grundler]

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

Re: [PATCH] parisc: remove CVS keywords

[Thibaut VARENE]

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

-- 
Thibaut VARENE
http://www.parisc-linux.org/~varenet/

Re: [PATCH] parisc: remove CVS keywords

[Matthew Wilcox]

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.

-- 
Matthew Wilcox				Intel Open Source Technology Centre
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step."

Re: [PATCH] parisc: remove CVS keywords

[Kyle McMartin]

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