Github user mxmrlv commented on a diff in the pull request:

    https://github.com/apache/incubator-ariatosca/pull/199#discussion_r148952959
  
    --- Diff: aria/__init__.py ---
    @@ -17,14 +17,45 @@
     The ARIA root package provides entry points for extension and storage 
initialization.
     """
     
    +import os
     import sys
    +import types
     
     from pkgutil import iter_modules
     import pkg_resources
    -
     aria_package_name = 'apache-ariatosca'
     __version__ = pkg_resources.get_distribution(aria_package_name).version
     
    +
    +
    +
    +try:
    +    import ruamel                                           # noqa: F401
    +except ImportError:
    --- End diff --
    
    The problem with ruamel is that it's missing the top level __init__. I have 
seen that the author of ruamel had no intention of fixing it. instead he 
supplied .pth files as part of the installation. This files do work in most 
cases but not when the destination dir is a custom dir. Placing this code under 
the `__init__.py` enables loading ruamel from each submodule of ARIA. I'm not 
sure how placing it under `aria.utils.yaml` would be a better fix. But if you 
feel it's better we could do that.


---

Reply via email to