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

fisher 4.0 no longer installs dependencies #614

Closed
NotAFile opened this issue Nov 11, 2020 · 9 comments
Closed

fisher 4.0 no longer installs dependencies #614

NotAFile opened this issue Nov 11, 2020 · 9 comments

Comments

@NotAFile
Copy link

NotAFile commented Nov 11, 2020

after updating to fisher 4.1 from 3.x, fisher no longer installs dependencies properly. E.g:

fisher install fish-extras/metro
[...]
fish: git_branch_name: command not found...
[...]
@NotAFile NotAFile changed the title fisher no longer correctly resolves dependencies fisher 4.0 no longer installs dependencies Nov 11, 2020
@jorgebucaran
Copy link
Owner

jorgebucaran commented Nov 11, 2020

@NotAFile Dependency support has been removed. Want to help port https://github.com/fish-extras/metro to Fisher 4.x? Just send me a PR to that repo. Otherwise, I'll get around to it in a few days.

@NotAFile
Copy link
Author

NotAFile commented Nov 11, 2020

I have to say, usual caveats about not looking into a gift horses mouth aside, I'm pretty disappointed with how this transition has been handled. Through a normal self-update, without any warning, everything is now unexpectedly broken to the point that I had to recreate my fish config directory from scratch and without even the slightest concern for backwards compatibility, existing users or a smooth migration on fisher's end (for example, not aliasing old commands and file names to new ones with a deprecation warning) and now the whole ecosystem of hundreds of plugins is suddenly incompatible overnight.

What was your plan for users here? Fish 4.x is obviously not ready for widespread use yet. There were no pre-releases to make sure it was either.

@jorgebucaran
Copy link
Owner

jorgebucaran commented Nov 11, 2020

There are very few plugins that use dependencies, and metro was one of them. Guess I forgot about that one. I'll get around to it soon enough.

I had to recreate my fish config directory from scratch

I think I've covered all possible upgrade paths already, other than no-support for dependencies, what else was broken for you?

for example, not aliasing old commands and file names to new ones with a deprecation warning)

If you use add it will translate to install automatically, and there's in fact a deprecation warning. ls and rm are still supported as is. The fishfile to fish_plugins is gracefully handled here.

@NotAFile
Copy link
Author

NotAFile commented Nov 11, 2020

Ah, I thought it's use was much more widespread. If the feature has been removed because of few users and the themes were just forgotten, that makes a lot more sense.

I'm not sure exactly what the other breakage was, but in retrospect it's probably mostly related to the symlink thing. Either way, after the upgrade, I no longer had a fisher command, and z was throwing wild errors everywhere. I'd have to try and recreate it to know more.

@jorgebucaran
Copy link
Owner

It's so odd you didn't have a fisher command. I think I am handling all possible migrations here. Is this related to #613? In that case, maybe consider helping me, as I haven't been able to reproduce it in my environment, and I won't be able to fix it until I do.

@zoomido
Copy link

zoomido commented Nov 12, 2020

I'm having problem with enhancd as well: https://github.com/b4b4r07/enhancd
I've been struggling with this and googling for hours now trying to debug why that plugin generated an error. Seems that the dependencies are missing when installing fisher and enhancd on a newer machine.
Is there a quick fix here?

@jorgebucaran
Copy link
Owner

@zoomido Depends on babarot/enhancd#139.

@jorgebucaran
Copy link
Owner

@NotAFile I just finished porting Metro (renamed as Metrofish) to Fisher 4.x and Fish 3.x. I no longer use this prompt, but I'm happy to review and merge PRs with bug fixes, and features. Enjoy!

fisher install metrofish/metrofish

@NotAFile
Copy link
Author

@jorgebucaran that's awesome, thanks so much!

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

3 participants