Gentoo Logo
Gentoo Spaceship




Note: Due to technical difficulties, the Archives are currently not up to date. GMANE provides an alternative service for most mailing lists.
c.f. bug 424647
List Archive: gentoo-portage-dev
Navigation:
Lists: gentoo-portage-dev: < Prev By Thread Next > < Prev By Date Next >
Headers:
To: gentoo-portage-dev@g.o
From: Zac Medico <zmedico@g.o>
Subject: Re: [RFC/PATCH] repoman: unroll escaped lines so we can check the entirety of it
Date: Wed, 23 May 2012 21:19:45 -0700
On 05/23/2012 09:06 PM, 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.  So
> automatically unroll those lines before passing the full content down
> to our checkers.
> 
> This seems to work, but maybe someone can suggest something simpler.

This code should come right after the line that says "We're not in a
here-document", because we only need it to trigger when we're not in a
here-document.

I think it's going to be cleaner to detect an escaped newline with a
regular expression, like r'(^|[^\\])\\$'.
-- 
Thanks,
Zac


Replies:
Re: [RFC/PATCH] repoman: unroll escaped lines so we can check the entirety of it
-- Mike Frysinger
References:
[RFC/PATCH] repoman: unroll escaped lines so we can check the entirety of it
-- Mike Frysinger
Navigation:
Lists: gentoo-portage-dev: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
[RFC/PATCH] repoman: unroll escaped lines so we can check the entirety of it
Next by thread:
Re: [RFC/PATCH] repoman: unroll escaped lines so we can check the entirety of it
Previous by date:
[RFC/PATCH] repoman: unroll escaped lines so we can check the entirety of it
Next by date:
Re: [RFC/PATCH] repoman: unroll escaped lines so we can check the entirety of it


Updated Jun 13, 2012

Summary: Archive of the gentoo-portage-dev mailing list.

Donate to support our development efforts.

Copyright 2001-2013 Gentoo Foundation, Inc. Questions, Comments? Contact us.