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

Don't use the Cream installer for Vim #267

Open
mmikeww opened this issue Oct 28, 2017 · 9 comments
Open

Don't use the Cream installer for Vim #267

mmikeww opened this issue Oct 28, 2017 · 9 comments

Comments

@mmikeww
Copy link

mmikeww commented Oct 28, 2017

I'm not sure what the reasoning is behind using the Cream distribution for Vim, rather than the official distribution. I thought maybe the official Vim installer was using some wacko installer method, but I checked and they are both using NSIS. While I do understand the "if it aint broke dont fix it" mindset, however, this is broke:

The Cream distribution includes the windows right click context menu item for "Edit with Vim", however on choco uninstall vim, it doesn't remove that context item.

Further, the official Vim windows installer either from the Vim FTP or from the Vim Github release, includes an additional context menu for "Edit with existing Vim", which is useful if you already have Vim running. The Cream installation lacks this. Plus, its 'official', so why not use it?

@ferventcoder
Copy link
Owner

It's probably more of a history thing - cream installer was best avail at that point in time

@ferventcoder
Copy link
Owner

Agreed that this should move. How to handle existing installs on the upgrade path?

@mmikeww
Copy link
Author

mmikeww commented Oct 29, 2017

How to handle existing installs on the upgrade path?

Good question. I'm not sure entirely what Cream does different. I know the installer is definitely different than the mainline distro. And sorry I hadn't searched, because Issue #239 also mentions the shell extension not being removed, and suggests having chocolatey manually deleting it. And vim-tux won't uninstall for me either. I'd rather just use the official

@ferventcoder
Copy link
Owner

Folks want things to not chamge things much past installer defaults without params on the community repo.

Switching out an installer on upgrade could upset some folks so having a good path for them would be kind of important.

@jacktose
Copy link

Maybe just move that to vim-cream or something?

@AeliusSaionji
Copy link
Contributor

I just became aware of this myself. For an upgrade path, maybe consider the following: deprecate vim, pointing to vim-cream as a dependency. Perhaps add an extra warning that the user should really remove the vim package from chocolatey's installation. After enough time has passed (a few months?), revive vim as its own package using upstream builds. That should spare 99% of users any sudden changes. Would that be acceptable?

My install scripts for vim-tux could mostly be copy-pasted for dealing with the official upstream zips.
https://github.com/AeliusSaionji/chocopkgs/blob/master/vim-tux.install/tools/chocolateyinstall.ps1
https://github.com/AeliusSaionji/chocopkgs/blob/master/vim-tux.portable/tools/chocolateyinstall.ps1

@mmikeww fwiw I recently fixed up the vim-tux package. It should uninstall fine now, as well as be much quicker to install. The upstream builder isn't including diff.exe, though... so vimdiff won't work unless you happen to already have diff on PATH. I've sent them two emails over this, still awaiting a response.

@mmikeww
Copy link
Author

mmikeww commented May 4, 2018

Thanks AeliusSaionji, your scripts should work for the official vim release from github, with the one exception that there is no x64 executable installer, only an x86 executable and an x64.zip. So chocolately would probably just always use the x86 (which is what i use on my x64 system anyway without any issues)

@AeliusSaionji
Copy link
Contributor

@mmikeww

I would not be pleased at all with the exclusion of x64. Luckily, my scripts are written for the zip distributions, not for exe installers. Vim includes an install.exe which I manually invoke after extracting. I had intended to share my scripts to demonstrate that there is no need to rely on an exe installer. This will do:

Start-ChocolateyProcessAsAdmin "-create-batfiles vim gvim evim view gview vimdiff gvimdiff -install-popup -install-openwith -add-start-menu" "$destDir\install.exe" -validExitCodes '0'

The downloaded filetype in my script is only exe because vim-tux provides self extracting exe releases. I switched to those because I was having troubles with chocolateyunzip taking upwards of 10 minutes, often failing validation because extraction was so slow it would exceed the validator's timeout! Which is an issue I fear we might run into again here.

Aside, vim-tux dev got back to me and tells me he intentionally excludes diff and patch from his builds so I guess I'll just fix his wrong decisions on my end 😉.

@mmikeww
Copy link
Author

mmikeww commented May 24, 2022

it looks like this was changed at some point?

i cant seem to check the history of the vim package, but chocolatey doesnt look like its using the cream version anymore

edit:
found this:
chocolatey-community/chocolatey-packages@6f5bf87

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

4 participants