CJTozer / SublimeDiffView

Side-by-Side Git, SVN and Bazaar Diff Viewer for Sublime Text 3
https://packagecontrol.io/packages/DiffView
MIT License
20 stars 7 forks source link

the left comparing window content is always empty #41

Closed bruceman closed 6 years ago

bruceman commented 8 years ago

Hi, I have a svn project. i made some changes and use DiffView to check the different. But the left comparing window content is empty. please see the attachment.

Another issue is : when i use diff options, for example "-r 123". it always show "No changes to report..."

Thanks for your hard work on this wonderful project.

1

bruceman commented 8 years ago

I use macOS serria 10.12 and sublime 3 latest beta version.

CJTozer commented 8 years ago

@bruceman thanks for the report.

A few questions to get me started:

bruceman commented 8 years ago

Thanks for your quick reply, here are the answers.

Q1: svn version: 1.9.4 (r1740329) svn status: M test.html svn diff:

bruce-workstation:dev brli$ svn diff test.html 
Index: test.html
===================================================================
--- test.html   (版本 97694)
+++ test.html   (工作副本)
@@ -6,5 +6,6 @@
 </head>
 <body>
 <h1>test</h1>
+<h2>this is new line</h2>
 </body>
 </html>
\ No newline at end of file

:-(, not any output after run DiffView

Q2: this is my first time to use this plugin.

Q3:

bruce-workstation:dev brli$ svn diff -r 97694 test.html 
Index: test.html
===================================================================
--- test.html   (版本 97694)
+++ test.html   (工作副本)
@@ -6,5 +6,6 @@
 </head>
 <body>
 <h1>test</h1>
+<h2>this is new line</h2>
 </body>
 </html>
\ No newline at end of file
CJTozer commented 8 years ago

Ah - well that makes me wonder if the Chinese characters are the problem - I'll try to add a test for this to find out.

bruceman commented 8 years ago

@CJTozer thanks, hope to fix this issue.

CJTozer commented 8 years ago

Ok so with a new test I don't think it's anything to do with the characters.

Since it's only the left (old) file that's broken, I suspect there may be a problem with the temporary file that DiffView uses to store the old version of the file. Can you check a couple of things:

import tempfile
t = tempfile.mkdtemp()
print t

Please let me know if you need any more explanation here!

bruceman commented 8 years ago

@CJTozer here are the test screenshots test-1 test-2

CJTozer commented 8 years ago

Sorry that's my bad - forgot we're in Python 3 world - please replace print t with print(t) in the console example above.

bruceman commented 8 years ago

it works, output is : /var/folders/kn/0skk6hbd00g7wj57dw39j7b00000gn/T/tmpofdg9w

bruceman commented 8 years ago

@CJTozer Here are the data in temp path, but the content of test.html is empty. test-3

bruceman commented 8 years ago

@CJTozer any update on this issue?

CJTozer commented 8 years ago

OK I've just released v1.1.1, which should enable us to get more useful info.

This might take a while to permeate through PackageControl, but you should be able to see when you've got the latest version using the Package Control: List Packages command - DiffView should be on v1.1.1 (may require Sublime restart to pick up).

Once you're on v1.1.1, add the setting debug: true to the DiffView user settings (Preferences -> Package Settings -> DiffView -> Settings - User).

Then re-run your failing diff, and let me have the output from ST console. (Note that you should be able to copy-paste from the ST console, rather than posting screenshots, which would make it easier for me to handle).

Thanks.

CJTozer commented 6 years ago

closing as it's very stale - please re-open if you see this and have the diags requested...