Source diff not displaying in changesets

[genesisdsi@…]

When I click on a ChangeSet or download diff from ChangeSet page, no diff is displayed.

• Trac 0.7.1
• SVN 1.0.4

on Win2K Pro

The FAQ recommends using diffutils from GNU or installing the latest SVN (1.0.4). I have the latest SVN installed. I install diffutils with no result.

I know a similar ticket has been closed with a wontfix severity, but I still need help.

[psohm@…]

i've the same problem on windows 2003 with trac 0.7

[daniel]

I will look into this again, and see if there's problems with subversion 1.0.4. It works in Subversion 1.0.1 on windows. Very strange.

[swillison@…]

I've had the same problem with Trac 0.7.1 running on Red Hat ES with Suversion 1.0.6 - see bug #725

[anonymous]

I was having this problem, and based off of #330 I checked my path, and sure enough that was the problem.

After installing diffutils make sure you've added "C:\Program Files\GnuWin32\bin" (or wherever you installed it) to you System's Path environment variable (not your user's path).

It seems to be necessary to restart the computer for the path change to take effect. As suggested in #330, try using the printenv.pl script which should be included with Apache to make sure the path is set to what you think it is.

[Ken Barker <ken.barker@…>]

Followed the advise in the prior comment and discovered three things:

• I only downloaded the source for diffutils. (That will do it every time!)
• The path is not set by the diffutils installer on Win 2K.
• It works!!

I would close this ticket as invalid, but second item should be taken care of in someway during the installation of trac.

[daniel]

re-investigate for 0.9.

[daniel]

Possibly add some configuration option.

[daniel]

The python svn-bindings (svn.fs.FileDiff.get_pipe) to be exact, is the culprit.

It should be fixed.

[Christopher Lenz]

#725 has been marked as duplicate of this ticket. It's not about Windows, so removed the windows keyword from the ticket.

[Christopher Lenz]

I think we should migrate to using ​difflib for the diffs in 0.9, which would fix this problem, and make a replacement of svn.fs.FileDiff.get_pipe() obsolete.

In the meantime, this problem can apparently be worked around by making sure that the diff command is on the path.

[Christopher Lenz]

I think we should migrate to using ​difflib for the diffs in 0.9, which would fix this problem, and make a replacement of svn.fs.FileDiff.get_pipe() obsolete.

In the meantime, this problem can apparently be worked around by making sure that the diff command is on the path.

[Christopher Lenz]

/me is currently working on this

[Christopher Lenz]

[1314] completely removes the dependency on the diff utility.