Gentoo Archives: gentoo-commits

From: "André Erdmann" <dywi@×××××××.de>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] proj/R_overlay:master commit in: roverlay/rpackage/
Date: Mon, 02 Jul 2012 16:53:02
Message-Id: 1341247369.39ec3079553c96ff8f1d0522b47d8f6f09fa6d7c.dywi@gentoo
1 commit: 39ec3079553c96ff8f1d0522b47d8f6f09fa6d7c
2 Author: André Erdmann <dywi <AT> mailerd <DOT> de>
3 AuthorDate: Mon Jul 2 16:42:49 2012 +0000
4 Commit: André Erdmann <dywi <AT> mailerd <DOT> de>
5 CommitDate: Mon Jul 2 16:42:49 2012 +0000
6 URL: http://git.overlays.gentoo.org/gitweb/?p=proj/R_overlay.git;a=commit;h=39ec3079
7
8 DescriptionReader: decode non-str lines
9
10 modified: roverlay/rpackage/descriptionreader.py
11
12 ---
13 roverlay/rpackage/descriptionreader.py | 15 ++++++++-------
14 1 files changed, 8 insertions(+), 7 deletions(-)
15
16 diff --git a/roverlay/rpackage/descriptionreader.py b/roverlay/rpackage/descriptionreader.py
17 index 94f36a3..1b3f62f 100644
18 --- a/roverlay/rpackage/descriptionreader.py
19 +++ b/roverlay/rpackage/descriptionreader.py
20 @@ -215,22 +215,23 @@ class DescriptionReader ( object ):
21 else:
22 fh = th.extractfile ( config.get ( 'DESCRIPTION.file_name' ) )
23
24 - # have to decode the lines
25 - read = lambda lines : [ line.decode().rstrip() for line in lines ]
26 else:
27 # open file handle only
28 fh = open ( filepath, 'r' )
29 - read = lambda lines : [ line.rstrip() for line in lines ]
30
31 - x = None
32 - read_lines = read ( fh.readlines() )
33 - del x, read
34 +
35 + # decode lines of they're only bytes, using isinstance ( <>, str )
36 + # 'cause isinstance ( <str>, bytes ) returns True
37 + read_lines = tuple (
38 + ( line if isinstance ( line, str ) else line.decode() ).rstrip()
39 + for line in fh.readlines()
40 + )
41
42 fh.close()
43 if not th is None: th.close()
44 del fh, th
45
46 - if hasattr ( self, 'write_desc_file' ):
47 + if read_lines and hasattr ( self, 'write_desc_file' ):
48 try:
49 util.dodir ( DescriptionReader.WRITE_DESCFILES_DIR )
50 fh = open ( self.write_desc_file, 'w' )