[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-08-25 Thread lslebodn 
  URL: https://github.com/SSSD/sssd/pull/211
Title: #211: IFP: Fix of limit = 0 (unlimited result)

lslebodn commented:
"""
master:
* b0b9222f7dd62b19ec702afe295ec71624888e87
* 5d855b5d546eb995023d80d61433bbe91888dbdf
* 3c31ce392ad9da4ac7c3d8190db89efcdbbc8b85
* 5fe1e8ba91a1e2e95aadf94ecc5148bec804aa5a

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/211#issuecomment-325010705
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-08-25 Thread fidencio 
  URL: https://github.com/SSSD/sssd/pull/211
Title: #211: IFP: Fix of limit = 0 (unlimited result)

fidencio commented:
"""
Added "Accepted" label per @pbrezina's review.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/211#issuecomment-324901151
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-08-25 Thread pbrezina 
  URL: https://github.com/SSSD/sssd/pull/211
Title: #211: IFP: Fix of limit = 0 (unlimited result)

pbrezina commented:
"""
Ack. I opened: https://github.com/SSSD/sssd/pull/356 for a typo in man pages.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/211#issuecomment-324879332
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-07-31 Thread fidencio 
  URL: https://github.com/SSSD/sssd/pull/211
Title: #211: IFP: Fix of limit = 0 (unlimited result)

fidencio commented:
"""
@pbrezina, could you take a look on this patchset, please?

I've reviewed @celestian's patches, but ended up adding two patches on top of 
his, so I can't ACK my own patches :-)
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/211#issuecomment-319084394
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-07-26 Thread fidencio 
  URL: https://github.com/SSSD/sssd/pull/211
Title: #211: IFP: Fix of limit = 0 (unlimited result)

fidencio commented:
"""
retest this, please
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/211#issuecomment-318154121
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-05-09 Thread fidencio 
  URL: https://github.com/SSSD/sssd/pull/211
Title: #211: IFP: Fix of limit = 0 (unlimited result)

fidencio commented:
"""
And there's one extra patch from me that's rebased on top of this series that 
ended up being pushed together (and, IMO, it's okay as it has to rebased on top 
of these patches anyways).
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/211#issuecomment-300142263
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-05-09 Thread fidencio 
  URL: https://github.com/SSSD/sssd/pull/211
Title: #211: IFP: Fix of limit = 0 (unlimited result)

fidencio commented:
"""
I've updated @celestian's patchset and added one patch on top of his.

His patch looks okay to go, but I'll need a review on mine.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/211#issuecomment-300141692
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-03-31 Thread lslebodn 
  URL: https://github.com/SSSD/sssd/pull/211
Title: #211: IFP: Fix of limit = 0 (unlimited result)

lslebodn commented:
"""
retest this please
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/211#issuecomment-290705474
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-03-28 Thread celestian 
  URL: https://github.com/SSSD/sssd/pull/211
Title: #211: IFP: Fix of limit = 0 (unlimited result)

celestian commented:
"""
@lslebodn thanks, that's good point. I synced new version of the patch.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/211#issuecomment-289727876
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#211][comment] IFP: Fix of limit = 0 (unlimited result)

2017-03-28 Thread lslebodn 
  URL: https://github.com/SSSD/sssd/pull/211
Title: #211: IFP: Fix of limit = 0 (unlimited result)

lslebodn commented:
"""
I didn't check the code. But would it be possible to hadle it
inside function `ifp_list_ctx_remaining_capacity` ?

Becasue newly added dbus call can have the same problem.
And IMHO it's better to solve problem on one place.
I hope it's possible.

LS

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/211#issuecomment-289691209
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org