Re: [Prism54-devel] Re: [PATCH 4/14 linux-2.6.7-rc1] prism54: addsupport for avs header in
From: Jeff Garzik
Date: Thu May 27 2004 - 14:36:58 EST
Luis R. Rodriguez wrote:
On Thu, May 27, 2004 at 02:21:39PM -0400, Jeff Garzik wrote:
Luis R. Rodriguez wrote:
diff -u -r1.31 -r1.33
--- linux-2.6.7-rc1/drivers/net/wireless/prism54/islpci_eth.c 18 Mar 2004
15:27:44 -0000 1.31
+++ linux-2.6.7-rc1/drivers/net/wireless/prism54/islpci_eth.c 19 Mar 2004
23:03:58 -0000 1.33
@@ -1,4 +1,4 @@
-/* $Header: /var/lib/cvs/prism54-ng/ksrc/islpci_eth.c,v 1.31 2004/03/18
15:27:44 ajfa Exp $
+/* $Header: /var/lib/cvs/prism54-ng/ksrc/islpci_eth.c,v 1.33 2004/03/19
23:03:58 ajfa Exp $
Please remove CVS substitions from your code, they cause endless patch
rejects if I choose to apply (for example) 10 out of 14 patches.
Will do. So if you get
--- ksrc/islpci_eth.c
+++ ksrc-new/islpci_eth.c
patches, that'll be OK? I substituted ksrc to
linux-2.6.7-rc1/drivers/net/wireless/prism54 thinking that'll ease your
job. Sorry for any inconvenience.
I think you misunderstand (and I apologize for causing the confusion).
It is _required_ that the patches include the full path in the header.
You did this correctly:
--- linux-2.6.7-rc1/drivers/net/wireless/prism54/islpci_eth.c 18 Mar
2004 15:27:44 -0000 1.31
+++ linux-2.6.7-rc1/drivers/net/wireless/prism54/islpci_eth.c 19 Mar 2004
I am referring to the CVS substitution variables embedded in your source
code. In this case $Header$.
However, consider what happens when I do:
1) apply patch #1
2) reject patch #2
3) attempt to apply patch #3
If each patch updates the $Header$, then patch #3 cannot be applied
because patch(1) will reject it due to the now-incorrect $Header$ line.
The $Header$ _forces_ me to apply your patches in order, all or none. I
don't think you want that ;-)
Jeff
-
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/