| Summary: | app-portage/layman-2.1.0-r3: python API layman.config output issue | ||
|---|---|---|---|
| Product: | Portage Development | Reporter: | Maxim Odinintsev <gwynn> |
| Component: | Tools | Assignee: | Portage Tools Team <tools-portage> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | gwynn |
| Priority: | Normal | Keywords: | InVCS |
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Attachments: | Layman python API output patch | ||
Created attachment 385132 [details, diff]
Layman python API output patch
Patch tested and applied. You can recompile layman and it will be in there. Thank you for your contribution to layman's codebase. Available in layman-2.2.0, closing as fixed. |
'BareConfig' object has no attribute 'output' Reproducible: Always Steps to Reproduce: Simple python script: try: from layman.api import LaymanAPI from layman.config import BareConfig HAS_LAYMAN_API = True except ImportError: HAS_LAYMAN_API = False def init_layman(config=None): if config is None: config = BareConfig(read_configfile=True, quietness=1) return LaymanAPI(config) layman = init_layman() name = layman.get_available() print(name) Actual Results: Traceback (most recent call last): File "./layman-api-test.py", line 17, in <module> layman.get_available() File "/usr/lib64/python2.7/site-packages/layman/api.py", line 559, in get_available self._available_ids = self._get_remote_db(dbreload).list_ids() File "/usr/lib64/python2.7/site-packages/layman/api.py", line 582, in _get_remote_db self._available_db = RemoteDB(self.config) File "/usr/lib64/python2.7/site-packages/layman/remotedb.py", line 50, in __init__ self.proxies = config.proxies File "/usr/lib64/python2.7/site-packages/layman/config.py", line 299, in proxies self.output.debug("Warning: unable to determine proxies.", 6) AttributeError: 'BareConfig' object has no attribute 'output' Expected Results: [u'Armageddon', u'AstroFloyd', u'AzP', u'DuPol', u'FireBurn', u'GoGooOS', u'LaraCraft304', u'OSSDL', u'ROKO__', u'THE', u'TomWij', u'abendbrot', u'abnorm', u'activehome', u'ago', u'aidecoe', u'akoya', u'alatar-lay', u'alba-overlay', u'alexcepoi', u'alexxy', u'alice', u'aliceinwire', u'aluco', u'alunduil', u'amielke-overlay', u'anarchy', u'anaximander', u'and3k-sunrise', u'anderse', u'andy', u'angelos', u'anyc', u'arax-overlay', u'arc', u'arces', u'arcon', u'armagetron', u'aross', u'arx-libertatis', u'asux'] # diff -u /usr/lib64/python2.7/site-packages/layman/config.py.orig /usr/lib64/python2.7/site-packages/layman/config.py --- /usr/lib64/python2.7/site-packages/layman/config.py.orig 2014-09-19 20:54:21.216670781 +0300 +++ /usr/lib64/python2.7/site-packages/layman/config.py 2014-09-19 20:54:59.463691942 +0300 @@ -296,7 +296,7 @@ elif os.getenv(proxy): proxies[proxy.split('_')[0]] = os.getenv(proxy) if self.config and proxies == {}: - self.output.debug("Warning: unable to determine proxies.", 6) + self._options['output'].debug("Warning: unable to determine proxies.", 6) return proxies