xdeb-org / xdeb-install

Simple tool to automatically download, convert, and install DEB packages on Void Linux via the awesome xdeb tool.
MIT License
6 stars 2 forks source link

Doesn't find xdeb #18

Closed subhappy closed 3 months ago

subhappy commented 4 months ago

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'"

❯ which xdeb
/usr/local/bin/xdeb

What gives? I've tried both xdeb-install xdeb and git.

thetredev commented 4 months ago

Hmmm we use Go's LookPath() which should return the same thing as which.

Can you double check that xdeb is executable?

subhappy commented 4 months ago

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 https://github.com/xdeb-org/xdeb-install/issues/18#issuecomment-2191378729, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACN37RYRJLJWSBAY5ORUVTLZJKLLLAVCNFSM6AAAAABJ5SWXL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJRGM3TQNZSHE . You are receiving this because you authored the thread.Message ID: @.***>

-- Lars Augensen (+47)40224293

thetredev commented 4 months ago

Thanks for checking. I'll see what I can do.

subhappy commented 4 months ago

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. After rm'ing the one in my home folder, xdeb-install works. I guess LookPath() returned more than one result or something.

thetredev commented 4 months ago

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?

subhappy commented 4 months ago

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 https://github.com/xdeb-org/xdeb-install/issues/18#issuecomment-2198104299, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACN37R4W4W7E4LZLO2HQBM3ZJ2GYZAVCNFSM6AAAAABJ5SWXL2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJYGEYDIMRZHE . You are receiving this because you authored the thread.Message ID: @.***>

thetredev commented 3 months ago

Closing. Fixed with 0b41d9d022f046a2b2c68dc363ea8f2af329dd79