URL: https://github.com/freeipa/freeipa/pull/503
Author: Akasurde
Title: #503: [WIP] Update testcase for cert plugin
Action: opened
PR body:
"""
Fixes https://fedorahosted.org/freeipa/ticket/6275
Signed-off-by: Abhijeet Kasurde
"""
To pull the PR as Git branch:
git
On 02/24/2017 08:34 AM, Standa Laznicka wrote:
> On 02/24/2017 08:29 AM, Jan Cholasta wrote:
>> On 23.2.2017 19:06, Martin Basti wrote:
>>>
>>>
>>> On 23.02.2017 15:09, Tomas Krizek wrote:
On 02/22/2017 01:44 PM, Fraser Tweedale wrote:
> On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas
URL: https://github.com/freeipa/freeipa/pull/502
Author: tiran
Title: #502: Make pylint and jsl optional
Action: opened
PR body:
"""
./configure no longer fails when pylint or jsl are not available. The
make targets for pylint and jsl are no longer defined without the tools.
Rational:
pylint
On 02/24/2017 08:29 AM, Jan Cholasta wrote:
On 23.2.2017 19:06, Martin Basti wrote:
On 23.02.2017 15:09, Tomas Krizek wrote:
On 02/22/2017 01:44 PM, Fraser Tweedale wrote:
On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas Krizek wrote:
On 02/22/2017 12:28 AM, Fraser Tweedale wrote:
On Tue,
URL: https://github.com/freeipa/freeipa/pull/501
Author: tiran
Title: #501: C compilation fixes and hardening
Action: opened
PR body:
"""
Fix "implicit declaration of function ‘strlen’" in ipa_pwd_ntlm.c,
credits to Lukas.
Add -Werror=implicit-function-declaration to CFLAGS to point
On 23.2.2017 19:06, Martin Basti wrote:
On 23.02.2017 15:09, Tomas Krizek wrote:
On 02/22/2017 01:44 PM, Fraser Tweedale wrote:
On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas Krizek wrote:
On 02/22/2017 12:28 AM, Fraser Tweedale wrote:
On Tue, Feb 21, 2017 at 05:23:07PM +0100, Standa
URL: https://github.com/freeipa/freeipa/pull/500
Author: tomaskrizek
Title: #500: Replace sha1 fingerprints with sha256
Action: opened
PR body:
"""
- we probably want to keep SHA1 for DNS SSHFP (along with SHA256)
- removing SHA1 from RSA-OAEP probably doesn't have any benefits
URL: https://github.com/freeipa/freeipa/pull/367
Title: #367: Remove nsslib from IPA
stlaz commented:
"""
Hopefully all issues were addressed + `radb` removed. If the Travis check
passes then this is ready for review again.
"""
See the full comment at
On 23.02.2017 15:09, Tomas Krizek wrote:
On 02/22/2017 01:44 PM, Fraser Tweedale wrote:
On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas Krizek wrote:
On 02/22/2017 12:28 AM, Fraser Tweedale wrote:
On Tue, Feb 21, 2017 at 05:23:07PM +0100, Standa Laznicka wrote:
On 02/21/2017 04:24 PM,
URL: https://github.com/freeipa/freeipa/pull/483
Title: #483: lite-server: validate LDAP connection and cache schema
MartinBasti commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/dcb618152572ca013a447336e13d24399b5f7960
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/483
Author: tiran
Title: #483: lite-server: validate LDAP connection and cache schema
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/483/head:pr483
git checkout
URL: https://github.com/freeipa/freeipa/pull/483
Title: #483: lite-server: validate LDAP connection and cache schema
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/482
Title: #482: Remove MD5 certificate fingerprints
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/412
Title: #412: Define template version in certmap.conf
MartinBasti commented:
"""
needs rebase
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/412#issuecomment-282069402
--
Manage your subscription for the Freeipa-devel
URL: https://github.com/freeipa/freeipa/pull/482
Author: stlaz
Title: #482: Remove MD5 certificate fingerprints
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/482/head:pr482
git checkout pr482
--
Manage your
URL: https://github.com/freeipa/freeipa/pull/498
Title: #498: compat: fix `Any` params in `batch` and `dnsrecord`
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/499
Author: tscherf
Title: #499: added help about default value for --external-ca-type option
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/499/head:pr499
git
URL: https://github.com/freeipa/freeipa/pull/499
Title: #499: added help about default value for --external-ca-type option
MartinBasti commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/573a0f1ffe5035b75fb88ee7752029e34c6b37af
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/499
Title: #499: added help about default value for --external-ca-type option
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/498
Author: HonzaCholasta
Title: #498: compat: fix `Any` params in `batch` and `dnsrecord`
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/498/head:pr498
git checkout
URL: https://github.com/freeipa/freeipa/pull/498
Title: #498: compat: fix `Any` params in `batch` and `dnsrecord`
MartinBasti commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/19060db1b8fa9d1d3e8f3ac3fcd1f387e9a39c94
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/498
Title: #498: compat: fix `Any` params in `batch` and `dnsrecord`
MartinBasti commented:
"""
Works for me
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/498#issuecomment-282068296
--
Manage your subscription for the
URL: https://github.com/freeipa/freeipa/pull/498
Title: #498: compat: fix `Any` params in `batch` and `dnsrecord`
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/472
Author: tiran
Title: #472: Packaging: Add placeholder packages
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/472/head:pr472
git checkout pr472
From
URL: https://github.com/freeipa/freeipa/pull/479
Author: martbab
Title: #479: Merge AD trust installer into composite ones
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/479/head:pr479
git checkout pr479
URL: https://github.com/freeipa/freeipa/pull/139
Author: pvomacka
Title: #139: WebUI: Vault Management
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/139/head:pr139
git checkout pr139
From
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
pvomacka commented:
"""
@tiran Yes, rebased.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/139#issuecomment-282060928
--
Manage your subscription for the Freeipa-devel mailing list:
URL: https://github.com/freeipa/freeipa/pull/139
Title: #139: WebUI: Vault Management
tiran commented:
"""
@MartinBasti you approved this PR a month ago but it has neither the ACK flag
nor was it merged.
@pvomacka Your work would be useful for my Custodia Vault work. Can you rebase
this PR
URL: https://github.com/freeipa/freeipa/pull/483
Title: #483: lite-server: validate LDAP connection and cache schema
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/483
Title: #483: lite-server: validate LDAP connection and cache schema
MartinBasti commented:
"""
Code looks good to me, as far as this is just code for developers I assume that
@tiran tested it enough, so ACK.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/379
Title: #379: Packaging: Add placeholder and IPA commands packages
tiran commented:
"""
I have postponed the ```ipacommands``` part. Placeholders are covered by #472.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/379
Title: #379: Packaging: Add placeholder and IPA commands packages
Label: +postponed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/472
Author: tiran
Title: #472: Packaging: Add placeholder packages
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/472/head:pr472
git checkout pr472
From
URL: https://github.com/freeipa/freeipa/pull/482
Title: #482: Remove MD5 certificate fingerprints
tomaskrizek commented:
"""
ACK, there is no disagreement on the freeipa-devel.
I'm already working on replacing SHA1 with SHA256 given the [recent
URL: https://github.com/freeipa/freeipa/pull/482
Title: #482: Remove MD5 certificate fingerprints
Label: +ack
--
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
URL: https://github.com/freeipa/freeipa/pull/472
Author: tiran
Title: #472: Packaging: Add placeholder packages
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/472/head:pr472
git checkout pr472
From
On 02/22/2017 01:44 PM, Fraser Tweedale wrote:
> On Wed, Feb 22, 2017 at 01:41:22PM +0100, Tomas Krizek wrote:
>> On 02/22/2017 12:28 AM, Fraser Tweedale wrote:
>>> On Tue, Feb 21, 2017 at 05:23:07PM +0100, Standa Laznicka wrote:
On 02/21/2017 04:24 PM, Tomas Krizek wrote:
> On 02/21/2017
URL: https://github.com/freeipa/freeipa/pull/484
Author: stlaz
Title: #484: FIPS: Remove pkispawn cruft
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/484/head:pr484
git checkout pr484
--
Manage your
URL: https://github.com/freeipa/freeipa/pull/484
Title: #484: FIPS: Remove pkispawn cruft
pvoborni commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/728a6bd4229ba170b2e94f216127b19d5d94e2ba
URL: https://github.com/freeipa/freeipa/pull/484
Title: #484: FIPS: Remove pkispawn cruft
Label: +pushed
--
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
URL: https://github.com/freeipa/freeipa/pull/499
Title: #499: added help about default value for --external-ca-type option
MartinBasti commented:
"""
Thanks!
Next time please use `git push --force` to update current pull request instead
creating a new one.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/497
Title: #497: added more meaningful help for --external-ca-type option
Label: +rejected
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/497
Title: #497: added more meaningful help for --external-ca-type option
MartinBasti commented:
"""
Replaced by #499
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/497#issuecomment-281995516
--
Manage your subscription for
URL: https://github.com/freeipa/freeipa/pull/499
Title: #499: added help about default value for --external-ca-type option
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/499
Title: #499: added help about default value for --external-ca-type option
MartinBasti commented:
"""
Thanks!
Next time please use 'git push --force' to update current pull request instead
a creating a new one.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
tiran commented:
"""
Commit 1f8326aa fixes an issue in ```Makefile.python.am```. I think the issue
caused ```ipaplatform``` and ```pypi/ipaplatform``` to cross streams.
"""
See the full comment
URL: https://github.com/freeipa/freeipa/pull/412
Title: #412: Define template version in certmap.conf
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/412
Title: #412: Define template version in certmap.conf
tomaskrizek commented:
"""
Works as expected.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/412#issuecomment-28194
--
Manage your subscription for the Freeipa-devel
URL: https://github.com/freeipa/freeipa/pull/472
Author: tiran
Title: #472: Packaging: Add placeholder packages
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/472/head:pr472
git checkout pr472
From
URL: https://github.com/freeipa/freeipa/pull/499
Author: tscherf
Title: #499: added help about default value for --external-ca-type option
Action: opened
PR body:
"""
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa
URL: https://github.com/freeipa/freeipa/pull/497
Author: tscherf
Title: #497: added more meaningful help for --external-ca-type option
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/497/head:pr497
git checkout
URL: https://github.com/freeipa/freeipa/pull/498
Author: HonzaCholasta
Title: #498: compat: fix `Any` params in `batch` and `dnsrecord`
Action: opened
PR body:
"""
The `methods` argument of `batch` and `dnsrecords` attribute of `dnsrecord`
were incorrectly defined as `Str` instead of `Any`.
URL: https://github.com/freeipa/freeipa/pull/497
Title: #497: added more meaningful help for --external-ca-type option
MartinBasti commented:
"""
This is already fixed in master, however it missed what is the default value
```
--external-ca-type={generic,ms-cs}
Type
URL: https://github.com/freeipa/freeipa/pull/497
Title: #497: added more meaningful help for --external-ca-type option
MartinBasti commented:
"""
This is already fixed in master, however it missed what is the default value
```
--external-ca-type={generic,ms-cs}
Type
URL: https://github.com/freeipa/freeipa/pull/497
Author: tscherf
Title: #497: added more meaningful help for --external-ca-type option
Action: opened
PR body:
"""
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa
URL: https://github.com/freeipa/freeipa/pull/484
Title: #484: FIPS: Remove pkispawn cruft
Label: +ack
--
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
URL: https://github.com/freeipa/freeipa/pull/484
Title: #484: FIPS: Remove pkispawn cruft
tomaskrizek commented:
"""
Works as expected.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/484#issuecomment-281979229
--
Manage your subscription for the Freeipa-devel mailing
Hi,
for a while make causes unsolicited modifications to all translation
files. I have to reset all PO files a couple of times a day during
development:
git checkout -- po/*.po
It's slowly wearing me off. I opened ticket
https://fedorahosted.org/freeipa/ticket/6605 a while ago. It contains
URL: https://github.com/freeipa/freeipa/pull/476
Author: HonzaCholasta
Title: #476: vault: cache the transport certificate on client
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/476/head:pr476
git
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
MartinBasti commented:
"""
The old pylint issue is back
```
* Module ipaserver.install.installutils
ipaserver/install/installutils.py:1209: [E1101(no-member), store_version]
Module
On 23.02.2017 12:31, Martin Kosek wrote:
Hi all,
Based on my recent Contributors.txt update and on Martin Basti's request in the
pull request:
https://github.com/freeipa/freeipa/pull/493#issuecomment-281938080
I added my (hacky) script for updating the file in the freeipa-tools repo and
Hi all,
Based on my recent Contributors.txt update and on Martin Basti's request in the
pull request:
https://github.com/freeipa/freeipa/pull/493#issuecomment-281938080
I added my (hacky) script for updating the file in the freeipa-tools repo and
updated our Release page:
URL: https://github.com/freeipa/freeipa/pull/492
Author: HonzaCholasta
Title: #492: [WIP] config: remove meaningless defaults
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/492/head:pr492
git checkout
URL: https://github.com/freeipa/freeipa/pull/367
Title: #367: Remove nsslib from IPA
stlaz commented:
"""
Some more fixes for Travis to check.
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/367#issuecomment-281950085
--
Manage your subscription for the Freeipa-devel
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
tiran commented:
"""
@MartinBasti I dropped the last commit. make check no longer checks wheel
packages.
I'm going to open a new ticket for @martbab and ask him to add to add a proper
test for
URL: https://github.com/freeipa/freeipa/pull/472
Author: tiran
Title: #472: Packaging: Add placeholder packages
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/472/head:pr472
git checkout pr472
From
URL: https://github.com/freeipa/freeipa/pull/397
Author: tiran
Title: #397: Improve wheel building and provide ipaserver wheel for local
testing
Action: synchronized
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
MartinBasti commented:
"""
@tiran I thought we agreed on having `with_wheels` in specfile and install
dependencies only when you want to build wheel packages, what is not the case
of RHEL. So
On 20.02.2017 20:24, Alexander Bokovoy wrote:
> On la, 18 helmi 2017, Timo Aaltonen wrote:
>>
>> Hi,
>>
>> So Fedora puts all of dist-packages/ipaserver/* in python-ipaserver,
>> but dcerpc.py imports python-samba which -ipaserver does not depend on.
>> So I've kept dcerpc.py and
URL: https://github.com/freeipa/freeipa/pull/398
Title: #398: Support for Certificate Identity Mapping
sumit-bose commented:
"""
Ok, sorry for the noise, I tested on a fresh install again and now it is
working as expected. I guess I shouldn't have tried to update from an older
version of your
URL: https://github.com/freeipa/freeipa/pull/493
Title: #493: Update Contributors.txt
MartinBasti commented:
"""
If script for generating contributors file was published it may be documented
in releasing page and it may be part of each release.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/493
Title: #493: Update Contributors.txt
Label: +pushed
--
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
URL: https://github.com/freeipa/freeipa/pull/493
Title: #493: Update Contributors.txt
MartinBasti commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/b367c3a622c8c8b96777e4ce50334d2a4477bbe7
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/493
Author: mkosek
Title: #493: Update Contributors.txt
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/493/head:pr493
git checkout pr493
--
Manage your subscription
URL: https://github.com/freeipa/freeipa/pull/495
Title: #495: Fix ipa-server-upgrade
MartinBasti commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/32076df10231b381a80c9ef850c2c31d7a25feb8
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/496
Author: stlaz
Title: #496: Use newer Certificate.serial_number in krainstance.py
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/496/head:pr496
git checkout pr496
URL: https://github.com/freeipa/freeipa/pull/495
Author: stlaz
Title: #495: Fix ipa-server-upgrade
Action: closed
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/495/head:pr495
git checkout pr495
--
Manage your subscription
URL: https://github.com/freeipa/freeipa/pull/495
Title: #495: Fix ipa-server-upgrade
Label: +pushed
--
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
URL: https://github.com/freeipa/freeipa/pull/496
Title: #496: Use newer Certificate.serial_number in krainstance.py
MartinBasti commented:
"""
Fixed upstream
master:
https://fedorahosted.org/freeipa/changeset/8c2cd66269d7be976ae7fc990343ed8e9b5282a3
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/496
Title: #496: Use newer Certificate.serial_number in krainstance.py
Label: +pushed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/439
Title: #439: [WIP] [Py3] testing both py2/py3 in travis
Label: +postponed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/460
Title: #460: [Py3] ipa-server-install, ipa-server-upgrade fixes
Label: +postponed
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
lslebodn commented:
"""
NACK for downstream patch. The intentin of build system refactoring was make
packaging in downstream simpler.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
lslebodn commented:
"""
NACK for downstream patch. The intentin of build system refactoring was make
packaging in downstream simpler.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
pvoborni commented:
"""
Some distros like RHEL doesn't have python-wheel packaged. It can be disabled
by downstream patch, but better would be to remove it or make it configurable.
"""
See the
URL: https://github.com/freeipa/freeipa/pull/493
Title: #493: Update Contributors.txt
Label: +ack
--
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
URL: https://github.com/freeipa/freeipa/pull/493
Title: #493: Update Contributors.txt
stlaz commented:
"""
I don't give two poops and a popsicle about the order of names in the
Contributors.txt file. ACK.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
lslebodn commented:
"""
FYI: At the moment, `make check` run just C-based unit test and all of them are
optional.
If required dependency is not found at configure time then test is not
URL: https://github.com/freeipa/freeipa/pull/397
Title: #397: Improve wheel building and provide ipaserver wheel for local
testing
tiran commented:
"""
I didn't know about the ```SkipPluginModule``` feature. I agree with you, your
solution is more elegant.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/412
Title: #412: Define template version in certmap.conf
flo-renaud commented:
"""
Bump for review
"""
See the full comment at
https://github.com/freeipa/freeipa/pull/412#issuecomment-281931336
--
Manage your subscription for the Freeipa-devel
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
tiran commented:
"""
That's not the point here. We are arguing about a new build dependency
(python-wheel).
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
lslebodn commented:
"""
I agree with @MartinBasti it is not a unit test.
IMHO better approach is to test it in CI/travis/...
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/496
Title: #496: Use newer Certificate.serial_number in krainstance.py
Label: +ack
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA:
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
tiran commented:
"""
I'm open to suggestions here, but I like to have automatic validation of
packaging.
"""
See the full comment at
URL: https://github.com/freeipa/freeipa/pull/496
Title: #496: Use newer Certificate.serial_number in krainstance.py
flo-renaud commented:
"""
Hi @stlaz ,
the warning
`/usr/lib/python2.7/site-packages/ipaserver/install/krainstance.py:316:
DeprecationWarning: Certificate serial is deprecated,
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
MartinBasti commented:
"""
Is really needed to have this is make check? It makes build dependencies of
wheel mandratory not optional
```
error: invalid command 'bdist_wheel'
make[5]: ***
URL: https://github.com/freeipa/freeipa/pull/472
Title: #472: Packaging: Add placeholder packages
MartinBasti commented:
"""
Is really needed to have this is make check? It makes build dependencies of
wheel mandratory not optional
```
error: invalid command 'bdist_wheel'
make[5]: ***
97 matches
Mail list logo