Re: [PATCH v1 12/25] contrib: remove 'vim'

2014-05-08 Thread Jeff King
On Thu, May 08, 2014 at 07:58:23PM -0500, Felipe Contreras wrote:

 There's nothing there.

It's not nothing; we used to carry the tools here, and replaced them
with pointers when the tools themselves went away. That was certainly
useful for a period of time.

However, I would certainly agree that that period of time is probably
over; the scripts started shipping in upstream vim in mid-2008. I'd be
happy to see this directory go away whether or not the rest of contrib/
is dropped.

-Peff
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v1 12/25] contrib: remove 'vim'

2014-05-08 Thread Jonathan Nieder
Hi,

Jeff King wrote:

 However, I would certainly agree that that period of time is probably
 over; the scripts started shipping in upstream vim in mid-2008. I'd be
 happy to see this directory go away whether or not the rest of contrib/
 is dropped.

RHEL 6 has vim 7.2.something, so yeah, this should be mostly safe.

Git needs to keep working for people stuck on RHEL 5, but niceties
like vim support seem less important there.  But I am not convinced it
is worth inconveniencing them (or putting any obstacles in the way of
upgrading) without a good reason, and one less directory in contrib/
does not seem like a very strong reason.

Here's a new commit message in case we want to do this.

-- 8 --
Subject: contrib: remove vim support instructions

The git support scripts started shipping in upstream vim in version
7.2 (2008-08-09).  Clean up contrib/ a little by removing the
instructions for people on older versions of vim.

RHEL 6 already has vim 7.2.something, so anyone on a reasonably modern
operating system should not be affected.  Users on RHEL 5 presumably
know that means sometimes missing out on niceties like syntax
highlighting, so this should be safe.

Signed-off-by: Jonathan Nieder jrnie...@gmail.com
---
 contrib/vim/README | 22 --
 1 file changed, 22 deletions(-)
 delete mode 100644 contrib/vim/README

diff --git a/contrib/vim/README b/contrib/vim/README
deleted file mode 100644
index 8f16d06..000
--- a/contrib/vim/README
+++ /dev/null
@@ -1,22 +0,0 @@
-Syntax highlighting for git commit messages, config files, etc. is
-included with the vim distribution as of vim 7.2, and should work
-automatically.
-
-If you have an older version of vim, you can get the latest syntax
-files from the vim project:
-
-  http://ftp.vim.org/pub/vim/runtime/syntax/git.vim
-  http://ftp.vim.org/pub/vim/runtime/syntax/gitcommit.vim
-  http://ftp.vim.org/pub/vim/runtime/syntax/gitconfig.vim
-  http://ftp.vim.org/pub/vim/runtime/syntax/gitrebase.vim
-  http://ftp.vim.org/pub/vim/runtime/syntax/gitsendemail.vim
-
-These files are also available via FTP at the same location.
-
-To install:
-
-  1. Copy these files to vim's syntax directory $HOME/.vim/syntax
-  2. To auto-detect the editing of various git-related filetypes:
-
-   $ curl http://ftp.vim.org/pub/vim/runtime/filetype.vim |
-   sed -ne '/^ Git$/, /^$/ p' $HOME/.vim/filetype.vim
-- 
1.9.1.423.g4596e3a

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html