Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

0.4.2 new issue finding files for diff #297

Closed
RevFry opened this issue Jul 17, 2018 · 23 comments
Closed

0.4.2 new issue finding files for diff #297

RevFry opened this issue Jul 17, 2018 · 23 comments

Comments

@RevFry
Copy link

RevFry commented Jul 17, 2018

When I try to view and compare against file in my work space I get an error dialog that says, "Corresponding work space file does not exist".

I think the cause is related to the this other issue. When I try to compare against a previous version I see an error in the output that says "fatal: '../../C:/[pathtofile]/[file]' is outside repository"

Looks like it's prepending unnecessarily to an already fully qualified path.

@DonJayamanne
Copy link
Owner

DonJayamanne commented Jul 17, 2018

Please could you send a sample repo with details to replicate this issue. Thanks.

@RevFry
Copy link
Author

RevFry commented Jul 17, 2018

I'll try and locate one to use. Everything I usually work on comes from private repositories. I'll add a comment when I have an example repository.

@RevFry
Copy link
Author

RevFry commented Jul 17, 2018

I actually cloned your repo and tried it.

image

To make sure I uninstalled the extension, deleted the folder, installed it, reloaded and then attempted to compare git.ts to previous and work space.

@JWleo
Copy link

JWleo commented Jul 18, 2018

me too.
os: win10
vscode: 1.25.1
git history: 0.4.2
@DonJayamanne same as upstairs @RevFry
problem1
the ability of compare to previous or workspace has something wrong

@marchage
Copy link

marchage commented Jul 23, 2018

Oh but this means I have to use... 🥁 ...

⚡       (?! [A-Z]+) 💥 !!!       ⚡

your (this || that || ...) code editor and/or git log gui client

😱 nooohhhhwww...!

This extension shows timeline graphics among git chaos (👍 ), while using Visual Studio code --disable-gpu. For me at least.

@stamminator
Copy link

Same for me, can't do file diffs at all

git show 83e0d3a8852c41145d3a5a5200854dda808ffe48:../../../../../c:/Users/JStamm/Documents/GitHub/piranha.core.examples/AddBlogToMvcApplication/Startup.cs  (completed in 0.363s)

fatal: '../../../../../c:/Users/JStamm/Documents/GitHub/piranha.core.examples/AddBlogToMvcApplication/Startup.cs' is outside repository

@stuartZhang
Copy link

stuartZhang commented Jul 24, 2018

I have run across the same issue as well.

image

Moreover, I'd like to contribute more useful information for the issue:

  1. Duplicate the git command "git show daefed355709ab1abc070ab706400a50eceea3c0:../../../d:/workshop/github/mobile_attendance/src/modules/attendance/pages/Edit.vue" from the Output panel.
  2. Rewrite the git command to "git show daefed355709ab1abc070ab706400a50eceea3c0:src/modules/attendance/pages/Edit.vue"
  3. Execute the new command in the VSCode Internal Terminal. The git-diff becomes available.

As a result, I doubt whether the git command "git show ***" is incorrectly built.

@AdrianCuartas
Copy link

I'm having the same issue.

image

@RevFry
Copy link
Author

RevFry commented Aug 7, 2018

Oddly, this doesn't seem to be constant. Today I was frustrated and kept clicking compare to previous and at one point it just worked. I didn't make any changes or even leave that window. Just tried it multiple times.

@hussem1
Copy link

hussem1 commented Aug 8, 2018

+1 I am seeing the same issues

git show --pretty=oneline --decorate=full --numstat f4728459bdb0ff087e529b8accd0163f72aa726a  (completed in 0.151s)
git show --numstat --pretty=oneline -M f4728459bdb0ff087e529b8accd0163f72aa726a  (completed in 0.137s)
git show --name-status --pretty=oneline -M f4728459bdb0ff087e529b8accd0163f72aa726a  (completed in 0.145s)
git remote get-url origin  (completed in 0.153s)
git log --pretty=oneline f4728459bdb0ff087e529b8accd0163f72aa726a^1 -n 1 -- ..\..\..\..\..\c:\Users\hussem1\Documents\Projects\DemoProject\README.md  (completed in 0.142s)
fatal: ..\..\..\..\..\c:\Users\hussem1\Documents\Projects\DemoProject\README.md: '..\..\..\..\..\c:\Users\hussem1\Documents\Projects\DemoProject\README.md' is outside repository

@pacher
Copy link

pacher commented Sep 3, 2018

+1 having issue

@lavaner
Copy link

lavaner commented Sep 19, 2018

+1 same issue

@azhukov87
Copy link

+1 to issue having

@JWleo
Copy link

JWleo commented Sep 20, 2018

guys! if you have the same problem on WINDOWS, you could get your answer at #316 . Install that unofficial git-history-vscode, and it will work for this function

@ole1986
Copy link
Contributor

ole1986 commented Sep 20, 2018

@JWleo thank you for telling others about the PR

but, better you change "gays" to "guys" in your sentence ;-)

@hussem1
Copy link

hussem1 commented Sep 21, 2018

@JWleo vsix is no longer available for download.

@hussem1
Copy link

hussem1 commented Sep 21, 2018

It says

The page could not be found.
Nothing was found at this point. Try a search or try the links below

@JWleo
Copy link

JWleo commented Sep 22, 2018

@hussem1 May be something wrong about your internet.
default

@MJayyusi
Copy link

@JWleo Thank you, I downloaded and tested it. Works great. I hope this gets merged soon.

DonJayamanne pushed a commit that referenced this issue Sep 26, 2018
Alternate to PR #314  which should make the fixFileUrl unnecessary I guess
I assume some path modification take place before fsPath() got initially called.

So the call immediately after Uri.file([...]) is called fixes it for me.

More details on _makeFsPath here:
https://github.com/Microsoft/vscode/blob/master/src/vs/base/common/uri.ts#L547

May also fix #307, #296, #297 and others.
@ole1986
Copy link
Contributor

ole1986 commented Oct 1, 2018

This issue should be fixed with version 0.4.3, too

@ole1986 ole1986 closed this as completed Oct 1, 2018
@giammi56
Copy link

the issue is still there

@ole1986
Copy link
Contributor

ole1986 commented Oct 23, 2020

@giammi56 feel free to please raise a new issue with some log and environment details. Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests