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

wslusc shortcut not working #129

Closed
alexynior28 opened this issue Jun 4, 2020 · 3 comments · Fixed by #128
Closed

wslusc shortcut not working #129

alexynior28 opened this issue Jun 4, 2020 · 3 comments · Fixed by #128
Labels
Bug This is either an issue that describes a bug or a PR that resolves a bug Help Wanted Needs help on addressing this bug/feature. Status: In Progress This is being worked on and not ready. Target: 3.x This is targeted to the 3.x version.

Comments

@alexynior28
Copy link

alexynior28 commented Jun 4, 2020

I am creating the following, however when I double-click, the screen opens and closes.

wslusc -n sc script.sh

The path of the direct link is as follows:

C:\Users\User\AppData\Local\Microsoft\WindowsApps\KaliLinux.54290C8133FEE_ey8k8hqnwqnmg\kali.exe run " bash -l -c script.sh"

@patrick330602 patrick330602 added Bug This is either an issue that describes a bug or a PR that resolves a bug Status: In Progress This is being worked on and not ready. Target: 3.x This is targeted to the 3.x version. labels Jun 5, 2020
@patrick330602
Copy link
Member

patrick330602 commented Jun 5, 2020

Thanks for the report. It seems that it is not properly reading the absolute path of your script. I will try to investigate this issue.

@alexynior28
Copy link
Author

Thank you

patrick330602 added a commit that referenced this issue Jun 8, 2020
- fixes #129
- fixes a possible white space path issue
- When TMP is empty, try TEMP
@patrick330602 patrick330602 reopened this Jun 9, 2020
@patrick330602 patrick330602 added Status: Pending Release This is fixed but not yet released. and removed Status: In Progress This is being worked on and not ready. labels Jun 9, 2020
@patrick330602 patrick330602 added Help Wanted Needs help on addressing this bug/feature. Status: In Progress This is being worked on and not ready. and removed Status: Pending Release This is fixed but not yet released. labels Jun 15, 2020
@patrick330602
Copy link
Member

[HW] Encountered some issues when handling the space in paths. Will insvestigate the issue and release the fix until this is getting fixed.

patrick330602 added a commit that referenced this issue Jun 24, 2020
* 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>
patrick330602 added a commit that referenced this issue Jun 28, 2020
* 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>
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 Help Wanted Needs help on addressing this bug/feature. Status: In Progress This is being worked on and not ready. 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