Uploaded image for project: 'SX Spring Python'
  1. SX Spring Python
  2. SESPRINGPYTHONPY-114

Move amara import statement inside XML-based parsers, move pyyaml inside YamlConfig

This issue belongs to an archived project. You can view it, but you can't modify it. Learn more

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.1.0.M2
    • IoC
    • None

    Description

      Moving key library import statements to inside the class would make it an optional import. No XMLConfig, no amara. Regarding elementTree, see SESPRINGPYTHONPY-98, where we are replacing amara with elementTree.

      ===============================================================================

      Here I see two issues:

      • the amara dep is not resolved
      • a minor one: if the same thing could be achieved via elementtree, it
        would be far better, since the latter module is part of the standard
        python dist since python 2.5. This would let springpython to run with
        one dep less.

      Another one:
      >>> from springpython.config import PythonConfig
      Traceback (most recent call last):
      File "<stdin>", line 1, in <module>
      File
      "build/bdist.macosx-10.5-i386/egg/springpython/config/_init_.py", line
      16, in <module>
      import logging
      ImportError: No module named amara
      >>>

      This is another drawback of putting too many things into a single
      _init_.py - why should I need amara (an xml toolkit) if I don't need
      XMLConfig ? It is an unnecessary dep.

      Attachments

        Issue Links

          Activity

            People

              gregturn Greg Turnquist
              gregturn Greg Turnquist
              Archiver:
              tmarshall Trevor Marshall

              Dates

                Created:
                Updated:
                Resolved:
                Archived: