[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-24 Thread dkupka
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires dkupka commented: """ Fixed upstream master: https://fedorahosted.org/freeipa/changeset/21395d1724e6bf044438a8bc25ba028ed38cde8c

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-20 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires pspacek commented: """ This pull request was (again) rebased on top of PR#171. PR#171 changes ipaplatform handling to symlinks so the issue caused by `__path__` trick should go away. I.e. the

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-20 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires pspacek commented: """ This pull request was (again) rebased on top of PR#171. PR#171 changes ipaplatform handling to symlinks so the issue caused by `__path__` trick should go away. I.e. the

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-20 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires pspacek commented: """ Commit 6256e8014aa3d68dc46e6c878be813473bffd1ac breaks pylint because it is not compatible with ipaplatform implementation introduced in

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-20 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires pspacek commented: """ Commit 6256e8014aa3d68dc46e6c878be813473bffd1ac breaks pylint because it is not compatible with ipaplatform implementation introduced in

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-20 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires pspacek commented: """ Version rebased on top of current master (without PR 171) is available from https://github.com/pspacek/freeipa/tree/pr159-rebase . """ See the full comment at

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-19 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires stlaz commented: """ @martbab Thanks, that worked. However, first set of patches was not yet ACKed in https://github.com/freeipa/freeipa/pull/171. """ See the full comment at

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-19 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires stlaz commented: """ @martbab Thanks, that worked. ACK. """ See the full comment at https://github.com/freeipa/freeipa/pull/159#issuecomment-254843561 -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-19 Thread martbab
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires martbab commented: """ On F24 you need to explicitly install python-srpm-macros due to broken package dependencies dnf install -y python-srpm-macros otherwise dnf/rpm is unable to expand

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-19 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires stlaz commented: """ For some reason, after running `sudo dnf builddep freeipa.spec`, which is successful, if I run the same command again, if fails: ``` [login@vm freeipa-git]$ sudo dnf builddep

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-19 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires stlaz commented: """ For some reason, after running `sudo dnf builddep freeipa.spec`, which is successful, if I run the same command again, if fails: ``` [login@vm freeipa-git]$ sudo dnf builddep

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-19 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires stlaz commented: """ For some reason, after running `sudo dnf builddep freeipa.spec`, which is successful, if I run the same command again, if fails: ``` [login@vm freeipa-git]$ sudo dnf builddep

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-19 Thread stlaz
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires stlaz commented: """ For some reason, after running `sudo dnf builddep freeipa.spec`, which is successful, if I run the same command again, if fails: ``` [login@vm freeipa-git]$ sudo dnf builddep

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-18 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires pspacek commented: """ - python2-dateutil and python2-sss-murmur packages are missing in with_lint condition. - the message from import error was confusing to me: >

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-13 Thread jcholast
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires jcholast commented: """ @pspacek, @mbasti-rh, fixed. """ See the full comment at https://github.com/freeipa/freeipa/pull/159#issuecomment-253491282 -- Manage your subscription for the

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-13 Thread mbasti-rh
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires mbasti-rh commented: """ Please update BUILD.txt with how to run pylint with build, probably freeipa.org should be updated as well """ See the full comment at

[Freeipa-devel] [freeipa PR#159][comment] spec file: clean up BuildRequires

2016-10-13 Thread pspacek
URL: https://github.com/freeipa/freeipa/pull/159 Title: #159: spec file: clean up BuildRequires pspacek commented: """ Interestingly, the CI failed with following errors: * Module ipatests.test_xmlrpc.test_automount_plugin ipatests/test_xmlrpc/test_automount_plugin.py:34: