Hi Dana,

Thought someone with more Win32 deployment experience would reply to this by
now.

See if the diff in this package works better:
http://gnuwin32.sourceforge.net/packages/diffutils.htm

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Fri, Feb 12, 2010 at 1:11 PM, Dana Lacoste <dlaco...@aperio.com> wrote:

>  OK, so I’m looking at rolling out ReviewBoard in my new job and
> everything’s fine, EXCEPT:
>
>
>
> What “diff” command to use?
>
>
>
> I know MKS Toolkit won’t work, I know that it pretty much needs to be the
> GNU diff, so I did what I normally do and installed Cygwin.
>
>
>
> The thing is, the cygwin latest (1.7) “diff” command won’t work with DOS
> paths.  Well, it can, but it complains, resulting in this:
>
>
>
> C:\TEMP>post-review -d 31214
>
> >>> p4 info
>
> >>> repository info: Path: server:1666, Base path: None, Supports
> changesets: True
>
> >>> p4 counters
>
> >>> Generating diff for changenum 31214
>
> >>> p4 describe -s 31214
>
> >>> Processing edit of //path/to/filename.txt
>
> >>> Writing "//path/to/filename.txt #3" to "c:\temp\tmpszo_ox"
>
> >>> p4 print -o c:\temp\tmpszo_ox -q //path/to/filename.txt#3
>
> >>> diff -urNp c:\temp\tmpszo_ox C:\Perforce\path\to\filename.txt
>
> Unable to parse diff header:   MS-DOS style path detected:
> c:\temp\tmpszo_ox
>
>
>
> HUH?????
>
>
>
> So, it turns out that you can do this:
>
>
>
> set CYGWIN=nodosfilewarning
>
>
>
> …which tells CYGWIN to not complain about using dos style pathnames, but I
> was wondering if anyone had a suggestion for an alternative solution.
>
>
>
> Thanks for any advice!
>
>
>
> Dana Lacoste
>
> --
> Want to help the Review Board project? Donate today at
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~----------~----~----~----~------~----~------~--~---
> To unsubscribe from this group, send email to
> reviewboard+unsubscr...@googlegroups.com<reviewboard%2bunsubscr...@googlegroups.com>
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
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

Reply via email to