* comments

pisi/specfile.py

@@ -409,6 +409,8 @@ class SpecFile(XmlFile):
 
     def has_errors(self):
         """Return errors of the PSPEC file if there are any."""
+        #TODO: has_errors name is misleading for a function that does
+        #not just return a boolean value. check() would be better - exa
         err = Checks()
         err.join(self.source.has_errors())
         if len(self.packages) <= 0:

pisi/toplevel.py

@@ -103,8 +103,6 @@ def install_pkg_files(package_URIs):
     extra_packages = [x.package for x in dep_unsatis]
     if (extra_packages and install_pkg_names(extra_packages)) or \
            (not extra_packages):
-        #FIXME: Construct a dependency graph for files
-        # and install in rev. topological order
     
         class PackageDB:
             def __init__(self):