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

Update README.md #55

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update README.md #55

wants to merge 2 commits into from

Conversation

tkh44
Copy link

@tkh44 tkh44 commented Mar 23, 2018

Remove brew linkapps step

Warning: Calling 'brew linkapps' is deprecated!

Remove `brew linkapps` step

> Warning: Calling 'brew linkapps' is deprecated!
@gnapse
Copy link

gnapse commented Mar 29, 2018

I too get the message that this command is deprecated, and that there's no replacement. However without it the app is not available in /Applications. What should users do instead?

@jordwalke
Copy link
Owner

How about not using brew and instead using the link mentioned above it? (http://macvim-dev.github.io/macvim/).

Was that option clear, or right now does it sound like you have to use brew?

We should make a custom bundling of MacVim bundled as VimBox where:

  • All plugins are installed separately from vim ~/.vimbox or something.
  • Fonts are installed by default.
  • Modern MacVim is ensured.
  • Icon is already placed on the app.
  • App installer that installs necessary fonts.

This way your original Vim/MacVim is also left untouched.

@tkh44
Copy link
Author

tkh44 commented Apr 2, 2018

My intention with this PR is simply to keep users from even asking this question. Should I just remove any mention of brew in the docs? I could use some VIM practice with delete.

@jordwalke
Copy link
Owner

yeah, removing brew discussion would be a welcome change because at this point MacVim has really good/high quality releases - brew shouldn't be needed except for the few that actually know a lot about special MacVim builds and in that case, they don't need instructions.

@tkh44
Copy link
Author

tkh44 commented Apr 17, 2018

Mentions of installing MacVim via brew have been removed.

@idkjs
Copy link

idkjs commented May 3, 2018

Not committed to master yet. Hence brew remove macvim

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

Successfully merging this pull request may close these issues.

4 participants