[GH-ISSUE #293] Protocol app can't be run from the PATH

kerem commented

2026-03-01 17:18:22 +03:00

Owner

Originally created by @majkinetor on GitHub (Sep 26, 2022).
Original GitHub issue: https://github.com/1Remote/1Remote/issues/293

Originally assigned to: @VShawn on GitHub.

Is your enhancement related to a problem? Please describe.
I have wfreerdp.exe on the PATH. I still have to specify full PATH to it in PRM options.

Describe the solution you'd like
If it is on the PATH it should run just by the name. This can be used to integrate it with less config with other tools.

Additional context

The chocolatey bin folder is on the PATH:

BTW, since latest release of PRM, wfreerdp is not embedded as connection tab, in previous it was (the same version of wfreerdp).

Originally created by @majkinetor on GitHub (Sep 26, 2022). Original GitHub issue: https://github.com/1Remote/1Remote/issues/293 Originally assigned to: @VShawn on GitHub. **Is your enhancement related to a problem? Please describe.** I have wfreerdp.exe on the PATH. I still have to specify full PATH to it in PRM options. **Describe the solution you'd like** If it is on the PATH it should run just by the name. This can be used to integrate it with less config with other tools. **Additional context** The chocolatey `bin` folder is on the PATH: ![image](https://user-images.githubusercontent.com/85767/192323079-d38d93db-fe24-46b1-b3dd-26104168fb0d.png) BTW, since latest release of PRM, wfreerdp is not embedded as connection tab, in previous it was (the same version of wfreerdp).

kerem

2026-03-01 17:18:22 +03:00

closed this issue
added the
meta-enhancement

bug
labels

kerem commented

2026-03-01 17:18:23 +03:00

Author

Owner

@majkinetor commented on GitHub (Sep 27, 2022):

Env vars could be supported to for portability reasons.

@majkinetor commented on GitHub (Sep 27, 2022): Env vars could be supported to for portability reasons.

kerem commented

2026-03-01 17:18:23 +03:00

Author

Owner

@VShawn commented on GitHub (Sep 29, 2022):

Reading paths from environment variables was never considered before.

I make this feature now:

PRemoteM.0.7.1.7.beta.2-x64.zip

@VShawn commented on GitHub (Sep 29, 2022): Reading paths from environment variables was never considered before. I make this feature now: [PRemoteM.0.7.1.7.beta.2-x64.zip](https://github.com/1Remote/PRemoteM/files/9671894/PRemoteM.0.7.1.7.beta.2-x64.zip)

kerem commented

2026-03-01 17:18:23 +03:00

Author

Owner

@majkinetor commented on GitHub (Sep 29, 2022):

That doesnt seem to work:

You implemented usage of env vars, such as: %ChocolateyInstall%\bin\wfreerdp.exe and %FOO%\bar.exe and in both cases it doesn't work, and doesn't even report exe missing - i just get blank black screen.

Also, since I have %ChocolateyInstall%\bin on the PATH env. var, it should work without any path.

So we have 2 features here:

Accept arbitrary env var in path
Check if exe is available on PATH env var

@majkinetor commented on GitHub (Sep 29, 2022): That doesnt seem to work: You implemented usage of env vars, such as: `%ChocolateyInstall%\bin\wfreerdp.exe` and `%FOO%\bar.exe` and in both cases it doesn't work, and doesn't even report exe missing - i just get blank black screen. Also, since I have `%ChocolateyInstall%\bin` on the `PATH` env. var, it should work without any path. So we have 2 features here: 1. Accept arbitrary env var in path 2. Check if exe is available on `PATH` env var

kerem commented

2026-03-01 17:18:23 +03:00

Author

Owner

@majkinetor commented on GitHub (Sep 29, 2022):

Second thing is way more important, as it allows one to install arbitrary protocols and PRM will just be able to work with them:

cinst premotem vnc winspc wfreerdp ....

Then Premotem can even come with predefined protocols that will just work if the app is somewhere on the system.

@majkinetor commented on GitHub (Sep 29, 2022): Second thing is way more important, as it allows one to install arbitrary protocols and PRM will just be able to work with them: `cinst premotem vnc winspc wfreerdp ....` Then Premotem can even come with predefined protocols that will just work if the app is somewhere on the system.

kerem commented

2026-03-01 17:18:23 +03:00

Author

Owner

@VShawn commented on GitHub (Sep 30, 2022):

IT WORKS ON ME.

How your PATH be like? Maybe the path format will affect PRM reading.

@VShawn commented on GitHub (Sep 30, 2022): ![image](https://user-images.githubusercontent.com/10143738/193165350-fe1d6676-fab5-497c-a388-e7959835beca.png) ![image](https://user-images.githubusercontent.com/10143738/193165390-9634543e-5bf5-41a6-8638-8b56e9995825.png) ![image](https://user-images.githubusercontent.com/10143738/193165514-8708c595-12fd-41eb-bcac-db3295d7341b.png) IT WORKS ON ME. How your PATH be like? Maybe the path format will affect PRM reading.

kerem commented

2026-03-01 17:18:23 +03:00

Author

Owner

@VShawn commented on GitHub (Sep 30, 2022):

test the path with %BlabLA%\bio

works too

@VShawn commented on GitHub (Sep 30, 2022): test the path with `%BlabLA%\bio` works too ![image](https://user-images.githubusercontent.com/10143738/193184612-632582c3-7da7-44c3-bf08-8e78f6e094a7.png)

kerem commented

2026-03-01 17:18:23 +03:00

Author

Owner

@majkinetor commented on GitHub (Oct 2, 2022):

Using path in `PATH` env var

PATH

wfreerdp

Protocol

Connection

Result: popup

:

Setting path using env var inside it

I also tried to use ChocolateyInstall env var inside wfreerdp path:

wfreerdp

Connection

Result: black screen

@majkinetor commented on GitHub (Oct 2, 2022): ### _Using_ path in `PATH` env var <details><summary>PATH</summary> ![image](https://user-images.githubusercontent.com/85767/193466623-aff8e196-ba87-42f3-a28b-646b646d292a.png)</details> <details><summary>wfreerdp</summary> ![image](https://user-images.githubusercontent.com/85767/193466647-03f618bf-6cf7-48e5-bbbe-b0062174dcc0.png) </details> <details><summary>Protocol</summary> ![image](https://user-images.githubusercontent.com/85767/193466690-b2d34036-0df8-4b19-ad9c-aae52657a65c.png) </details> <details><summary>Connection</summary> ![image](https://user-images.githubusercontent.com/85767/193466756-f78fb096-6457-4d38-9cad-a49a3c89dab3.png) </details> <details><summary>Result: popup</summary> : ![image](https://user-images.githubusercontent.com/85767/193466766-fe2db36d-b512-4260-94db-85fd1480f02e.png) </details> ### Setting path using env var inside it I also tried to use `ChocolateyInstall` env var inside wfreerdp path: <details><summary>wfreerdp</summary> ![image](https://user-images.githubusercontent.com/85767/193466802-3b15c6e2-b0d9-4f8f-9b13-a69ebe5b062e.png) </details> <details><summary>Connection</summary> ![image](https://user-images.githubusercontent.com/85767/193467061-fe9353f6-e1c1-4e5f-b965-ee1600e47fc4.png) </details> <details><summary>Result: black screen</summary> ![image](https://user-images.githubusercontent.com/85767/193466897-68a31ddc-5db5-43ff-9e0e-0db06261fbcc.png) </details> ![image](https://user-images.githubusercontent.com/85767/193466923-0b5144ca-1d03-40c3-b01e-fdbe7d364304.png)

kerem commented

2026-03-01 17:18:23 +03:00

Author

Owner

@VShawn commented on GitHub (Oct 5, 2022):

Ahh, I tested it on APP protocol, no test on the runner, let me check the runner path after I back home from vacation.

@VShawn commented on GitHub (Oct 5, 2022): Ahh, I tested it on `APP` protocol, no test on the runner, let me check the runner path after I back home from vacation.

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@VShawn commented on GitHub (Oct 13, 2022):

A bug was found in external runner path checker which make PATH not work.

PRemoteM.0.7.1.7.beta.3-x64.zip

@VShawn commented on GitHub (Oct 13, 2022): A bug was found in external runner path checker which make PATH not work. [PRemoteM.0.7.1.7.beta.3-x64.zip](https://github.com/1Remote/PRemoteM/files/9771889/PRemoteM.0.7.1.7.beta.3-x64.zip)

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@majkinetor commented on GitHub (Oct 13, 2022):

First case now works, second case still doesn't.

@majkinetor commented on GitHub (Oct 13, 2022): First case now works, second case still doesn't.

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@VShawn commented on GitHub (Oct 13, 2022):

second case is chocolatey case? I don't have a chocolatey installed, maybe test it later.

@VShawn commented on GitHub (Oct 13, 2022): second case is chocolatey case? I don't have a chocolatey installed, maybe test it later.

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@majkinetor commented on GitHub (Oct 13, 2022):

No, I just used choco as an example:

For example try %SystemRoot%\notepad.exe

@majkinetor commented on GitHub (Oct 13, 2022): No, I just used choco as an example: For example try `%SystemRoot%\notepad.exe`

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@VShawn commented on GitHub (Oct 13, 2022):

Ahhh I see, you means using the environment variable in path directly.

@VShawn commented on GitHub (Oct 13, 2022): Ahhh I see, you means using the environment variable in path directly.

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@majkinetor commented on GitHub (Oct 13, 2022):

Yeah, things like that are needed for portability. If you carry your PRM with you (or copy it to another system), and it relies on tools on the local system, for it to work you generaly need to use env vars to avoid possible path changes on each system.

@majkinetor commented on GitHub (Oct 13, 2022): Yeah, things like that are needed for portability. If you carry your PRM with you (or copy it to another system), and it relies on tools on the local system, for it to work you generaly need to use env vars to avoid possible path changes on each system.

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@majkinetor commented on GitHub (Oct 13, 2022):

This will especially be useful in multiuser scneario with relational db holding data.

@majkinetor commented on GitHub (Oct 13, 2022): This will especially be useful in multiuser scneario with relational db holding data.

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@VShawn commented on GitHub (Oct 13, 2022):

goal, tested with winscp :)

PRemoteM.0.7.1.7.beta.3-x64.zip

@VShawn commented on GitHub (Oct 13, 2022): goal, tested with winscp :) ![image](https://user-images.githubusercontent.com/10143738/195555832-e3333264-91e2-45f2-beda-c08967bd9098.png) [PRemoteM.0.7.1.7.beta.3-x64.zip](https://github.com/1Remote/PRemoteM/files/9774934/PRemoteM.0.7.1.7.beta.3-x64.zip)

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@majkinetor commented on GitHub (Oct 13, 2022):

Works now.

@majkinetor commented on GitHub (Oct 13, 2022): Works now.

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@VShawn commented on GitHub (Oct 13, 2022):

god damn GFW, I can not push to github now.

fatal: Could not read from remote repository.

@VShawn commented on GitHub (Oct 13, 2022): god damn GFW, I can not push to github now. ` fatal: Could not read from remote repository. `

kerem commented

2026-03-01 17:18:24 +03:00

Author

Owner

@majkinetor commented on GitHub (Oct 13, 2022):

Probably temporary

@majkinetor commented on GitHub (Oct 13, 2022): Probably temporary

Rows
Columns

[GH-ISSUE #293] Protocol app can't be run from the PATH #2174

Using path in PATH env var

Setting path using env var inside it

Using path in `PATH` env var