From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from pigeon.gentoo.org ([208.92.234.80] helo=lists.gentoo.org) by finch.gentoo.org with esmtp (Exim 4.60) (envelope-from ) id 1SXX3x-0000at-Ie for garchives@archives.gentoo.org; Thu, 24 May 2012 12:22:41 +0000 Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id E79AEE07F2 for ; Thu, 24 May 2012 12:22:40 +0000 (UTC) Received: from mail-wi0-f169.google.com (mail-wi0-f169.google.com [209.85.212.169]) by pigeon.gentoo.org (Postfix) with ESMTP id 0CAC4E079E for ; Thu, 24 May 2012 10:27:36 +0000 (UTC) Received: by wibhn14 with SMTP id hn14so5671219wib.4 for ; Thu, 24 May 2012 03:27:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; bh=G/bYaFt2gGeJKFmslZ746OWpl2gB/Pq4WSWHEzu+fi4=; b=P4puTajZTBdKksuafPQrCzzIH7LEIDktwwXf/qVhf/t4SJGheEsvsOxUCh/VdFZN5H 6j9blIXo/shEkrI2N4WmYbM84nbKVCqujyMVu1Ps2BcWfAWl0bpaakXRr7lREifr18ay y9MdDF7tolJQkd9JAzR1aaLARjhBKEOP5UNqquR1d7yq4AFQ0CblpskR1qOoQahOlsWj yloYkYdF7zsb+8UfA7mejtflIO7pyJ8ioQ+w8IazvgLYwEaAoB21IgoFPKnUlGmRjC+g hA4iT105NMBglY+B+/7sTNDxBpgr9RHE78e6XZcw1cueIWOFHyYcI+qhwAplPq0YhK3S 8l8Q== Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-portage-dev@lists.gentoo.org Reply-to: gentoo-portage-dev@lists.gentoo.org MIME-Version: 1.0 Received: by 10.216.196.221 with SMTP id r71mr3322421wen.145.1337855256256; Thu, 24 May 2012 03:27:36 -0700 (PDT) Received: by 10.194.60.167 with HTTP; Thu, 24 May 2012 03:27:36 -0700 (PDT) In-Reply-To: <1337832399-14375-1-git-send-email-vapier@gentoo.org> References: <1337832399-14375-1-git-send-email-vapier@gentoo.org> Date: Thu, 24 May 2012 22:27:36 +1200 Message-ID: Subject: Re: [gentoo-portage-dev] [RFC/PATCH] repoman: unroll escaped lines so we can check the entirety of it From: Kent Fredric To: gentoo-portage-dev@lists.gentoo.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Archives-Salt: 4222b05f-826f-40f9-bb93-fa047a45a82d X-Archives-Hash: 6b96f731695e26d07ddfcbb052438d6e On 24 May 2012 16:06, Mike Frysinger wrote: > Sometimes people wrap long lines in their ebuilds to make it easier to > read, but this causes us issues when doing line-by-line checking. =C2=A0S= o > automatically unroll those lines before passing the full content down > to our checkers. > This may be a result of the default vim settings that get enabled if you're editing an ebuild. If you don't manually wrap the lines, vim will do it for you, because the ebuild profile for vim ( app-vim/gentoo-syntax , /usr/share/vim/vimfiles/ftplugin/ebuild.vim ) sets textwidth=3D80 This in my personal experience is unhelpful, and every time the text wrap fires it risks producing broken code, and I've resorted to manually wrapping things with the proper escapes just to avoid being attacked by the wrap. ( I had assumed that the forced 80 char text width was something that was enforced by policy and wasn't game to try bucking the curve ) If there is no such policy, and a forced text-wrap at 80 characters is not needed, I would love for that setting to be removed. --=20 Kent perl -e=C2=A0 "print substr( \"edrgmaM=C2=A0 SPA NOcomil.ic\\@tfrken\", \$_= * 3, 3 ) for ( 9,8,0,7,1,6,5,4,3,2 );" http://kent-fredric.fox.geek.nz