[Freeipa-devel] [freeipa PR#515][comment] Re-add ipapython.config.config for backwards compatibilty

2017-03-01 Thread tiran
  URL: https://github.com/freeipa/freeipa/pull/515
Title: #515: Re-add ipapython.config.config for backwards compatibilty

tiran commented:
"""
OK, I'm closing this PR then. 

@HonzaCholasta, please open another PR and revert 7b966e8.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/515#issuecomment-283275063
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#515][comment] Re-add ipapython.config.config for backwards compatibilty

2017-02-28 Thread HonzaCholasta
  URL: https://github.com/freeipa/freeipa/pull/515
Title: #515: Re-add ipapython.config.config for backwards compatibilty

HonzaCholasta commented:
"""
Could we please revert to the original `IPAConfig` implementation rather than 
wrapping around `api.env`? I know I'm the one who suggested it, but I have 
given it some thought and I would rather not have to import from `ipalib` into 
`ipapython` and keep the original behavior intact.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/515#issuecomment-283260315
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#515][comment] Re-add ipapython.config.config for backwards compatibilty

2017-02-28 Thread HonzaCholasta
  URL: https://github.com/freeipa/freeipa/pull/515
Title: #515: Re-add ipapython.config.config for backwards compatibilty

HonzaCholasta commented:
"""
Could we please revert to the original `IPAConfig` implementation rather than 
wrapping around `api.env`? I know I'm the one who suggested it, but I have 
given it some thought and I would rather not have to import from `ipalib` into 
`ipapython` and keep the original behavior intact.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/515#issuecomment-283260315
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#515][comment] Re-add ipapython.config.config for backwards compatibilty

2017-02-28 Thread tiran
  URL: https://github.com/freeipa/freeipa/pull/515
Title: #515: Re-add ipapython.config.config for backwards compatibilty

tiran commented:
"""
I can add a deprecation warning after we have agreed upon a new API. What's the 
official way to get the values w/o requiring credentials?
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/515#issuecomment-283111844
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [freeipa PR#515][comment] Re-add ipapython.config.config for backwards compatibilty

2017-02-28 Thread MartinBasti
  URL: https://github.com/freeipa/freeipa/pull/515
Title: #515: Re-add ipapython.config.config for backwards compatibilty

MartinBasti commented:
"""
IIRC we agreed that there should be warning that this is deprecated and 
`api.env` should be used instead.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/515#issuecomment-283098177
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code