From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3899 invoked by alias); 10 Sep 2014 19:17:11 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 3889 invoked by uid 89); 10 Sep 2014 19:17:11 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.8 required=5.0 tests=AWL,BAYES_00,SPF_NEUTRAL autolearn=no version=3.3.2 X-HELO: eggs.gnu.org Received: from eggs.gnu.org (HELO eggs.gnu.org) (208.118.235.92) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Wed, 10 Sep 2014 19:17:08 +0000 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XRnO0-0006We-RN for gcc-patches@gcc.gnu.org; Wed, 10 Sep 2014 15:17:06 -0400 Received: from gate.crashing.org ([63.228.1.57]:35748) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XRnNy-0006W8-9n for gcc-patches@gcc.gnu.org; Wed, 10 Sep 2014 15:17:00 -0400 Received: from gate.crashing.org (localhost.localdomain [127.0.0.1]) by gate.crashing.org (8.14.1/8.13.8) with ESMTP id s8AIq1pD026910; Wed, 10 Sep 2014 13:52:01 -0500 Received: (from segher@localhost) by gate.crashing.org (8.14.1/8.14.1/Submit) id s8AIpxr1026901; Wed, 10 Sep 2014 13:51:59 -0500 Date: Wed, 10 Sep 2014 19:17:00 -0000 From: Segher Boessenkool To: Yury Gribov Cc: GCC Patches , Laurynas Biveinis , Jeff Law , Richard Biener , Bernhard Reutner-Fischer , Trevor Saunders , Mike Stump Subject: Re: [PATCHv2] Vimrc config with GNU formatting Message-ID: <20140910185157.GA9233@gate.crashing.org> References: <540863C1.4000909@samsung.com> <54100735.5040700@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <54100735.5040700@samsung.com> User-Agent: Mutt/1.4.2.3i X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x X-Received-From: 63.228.1.57 X-IsSubscribed: yes X-SW-Source: 2014-09/txt/msg00846.txt.bz2 On Wed, Sep 10, 2014 at 12:09:25PM +0400, Yury Gribov wrote: > The patch caused a lively debate with Segher who wanted .local.vimrc to > not be enabled > by default. No, that is not what I said. I am saying it is very anti-social to make people's editor behave differently from what they are used to. Wars have been started for much less. But there are more problems with your approach. First, you are encouraging the use of a plugin that is a gaping wide security hole. I do not think GCC should encourage this. Secondly, this is a very poor imitation of the mechanism Vim has for dealing with filetypes, namely, ftplugins. > We basically have two options: > 1) put .local.vimrc to root (just like .dir-locals.el config for Emacs) > 2) put both .local.vimrc and .dir-locals.el to contrib and add Makefile > targets > to create symlinks in root folder per user's request As I said before, Emacs is not Vim. The Emacs dir-locals file simply configures some settings for files with certain major modes in that dir. For example, ours says that c-mode files should use GNU style. This is quite harmless, and probably what most Emacs users want. Your script on the other hand does something totally different: it overrides a bunch of settings the user has made elsewhere. [Snipped some overly optimistic stuff about this all increasing the quality of posted patches. Hint: the most frequently made formatting error is forgetting to put two spaces at the end of a sentence. Which this script does not handle at all (I'm not saying it should, it is hard to do reliably). The patch itself however does introduce another one of these.] Segher