-
Notifications
You must be signed in to change notification settings - Fork 2
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
Doesn't find xdeb #18
Comments
Hmmm we use Go's Can you double check that |
It is executable.
…On Wed, 26 Jun 2024 at 12:43, Timo Reichl ***@***.***> wrote:
Hmmm we use Go's LookPath() which *should* return the same thing as which.
Can you double check that xdeb is executable?
—
Reply to this email directly, view it on GitHub
<#18 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACN37RYRJLJWSBAY5ORUVTLZJKLLLAVCNFSM6AAAAABJ5SWXL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJRGM3TQNZSHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
--
Lars Augensen
(+47)40224293
|
Thanks for checking. I'll see what I can do. |
It seems to work now. I had xdeb in my home folder after a manual install a while back, as well as in /usr/local/bin/xdeb. |
Glad you could fix it! Do you think this is more of an edge case or something I should take a closer look at and try to fix? |
Nah, I guess you'd be fine with writing something in the docs like "If
xdeb-install claims xdeb isn't installed, check if you have multiple copies
of xdeb in your path and delete them." Problem solved. But it could be
more robust to just check if xdeb is present in the path, rather than
_where_ it is. Thanks for great software, anyway 👍
Lars Augensen
40224293
…On Sat, Jun 29, 2024, 12:53 Timo Reichl ***@***.***> wrote:
Glad you could fix it! Do you think this is more of an edge case or
something I should take a closer look at and try to fix?
—
Reply to this email directly, view it on GitHub
<#18 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACN37R4W4W7E4LZLO2HQBM3ZJ2GYZAVCNFSM6AAAAABJ5SWXL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJYGEYDIMRZHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
…issue Signed-off-by: Timo Reichl <[email protected]>
Closing. Fixed with 0b41d9d |
I get the message
"[xdeb-install] xdeb is not installed, please install it via 'xdeb-install xdeb' or manually from 'https://github.com/xdeb-org/xdeb/releases'"
What gives? I've tried both xdeb-install xdeb and git.
The text was updated successfully, but these errors were encountered: