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
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
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
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
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
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
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
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
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
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
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
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
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:
>
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
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
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:
16 matches
Mail list logo
