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

wslview open file by absolute path failed #118

Closed
panjie opened this issue Apr 30, 2020 · 3 comments · Fixed by #115 or #128
Closed

wslview open file by absolute path failed #118

panjie opened this issue Apr 30, 2020 · 3 comments · Fixed by #115 or #128
Labels
Bug This is either an issue that describes a bug or a PR that resolves a bug Status: Pending Release This is fixed but not yet released. Target: 3.x This is targeted to the 3.x version.

Comments

@panjie
Copy link

panjie commented Apr 30, 2020

I use wslview to open a file by absolute path failed with the following error message:

$ wslview /home/panjie/temp/aaa.docx

Start : 由于出现以下错误,无法运行此命令: 系统找不到指定的文件。。
所在位置 行:1 字符: 1
+ Start "/home/panjie/temp/aaa.docx"
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [Start-Process],InvalidOperationException
    + FullyQualifiedErrorId : InvalidOperationException,Microsoft.PowerShell.Commands.StartProcessCommand

The first two lines are in Chinese which means Start is error becuase system could not found specified file.

While using relative path is OK, such as wslview ./temp/aaa.docx.

The environment I am using:

  • Windows 10 2004 19041.208
  • WSL2 Ubuntu 18.04
@patrick330602
Copy link
Member

Thank you for the feedback. This is already fixed in wslu 3.1 that is going to release really soon. The update for Ubuntu will take a longer time, say longer than a month FYI.

@patrick330602 patrick330602 added Bug This is either an issue that describes a bug or a PR that resolves a bug Status: Pending Release This is fixed but not yet released. Target: 3.x This is targeted to the 3.x version. labels Apr 30, 2020
@panjie
Copy link
Author

panjie commented May 30, 2020

I noticed that the the corresponding code in PR #115 is still buggy because the following line should be added to between line 55 and 56.

             properfile_full_path="$(readlink -f "${lname//file:\/\//}")"   

@patrick330602
Copy link
Member

Hi @panjie,

Thank you for the heads up. I will try to fix the issue in the coming update wslu 3.1.1.

patrick330602 added a commit that referenced this issue Jul 1, 2020
commit 248304b
Author: Jinming Wu, Patrick <me@patrickwu.space>
Date:   Tue Jun 30 22:55:18 2020 +0800

    REL: 3.1.1

commit 552db30
Merge: 3d268b1 795440d
Author: Jinming Wu, Patrick <me@patrickwu.space>
Date:   Sun Jun 28 16:58:08 2020 +0800

    Merge pull request #128 from wslutilities/3.x/master

    wslu 3.1.1

commit 795440d
Author: Jinming Wu, Patrick <me@patrickwu.space>
Date:   Sun Jun 28 16:50:44 2020 +0800

    wslusc: fix a problem in command generation

commit 20b9de8
Author: 谭九鼎 <109224573@qq.com>
Date:   Sun Jun 28 12:01:19 2020 +0800

    Readme: use https for alpine src (#131)

    * wslusc: multiple bug fixes

    - fixes #129
    - fixes a possible white space path issue
    - When TMP is empty, try TEMP

    * Revert "wslusc: multiple bug fixes"

    This reverts commit d5fcd9e.

    * Readme: use https for alpine src

    * Update README.hans.md

    * Update README.hant.md

    * Update README.eo.md

    Co-authored-by: Jinming Wu, Patrick <me@patrickwu.space>

commit aa96398
Author: 谭九鼎 <109224573@qq.com>
Date:   Wed Jun 24 20:12:17 2020 +0800

    Readme: fix semicolon (#130)

    * wslusc: multiple bug fixes

    - fixes #129
    - fixes a possible white space path issue
    - When TMP is empty, try TEMP

    * Revert "wslusc: multiple bug fixes"

    This reverts commit d5fcd9e.

    * Readme: fix semicolon

    * Update README.hant.md

    Co-authored-by: Jinming Wu, Patrick <me@patrickwu.space>

commit eaf7057
Author: Jinming Wu, Patrick <me@patrickwu.space>
Date:   Tue Jun 9 00:19:52 2020 +0800

    wslusc: multiple bug fixes

    - fixes #129
    - fixes a possible white space path issue
    - When TMP is empty, try TEMP

commit 3d268b1
Author: Jinming Wu, Patrick <me@patrickwu.space>
Date:   Tue Jun 9 00:14:48 2020 +0800

    Revert "wslusc: multiple bug fixes"

    This reverts commit d5fcd9e.

commit d5fcd9e
Author: Jinming Wu, Patrick <me@patrickwu.space>
Date:   Tue Jun 9 00:02:10 2020 +0800

    wslusc: multiple bug fixes

    - fixes #129
    - fixes a possible white space path issue
    - When TMP is empty, try TEMP

commit c30b3a9
Merge: 8808125 b753788
Author: Jinming Wu, Patrick <me@patrickwu.space>
Date:   Wed Jun 3 07:53:57 2020 +0800

    Merge pull request #123 from WhitewaterFoundry/3.x/master

    Update Fedora Remix Logo

commit b753788
Author: Carlos Rafael Ramirez <crramirez@users.noreply.github.com>
Date:   Tue Jun 2 23:31:35 2020 +0200

    Fix color for Fedora Remix description

commit a9ac6d6
Author: Carlos Rafael Ramirez <crramirez@users.noreply.github.com>
Date:   Tue Jun 2 23:29:53 2020 +0200

    Add the deep purple color

commit 5e4a2b8
Author: Carlos Rafael Ramirez <crramirez@users.noreply.github.com>
Date:   Tue Jun 2 23:28:47 2020 +0200

    Change to color in Fedora Remix to deep purple

commit b40b77d
Merge: b203791 a814be7
Author: Carlos Rafael Ramirez <crramirez@users.noreply.github.com>
Date:   Tue Jun 2 23:22:01 2020 +0200

    Merge pull request #1 from crramirez/patch-2

    Patch 2

commit a814be7
Author: Carlos Rafael Ramirez <crramirez@users.noreply.github.com>
Date:   Tue Jun 2 23:20:05 2020 +0200

    Add detection specifically to Fedora Remix for WSL

commit 8808125
Author: Jinming Wu, Patrick <me@patrickwu.space>
Date:   Tue Jun 2 00:07:21 2020 +0800

    wslu-header: fix ubuntu.exe detection issue

commit cf10f0a
Author: Jinming Wu, Patrick <me@patrickwu.space>
Date:   Tue Jun 2 00:03:09 2020 +0800

    wslview: fixes #118 completely due to a mistake

commit b203791
Author: Carlos Rafael Ramirez <crramirez@users.noreply.github.com>
Date:   Wed May 27 16:47:45 2020 +0200

    Update Fedora Remix Logo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is either an issue that describes a bug or a PR that resolves a bug Status: Pending Release This is fixed but not yet released. Target: 3.x This is targeted to the 3.x version.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants