[gentoo-commits] proj/R_overlay:master commit in: roverlay/depres/simpledeprule/, roverlay/util/

[gentoo-commits] proj/R_overlay:master commit in: roverlay/depres/simpledeprule/, roverlay/util/

[André Erdmann]

commit:     44570dcbf68669e76398d31eda2b098e41f6c35c
Author:     André Erdmann <dywi <AT> mailerd <DOT> de>
AuthorDate: Wed Jul 10 14:21:07 2013 +0000
Commit:     André Erdmann <dywi <AT> mailerd <DOT> de>
CommitDate: Wed Jul 10 14:21:07 2013 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=proj/R_overlay.git;a=commit;h=44570dcb

roverlay/depres, rule maker: reset deptype

---
 roverlay/depres/simpledeprule/rulemaker.py |  6 ++++++
 roverlay/util/mapreader.py                 | 12 ++++++++++++
 2 files changed, 18 insertions(+)

diff --git a/roverlay/depres/simpledeprule/rulemaker.py b/roverlay/depres/simpledeprule/rulemaker.py
index 3f9ab73..d759b56 100644
--- a/roverlay/depres/simpledeprule/rulemaker.py
+++ b/roverlay/depres/simpledeprule/rulemaker.py
@@ -49,6 +49,12 @@ class SimpleRuleMaker ( roverlay.util.mapreader.MapFileParser ):
       self._deptype_once   = deptype.NONE
    # --- end of __init__ (...) ---
 
+   def read_lines_begin ( self ):
+      # reset deptype
+      self._deptype      = deptype.ALL
+      self._deptype_once = deptype.NONE
+   # --- end of read_lines_begin (...) ---
+
    def make_result ( self, as_pool=False ):
       rule_count = len ( self._items )
       if as_pool:

diff --git a/roverlay/util/mapreader.py b/roverlay/util/mapreader.py
index 2958616..c2cce1f 100644
--- a/roverlay/util/mapreader.py
+++ b/roverlay/util/mapreader.py
@@ -118,7 +118,16 @@ class MapFileParser ( object ):
          )
    # --- end of add (...) ---
 
+   def read_lines_done ( self ):
+      pass
+   # --- end of read_lines_done (...) ---
+
+   def read_lines_begin ( self ):
+      pass
+   # --- end of read_lines_begin (...) ---
+
    def read_lines ( self, lines ):
+      self.read_lines_begin()
       self.stop_reading = False
       ret = True
       for lino, line in enumerate ( lines ):
@@ -130,6 +139,9 @@ class MapFileParser ( object ):
 
          if self.stop_reading:
             break
+
+      self.read_lines_done()
+      return ret
    # --- end of read_lines (...) ---
 
    def read_file ( self, filepath, handle_compressed=True ):

[gentoo-commits] proj/R_overlay:gsoc13/next commit in: roverlay/depres/simpledeprule/, roverlay/util/

[André Erdmann]

commit:     d142a11212fdf194eb6d3940ce35e205dfa37e8c
Author:     André Erdmann <dywi <AT> mailerd <DOT> de>
AuthorDate: Fri Jul 19 17:42:17 2013 +0000
Commit:     André Erdmann <dywi <AT> mailerd <DOT> de>
CommitDate: Fri Jul 19 17:42:17 2013 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=proj/R_overlay.git;a=commit;h=d142a112

fix has_context() in mapreader/rulemaker

---
 roverlay/depres/simpledeprule/rulemaker.py | 7 +++++++
 roverlay/util/mapreader.py                 | 5 +----
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/roverlay/depres/simpledeprule/rulemaker.py b/roverlay/depres/simpledeprule/rulemaker.py
index d759b56..cfb2736 100644
--- a/roverlay/depres/simpledeprule/rulemaker.py
+++ b/roverlay/depres/simpledeprule/rulemaker.py
@@ -49,6 +49,13 @@ class SimpleRuleMaker ( roverlay.util.mapreader.MapFileParser ):
       self._deptype_once   = deptype.NONE
    # --- end of __init__ (...) ---
 
+   def has_context ( self ):
+      return (
+         self._deptype_once != deptype.NONE
+         or self._next is not None
+      )
+   # --- end of has_context (...) ---
+
    def read_lines_begin ( self ):
       # reset deptype
       self._deptype      = deptype.ALL

diff --git a/roverlay/util/mapreader.py b/roverlay/util/mapreader.py
index 5333bd9..56189ba 100644
--- a/roverlay/util/mapreader.py
+++ b/roverlay/util/mapreader.py
@@ -44,10 +44,7 @@ class MapFileParser ( object ):
    # --- end of zap (...) ---
 
    def has_context ( self ):
-      return (
-         self._deptype_once != deptype.NONE
-         or self._next is not None
-      )
+      return ( self_next is not None )
    # --- end of has_context (...) ---
 
    def make_result ( self ):

[gentoo-commits] proj/R_overlay:master commit in: roverlay/depres/simpledeprule/, roverlay/util/

[André Erdmann]

commit:     d142a11212fdf194eb6d3940ce35e205dfa37e8c
Author:     André Erdmann <dywi <AT> mailerd <DOT> de>
AuthorDate: Fri Jul 19 17:42:17 2013 +0000
Commit:     André Erdmann <dywi <AT> mailerd <DOT> de>
CommitDate: Fri Jul 19 17:42:17 2013 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=proj/R_overlay.git;a=commit;h=d142a112

fix has_context() in mapreader/rulemaker

---
 roverlay/depres/simpledeprule/rulemaker.py | 7 +++++++
 roverlay/util/mapreader.py                 | 5 +----
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/roverlay/depres/simpledeprule/rulemaker.py b/roverlay/depres/simpledeprule/rulemaker.py
index d759b56..cfb2736 100644
--- a/roverlay/depres/simpledeprule/rulemaker.py
+++ b/roverlay/depres/simpledeprule/rulemaker.py
@@ -49,6 +49,13 @@ class SimpleRuleMaker ( roverlay.util.mapreader.MapFileParser ):
       self._deptype_once   = deptype.NONE
    # --- end of __init__ (...) ---
 
+   def has_context ( self ):
+      return (
+         self._deptype_once != deptype.NONE
+         or self._next is not None
+      )
+   # --- end of has_context (...) ---
+
    def read_lines_begin ( self ):
       # reset deptype
       self._deptype      = deptype.ALL

diff --git a/roverlay/util/mapreader.py b/roverlay/util/mapreader.py
index 5333bd9..56189ba 100644
--- a/roverlay/util/mapreader.py
+++ b/roverlay/util/mapreader.py
@@ -44,10 +44,7 @@ class MapFileParser ( object ):
    # --- end of zap (...) ---
 
    def has_context ( self ):
-      return (
-         self._deptype_once != deptype.NONE
-         or self._next is not None
-      )
+      return ( self_next is not None )
    # --- end of has_context (...) ---
 
    def make_result ( self ):

[gentoo-commits] proj/R_overlay:master commit in: roverlay/depres/simpledeprule/, roverlay/util/

[André Erdmann]

commit:     b60aa29c76d44975a20ef6206473aa266947e225
Author:     André Erdmann <dywi <AT> mailerd <DOT> de>
AuthorDate: Fri Feb 21 23:00:06 2014 +0000
Commit:     André Erdmann <dywi <AT> mailerd <DOT> de>
CommitDate: Sat Feb 22 14:23:12 2014 +0000
URL:        http://git.overlays.gentoo.org/gitweb/?p=proj/R_overlay.git;a=commit;h=b60aa29c

roverlay/depres, rule reader: properly count files

fix a small bug that affects logging only where
SimpleDependencyRuleReader.read() always reports a file count of 0.

Also, remove an unnecessary "self._pool_add is [not] None" check.

---
 roverlay/depres/simpledeprule/reader.py    | 16 +++++++---------
 roverlay/depres/simpledeprule/rulemaker.py |  5 +++++
 roverlay/util/mapreader.py                 | 10 ++++++++++
 3 files changed, 22 insertions(+), 9 deletions(-)

diff --git a/roverlay/depres/simpledeprule/reader.py b/roverlay/depres/simpledeprule/reader.py
index 22f3cf3..3cdc7cf 100644
--- a/roverlay/depres/simpledeprule/reader.py
+++ b/roverlay/depres/simpledeprule/reader.py
@@ -36,8 +36,6 @@ class SimpleDependencyRuleReader ( object ):
 
       self._pool_add = pool_add
       self._when_done = when_done
-
-      self._fcount = 0
    # --- end of __init__  (...) ---
 
    def read ( self, files_or_dirs ):
@@ -52,17 +50,17 @@ class SimpleDependencyRuleReader ( object ):
             "Read method is for resolver, but pool_add is None."
       )
 
+      self._rmaker.file_count = 0
       self.read_files ( files_or_dirs )
 
       rule_count, pools = self._rmaker.done ( as_pool=True )
       self.logger.debug ( "Read {} rules in {} files.".format (
-         rule_count, self._fcount
+         rule_count, self._rmaker.file_count
       ) )
-      if self._pool_add is not None:
-         for p in pools: self._pool_add ( p )
 
-         if self._when_done is not None:
-            self._when_done()
-      else:
-         return pools
+      for p in pools:
+         self._pool_add ( p )
+
+      if self._when_done is not None:
+         self._when_done()
    # --- end of read (...) ---

diff --git a/roverlay/depres/simpledeprule/rulemaker.py b/roverlay/depres/simpledeprule/rulemaker.py
index 5daf058..8a4acc5 100644
--- a/roverlay/depres/simpledeprule/rulemaker.py
+++ b/roverlay/depres/simpledeprule/rulemaker.py
@@ -47,6 +47,7 @@ class SimpleRuleMaker ( roverlay.util.mapreader.MapFileParser ):
       self.deptype_kw      = 'deptype'
       self._deptype        = deptype.ALL
       self._deptype_once   = deptype.NONE
+      self.file_count      = 0
    # --- end of __init__ (...) ---
 
    def has_context ( self ):
@@ -62,6 +63,10 @@ class SimpleRuleMaker ( roverlay.util.mapreader.MapFileParser ):
       self._deptype_once = deptype.NONE
    # --- end of read_lines_begin (...) ---
 
+   def read_file_done ( self, filepath ):
+      self.file_count += 1
+   # --- end of read_file_done (...) ---
+
    def make_result ( self, as_pool=False ):
       rule_count = len ( self._items )
       if as_pool:

diff --git a/roverlay/util/mapreader.py b/roverlay/util/mapreader.py
index 36f4ba7..9ee0944 100644
--- a/roverlay/util/mapreader.py
+++ b/roverlay/util/mapreader.py
@@ -150,8 +150,17 @@ class MapFileParser ( object ):
       return ret
    # --- end of read_lines (...) ---
 
+   def read_file_begin ( self, filepath ):
+      pass
+   # --- end of read_file_begin (...) ---
+
+   def read_file_done ( self, filepath ):
+      pass
+   # --- end of read_file_done (...) ---
+
    def read_file ( self, filepath, handle_compressed=True ):
       ret = None
+      self.read_file_begin ( filepath )
       try:
          if handle_compressed:
             ret = self.read_lines (
@@ -166,6 +175,7 @@ class MapFileParser ( object ):
          )
          raise
 
+      self.read_file_done ( filepath )
       return ret
    # --- end of read_file (...) ---