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

SVN finds a lot of irrelevant repositories #828

Closed
Yanpas opened this issue Feb 19, 2020 · 5 comments
Closed

SVN finds a lot of irrelevant repositories #828

Yanpas opened this issue Feb 19, 2020 · 5 comments
Labels

Comments

@Yanpas
Copy link
Contributor

Yanpas commented Feb 19, 2020

  • VSCode Version: 1.42
  • OS Version: kubuntu linux 19.10
  • Extension Version: 2.7.2
  • System Language: ru_RU.UTF-8
  • SVN Version: 1.10.6

Issue

image

Steps to Reproduce

Don't know exact steps but seems this issue appeared recently, maybe after 2.7.1 or 2.7.2 , or maybe after vscode update.

  • open file history view
  • click on some older revisons, the diff is opened
  • vscode goes slow and searches for all folders with svn on my system

SVN Output

 svn info --xml
[cwd]$ svn info --xml //proc/4137/cwd
[cwd]$ svn stat --xml --no-ignore --ignore-externals
[cwd]$ svn info --xml
[cwd]$ svn info --xml //proc/4290/cwd
[cwd]$ svn info --xml
[cwd]$ svn info --xml //proc/4347/cwd
[cwd]$ svn info --xml
[cwd]$ svn: E125001: Couldn't determine absolute path of '.'
[cwd]$ svn: E000002: No such file or directory[cwd]$ svn info --xml
[cwd]$ svn: E125001: Couldn't determine absolute path of '.'
[cwd]$ svn: E000002: No such file or directory[cwd]$ svn info --xml
[cwd]$ svn info --xml //proc/6395/cwd
[cwd]$ svn info --xml
[cwd]$ svn info --xml //proc/6409/cwd
[co3]$ svn info --xml
[co3]$ svn info --xml //var/yan/co3
[searchtrunk]$ svn info --xml
[searchtrunk]$ svn info --xml //var/yan/searchtrunk
[trunk_co2]$ svn info --xml
[cwd]$ svn info --xml
[cwd]$ svn stat --xml --no-ignore --ignore-externals
[trunk_co2]$ svn info --xml //var/yan/trunk_co2
[cwd]$ svn info --xml
[cwd]$ svn stat --xml --no-ignore --ignore-externals
[cwd]$ svn info --xml
[cwd]$ svn stat --xml --no-ignore --ignore-externals
[cwd]$ svn info --xml
[cwd]$ svn stat --xml --no-ignore --ignore-externals
[cwd]$ svn info --xml
[co3]$ svn stat --xml --no-ignore --ignore-externals
[co3]$ svn info --xml
[searchtrunk]$ svn stat --xml --no-ignore --ignore-externals
[searchtrunk]$ svn info --xml
[trunk_co2]$ svn stat --xml --no-ignore --ignore-externals
[trunk_co2]$ svn info --xml

Screenshots

@JohnstonCode
Copy link
Owner

Can you have a look at 35ac8db to see if that is the cause

@Yanpas
Copy link
Contributor Author

Yanpas commented Feb 19, 2020

This commit looks ok (The bug reproduces after clicking on item log provider). I think it's tempfs, looks like it searches in /proc.

@JohnstonCode
Copy link
Owner

Yeah i can see the proc calls, strange.

@Yanpas
Copy link
Contributor Author

Yanpas commented Feb 19, 2020

image

@Yanpas
Copy link
Contributor Author

Yanpas commented Feb 19, 2020

A hotfix:

diff --git a/src/source_control_manager.ts b/src/source_control_manager.ts
index e16cefc..477973a 100644
--- a/src/source_control_manager.ts
+++ b/src/source_control_manager.ts
@@ -141,7 +141,7 @@ export class SourceControlManager implements IDisposable {
     );
     const onPossibleSvnRepositoryChange = filterEvent(
       onWorkspaceChange,
-      uri => !this.getRepository(uri)
+      uri => uri.scheme === "file" && !this.getRepository(uri)
     );
     onPossibleSvnRepositoryChange(
       this.onPossibleSvnRepositoryChange,

What was happening is that the whole root was scanned for the depth 4.

Yanpas added a commit to Yanpas/svn-scm that referenced this issue Feb 19, 2020
JohnstonCode pushed a commit that referenced this issue Feb 19, 2020
JohnstonCode added a commit that referenced this issue Feb 19, 2020
## [2.9.1](v2.9.0...v2.9.1) (2020-02-19)

### Bug Fixes

* scanning root ([#828](#828)) ([#829](#829)) ([5cf1387](5cf1387))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants