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

Install failed test: Expected a combining operator like '-a' at index 2 #54

Closed
mrichman opened this issue Jun 24, 2024 · 9 comments
Closed

Comments

@mrichman
Copy link

fisher install reitzig/sdkman-for-fish@v2.1.0
fisher install version 4.4.4
Fetching https://api.github.com/repos/reitzig/sdkman-for-fish/tarball/v2.1.0
Installing reitzig/sdkman-for-fish@v2.1.0
           /Users/mark/.config/fish/functions/sdk.fish
           /Users/mark/.config/fish/conf.d/sdk.fish
           /Users/mark/.config/fish/completions/sdk.fish
test: Expected a combining operator like '-a' at index 2
                           != mark
 ^
~/.config/fish/conf.d/sdk.fish (line 99):
if not set -q SDKMAN_CANDIDATES_DIR; or test (ls -ld "$SDKMAN_CANDIDATES_DIR" | awk '{print $3}') != (whoami)
                                        ^
from sourcing file ~/.config/fish/conf.d/sdk.fish
	called on line 189 of file ~/.config/fish/functions/fisher.fish
in function 'fisher' with arguments 'install reitzig/sdkman-for-fish@v2.1.0'
Updated 1 plugin/s
@reitzig
Copy link
Owner

reitzig commented Jun 24, 2024

fish --version
cat /etc/os-release

?

@mrichman
Copy link
Author

4.4.4
macOS 14.5

uninstall and reinstall seems to have fixed it

@reitzig
Copy link
Owner

reitzig commented Jun 25, 2024

Huh, that's odd. 🤔

I may have to keep an eye out for changes in fish.

@mrichman
Copy link
Author

@reitzig Just got it again on a different machine:

fisher install reitzig/sdkman-for-fish@v2.1.0
fisher install version 4.4.4
Fetching https://api.github.com/repos/reitzig/sdkman-for-fish/tarball/HEAD
Installing reitzig/sdkman-for-fish
           /Users/mark/.config/fish/functions/sdk.fish
           /Users/mark/.config/fish/conf.d/sdk.fish
           /Users/mark/.config/fish/completions/sdk.fish
test: Expected a combining operator like '-a' at index 2
   != mark
 ^
~/.config/fish/conf.d/sdk.fish (line 99):
if not set -q SDKMAN_CANDIDATES_DIR; or test (ls -ld "$SDKMAN_CANDIDATES_DIR" | awk '{print $3}') != (whoami)
                                        ^
from sourcing file ~/.config/fish/conf.d/sdk.fish
	called on line 189 of file ~/.config/fish/functions/fisher.fish
in function 'fisher' with arguments 'install reitzig/sdkman-for-fish'
Installed 1 plugin/s

Fisher 4.4.4
Fish 3.7.1

fisher list

oh-my-fish/plugin-osx
jorgebucaran/nvm.fish
reitzig/sdkman-for-fish@v2.1.0

macOS 13.6.7

@mrichman mrichman reopened this Jun 25, 2024
@reitzig
Copy link
Owner

reitzig commented Jun 30, 2024

Can't reproduce on CI with these exact versions: github.com/reitzig/sdkman-for-fish/actions/runs/9735550696

Say, do you have a non-standard test for some reason? 🤔

@mrichman
Copy link
Author

Don't think so, but I'm on macOS so it might behave differently. When I install on Linux I have no issues.

@mrichman
Copy link
Author

I figured it out. I had ls aliased to eza on macOS. Removing that solved the problem. Sorry for the trouble!

@reitzig
Copy link
Owner

reitzig commented Jun 30, 2024

You'll notice that the tests run on macOS as well, and are green.

Without a way to reproduce I won't be able to investigate; don't have a Mac here myself, so it'll have to be in the workflow. Can you provide a failing test?

@reitzig
Copy link
Owner

reitzig commented Jun 30, 2024

Ah. Yet another instance of #23, then. 🤔 Re-assigning GNU-tool names seems to become more common, for better or worse; may have to look at that issue, after all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants