Gentoo Archives: gentoo-dev

From: "Michał Górny" <mgorny@g.o>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] [PATCH 03/29] python-utils-r1.eclass: Add missing ||die on file read
Date: Thu, 26 Nov 2015 22:27:12
Message-Id: 20151126232642.3149fcf8.mgorny@gentoo.org
In Reply to: Re: [gentoo-dev] [PATCH 03/29] python-utils-r1.eclass: Add missing ||die on file read by "Michał Górny"
1 On Wed, 25 Nov 2015 17:16:24 +0100
2 Michał Górny <mgorny@g.o> wrote:
3
4 > On Sun, 22 Nov 2015 20:50:44 +0100
5 > Michał Górny <mgorny@g.o> wrote:
6 >
7 > > ---
8 > > eclass/python-utils-r1.eclass | 4 ++--
9 > > 1 file changed, 2 insertions(+), 2 deletions(-)
10 > >
11 > > diff --git a/eclass/python-utils-r1.eclass b/eclass/python-utils-r1.eclass
12 > > index 0a04e12..201b0c4 100644
13 > > --- a/eclass/python-utils-r1.eclass
14 > > +++ b/eclass/python-utils-r1.eclass
15 > > @@ -1038,12 +1038,12 @@ python_fix_shebang() {
16 > > local shebang i
17 > > local error= from=
18 > >
19 > > - IFS= read -r shebang <"${f}"
20 > > + IFS= read -r shebang <"${f}" || die
21 >
22 > This gives failure for empty files. We need to revert it, and possibly
23 > find a better way of distinguishing I/O failure from 'no input' error.
24
25 Going to add two more patches, one of them fixing this.
26
27 --
28 Best regards,
29 Michał Górny
30 <http://dev.gentoo.org/~mgorny/>