vim has not been completely removed

[kaliber]

Still remains references in HaikuImage and data/etc/vim/*. It should be removed or moved to the vim optional package.

[mmadia]

​HaikuPorts #308 hrev36082 improves the situation. Assigning to siarzhuk

[mmadia]

To note, installoptionalpackage is not capable of installing the vimrc file, though it won't prevent installation.

SEARCH on <etc>vimrc = [ FDirName $(HAIKU_TOP) data etc vim ] ; 
AddFilesToHaikuImage common etc vim : <etc>vimrc ; 

[siarzhuk]

Why shouldn't remove this vimrc from the Haiku basic installation? For what basic system has something that references to optional stuff?

[siarzhuk]

Replying to mmadia:

To note, installoptionalpackage is not capable of installing the vimrc file, though it won't prevent installation.

SEARCH on <etc>vimrc = [ FDirName $(HAIKU_TOP) data etc vim ] ; 
AddFilesToHaikuImage common etc vim : <etc>vimrc ; 

[the copy of my comment from HaikuPort's Trac]

In my opinion - the vimrc in common/etc/vim is a BeOS legacy. Modern versions of vim use ../share/vim/vimNN/ to look for configuration files. I'm going to check the contents of this file, collect all that can be useful for Haiku and remove it completely in favor of standard configuration files. Do you have any objections?

For what we need that vi -> vim symlink introduced in ​http://dev.haiku-os.org/changeset/36082? Is vi still required by some other Haiku utils using it as default editor? In such case we have to fix those tools to using "nano" as default editor instead of adding dangling symlink on optional stuff. Frankly speaking - that was a main reason keeping vim in the Haiku tree - because nobody completely knows from where vi is still referenced. But you have opened this can of worms ;-) - and now it is required to find all refernces to "vi" and replace them with "nano". I think the "vi -> vim" symlink doesn't solve our problem properly.

[mmadia]

Replying to siarzhuk:

[the copy of my comment from HaikuPort's Trac]

In my opinion - the vimrc in common/etc/vim is a BeOS legacy. Modern versions of vim use ../share/vim/vimNN/ to look for configuration files. I'm going to check the contents of this file, collect all that can be useful for Haiku and remove it completely in favor of standard configuration files. Do you have any objections?

For what we need that vi -> vim symlink introduced in ​http://dev.haiku-os.org/changeset/36082? Is vi still required by some other Haiku utils using it as default editor?

To be honest, I didn't realize that vimrc could be considered legacy or that vi -> vim may not be needed. (i'm more of an lpe fan) I figured moving those bits out of the default image and into the OptionalPackage file was at least a step in the right direction.

Grokking the sources for 'vi' shows that less, ncurses, readline, ctags, libs/edit have some notable references to 'vi'. For at least less, it still defaults to it.

[jackburton]

Replying to siarzhuk:

For what we need that vi -> vim symlink introduced in ​http://dev.haiku-os.org/changeset/36082? Is vi still required by some other Haiku utils using it as default editor? In such case we have to fix those tools to using "nano" as default editor

AFAIK, posix mandates a "VISUAL" environment variable which should point to the default editor. So instead of hardcoding nano, we should just define VISUAL to the location of the nano binary, and use that environment variable in those tools which hardcode vi / vim as editor.

[siarzhuk]

I have inspected mentioned data/etc/vim/vimrc file and think that it can be safely removed from the HaikuImage. It contains some mappings that were required to work with console vim in old BeOS "beterm" terminal. Haiku terminal support is more POSIX-compatible and default vimrc templates provided with Vim package for Haiku are OK. Most of other settings from this legacy data/etc/vim/vimrc are also exist in those templates. So I see no sense to keep this file in Haiku source tree. Also note that modern version of vim doesn't use any /etc/ directories to look for config files.

[mmadia]

vimrc removed in hrev36753.

[scottmc]

This appears to have been fixed but wasn't closed? Fixed in hrev36753?