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 self-update fails after fisher switched to main branch #584

Closed
DivineGod opened this issue Aug 2, 2020 · 13 comments
Closed

Fisher self-update fails after fisher switched to main branch #584

DivineGod opened this issue Aug 2, 2020 · 13 comments

Comments

@DivineGod
Copy link

I noticed fisher self-update failed with the following:

ᚦ fisher self-update                                                             
fetching https://raw.githubusercontent.com/jorgebucaran/fisher/master/fisher.fish
fisher: cannot update fisher -- are you offline?

Changing master to main in ~/.config/fish/functions/fisher.fish resolved the issue.

@meetaco
Copy link

meetaco commented Aug 2, 2020

https://git.io/fisher redirection has to be fixed as well.

@jorgebucaran
Copy link
Owner

@DivineGod @meetaco Working on it with GitHub as we speak! Thank you all for reporting. 🙏

@maparent
Copy link

maparent commented Aug 2, 2020

for the record: your fix allows the next self-update to work, but updating to the version with the fix still needs a hand-edit as proposed by @DivineGod . I suggest leaving the issue open until most users have been over that hump.

@ovv
Copy link

ovv commented Aug 2, 2020

Keeping a master branch pointing to c20d1b3 for a bit would allow for seamless self-updates

You can then remove the branch if a few days/weeks/months

@jorgebucaran
Copy link
Owner

@ovv @maparent @DivineGod Whoops, good point! I restored master and it has the latest changes now. It's just a matter of time now. 👍

@DivineGod
Copy link
Author

Awesome, thanks for the quick response :)

@anisse
Copy link

anisse commented Jan 26, 2021

Hey, I just hit this; was the old master deleted again ?

@anisse
Copy link

anisse commented Jan 26, 2021

Not only was it deleted, the script was moved to functions/ sub-directory. Changing both the branch and path allowed updating. Then I had to delete all installed plugins and reinstall them again.

@jorgebucaran
Copy link
Owner

I originally deleted the master branch after switching to main, which was a careless mistake. Then restored it to allow for seamless self-updates from older Fisher versions.

It's been a while since I published 4.x, and I needed to proceed with #651, which required moving the top-level fisher.fish file into a functions directory, thus invalidating the deprecated self-update command.

Fortunately, Fisher 3.x fails to upgrade if it can't download fisher.fish. That means we no longer have to maintain a tricky built-in migration script. It's true you can't self-update from 3.x anymore, though, so I created a step-by-step guide to help you all migrate to 4.x (#652). ✨💖

@anisse
Copy link

anisse commented Jan 26, 2021

@jorgebucaran Thanks, I wish I had read that before complaining here. Would you be open to linking to this tutorial from the README.md ? Maybe something like this at the end of the README:

How do I upgrade to fisher 4.x from fisher 3.x ?

@jorgebucaran
Copy link
Owner

What about a prominent notice right at the top of the README?

I like the last (shortest one), what do you think?

☝️ Psst! Migrating from v3.x? Check the migration guide and may the Fish be with you!
☝️ Upgrading to Fisher 4.x from 3.x or 2.x? Check out the migration guide.
☝️ How do I upgrade to Fisher 4.x from Fisher 3.x or 2.x?
☝️ How do I upgrade to Fisher 4.x?
☝️ Upgrading to Fisher 4.x?

@anisse
Copy link

anisse commented Jan 26, 2021

I prefer the third one, but honestly, anything works as long as it has a pointer with word "upgrade" or "update", which is what I was looking for. Thanks a lot for fisher.

@jorgebucaran
Copy link
Owner

Done, thank you for requesting this.

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

6 participants