Glad you worked it out :) For future reference, when you get funky behavior, it's always good to check which exe's you're running using 'which':
>which patch c:/bin/gnuwin32/bin/patch.EXE On Thu, Jan 8, 2009 at 10:19 AM, SamClark <pimp.roc...@gmail.com> wrote: > > Scratch that, patch.exe that came with cygwin works. I think I've been > running PATCH.EXE under c:\WINDOWS. Hope this isn't netbus but thats > my problem and another discussion. > > So now that PATCH.EXE is gone I can post a review by hand and view the > diff properly. Thanks for the help. Somewhat indirect but I never > would have noticed this otherwise. > > I'll give an update on whether post-review.py works when I try it. > > Thanks again and sorry I'm such a noob. > > On Jan 8, 10:38 am, SamClark <pimp.roc...@gmail.com> wrote: >> Okay, I noticed something interesting. I think I've been running the >> patch.exe in cygwin\bin (GnuWin32\bin and cygwin\bin are both on my >> PATH but GnuWin32\bin doesn't contain patch). I think I'm using the >> same diffutils package as you but it doesn't come with the patch tool. >> Anyway that would explain the goofy behaviour and why diff works but >> I'm wondering where you got your patch tool since even the .zip >> folders on: >> >> http://gnuwin32.sourceforge.net/packages/diffutils.htm >> >> Don't contain a 'patch' exec. >> >> While we're at it what does review board use to perform diffs? Would >> it be trying to use my hooped cygwin patchutils tools? >> >> On Jan 8, 7:20 am, "Muhammad Haggag" <mhag...@gmail.com> wrote: >> >> >> >> > On Wed, Jan 7, 2009 at 12:43 PM, SamClark <pimp.roc...@gmail.com> wrote: >> >> > > Well if I hack the file to look exactly like the svn diff output I get >> > > the 'patch didn't apply cleanly' issue, which is expected i guess. >> >> > > The patch command doesn't output anything. It just fails silently and >> > > doesn't leave me with any additional file (I'm assuming the patched >> > > file would appear in the same directory the patch command was run in). >> > > As a side note I'm using diff and patch from diffutils 2.8.7.1, should >> > > I try patchutils under cygwin (Is that even possible? my cygwin >> > > experience is zilch) >> >> > Yes, the patched file should appear in the same directory. Can you try >> > running patch without the --output=patched option? It should output to >> > stdout. >> > I use diff/patch from gnuwin32 (gnuwin32.sourceforge.net), but it >> > shouldn't make a difference. >> >> > > Maybe off-topic but is it possible that tortoiseSVN touching my >> > > repository messed things up? >> >> > Doesn't sound like it. There's something fundamentally wrong here if >> > we can't get patch to use the patch generated by diff. If we work out >> > what's going on here, we'll be able to get this to work. >> >> > > Ok maybe I missed a step somewhere. I haven't touched Cygwin except to >> > > install patchutils? All I have done is (beyond installing many >> > > dependencies): >> >> > > [snip] >> >> > > Am I missing something? >> >> > Looks alright to me. >> >> > Regards >> > -- >> > Muhammad Haggag- Hide quoted text - >> >> - Show quoted text - > > > -- Muhammad Haggag --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "reviewboard" group. To post to this group, send email to reviewboard@googlegroups.com To unsubscribe from this group, send email to reviewboard+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/reviewboard?hl=en -~----------~----~----~----~------~----~------~--~---