Patch formatting question: how picky should the robot be?
Dan Kegel
dank at kegel.com
Mon Aug 11 15:51:32 CDT 2008
So, git generates patches like this:
diff --git a/dlls/ddraw/tests/dsurface.c b/dlls/ddraw/tests/dsurface.c
index da66e2e..4569997 100644
--- a/dlls/ddraw/tests/dsurface.c
+++ b/dlls/ddraw/tests/dsurface.c
and that's the way we like it.
And sometimes people generate patches with cvs, like
http://winehq.org/pipermail/wine-patches/2008-August/059403.html
diff -u -3 -p -r1.57 vga.c
--- dlls/winedos/vga.c 14 Jul 2008 10:57:58 -0000 1.57
+++ dlls/winedos/vga.c 27 Jul 2008 23:13:08 -0000
(That even used to be the way we liked it.)
That's ok, too, isn't it?
If so, I'll make sure my patch robot can handle it.
But what about patches like
http://winehq.org/pipermail/wine-patches/2008-August/059404.html
--- dlls/appwiz.cpl/appwiz.rc 2008-07-26 09:24:56.000000000 +0900
+++ appwiz.rc 2008-07-28 09:48:00.000000000 +0900
...
--- /dev/null 2008-07-28 09:27:06.210506864 +0900
+++ Ko.rc 2008-07-28 09:48:00.000000000 +0900
The patch robot should probably complain about those, right?
- Dan
More information about the wine-devel
mailing list