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
Removing the Misc/Vim/ files #54102
Comments
While trying out vim, I noticed that the files in Misc/Vim are very poor compared to what is otherwise provided by the vim community:
Also, the vimrc uses "au BufRead,BufNewFile *.py" style selection, while the recommended modern style seems to use a ".vim/ftplugin" filetype-specific file instead. Does it still make sense to maintain these files separately? |
So the irony of saying that the files are poor is that I took that code from the highlight file from Vim itself and simply automated the creation of the files. As for the more proper way of selecting when to apply the highlights, I would not be surprised that I am wrong. The entire point of the Misc/Vim directory was so that we had files which were as up-to-date as svn is. I personally got tired of not having new keywords be recognized. I'm no Vim expert -- just a happy user -- so I'm sure there could be better out there. If we drop the files we will just have to go back to manually editing files to support new syntax until the community picks it up. But maybe devolving responsibility like that is not a bad thing. And your link for the Python 3 syntax is incorrect; it's the same as the PEP-8 indentation file. |
Those files are useful for me, even if I had to tweak them slightly. |
So I just looked at the syntax file linked by Antoine and that is definitely *not* what he meant to link to; probably meant http://www.vim.org/scripts/script.php?script_id=790 . As for the indentation file, it's out-of-date and so doesn't support 'with' statements (looks like people have already started to fork it: http://www.vim.org/scripts/script.php?script_id=3003). At this point I'm willing to either hand maintenance of the files over to someone else or to delete the files and shift what we point people at. |
On Sun, Dec 12, 2010 at 09:52:27PM +0000, Brett Cannon wrote:
I think, just pointing to the script at vim.org location is a good |
“It is maintained and updated by vimmers and we dont have to bother about it” is trumped by “I personally got tired of not having new keywords be recognized”. |
But if you have a local copy of the Vim files from the community what is preventing you from editing them for new keywords and sending a patch to the maintainer so that the rest of the community is brought up to speed that much faster? I suspect that not many people beyond core devs use the Misc/Vim file while more people in the community use the vim.org files. |
On Mon, Dec 27, 2010 at 07:59:46PM +0000, Brett Cannon wrote:
You mean the Python Core Dev maintaining a set of Vim files as a fork If that is the idea, it is indeed a good one. But often it has happened that someone at the vim.org has noticed the
Possibly and perhaps that is reason to discard our Misc/Vim in favor But if you think there *should* be an official recommendation, some |
Alright, let me change my opinion: Let’s replace the Vim files by a README.vim file explaining where to get good helper files (like Misc/README.emacs added in r85927). Then I will learn how to manage my Vim configuration to keep it updated (and adequate for each Python version), and when needed contact the Vim community to request updates. |
OK, I like Eric's idea of the README.vim file explaining where to grab the community-maintained files and how to get set up. Anyone care to take a stab at writing the file? |
Here is a first pass at the README.vim file. Suggestions welcome from VIM power-users! |
I think the bit about “enhanced version” could be puzzling. Either say “an enhanced version of the python.vim file that used to be included in the Python repository”, or just delete that part (my preference).
|
Files removed in 7e37a6c76e36. |
Did you decide against including README.vim after all or just forgot that part? |
I decided against it. If Emacs doesn't already have one I figured Vim shouldn't either. |
I had forgotten that README.Emacs was moved to the devguide. Adding doc about Vim support can be another bug report. Thanks. |
The files weres indeed removed but Misc/Readme was not updated accordingly. The last line needs to be removed. |
New changeset afd17a514117 by Brett Cannon in branch 'default': |
Thanks for the info, Févry. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: