Re: [Freeipa-devel] [freeipa PR#184][comment] Minor install script fixes

2016-10-24 Thread Alexander Bokovoy

On ma, 24 loka 2016, Simo Sorce wrote:

On Mon, 2016-10-24 at 17:38 +0200, abbra wrote:

  URL: https://github.com/freeipa/freeipa/pull/184
Title: #184: Minor install script fixes

abbra commented:
"""
ACK from my side if you would split the commit into two small ones, please.

Note that CI integration is currently broken so travis says your commits failed 
the checks.
"""


Done, and the CI seem happy ?

Yes, thank you. I acked the request.

--
/ Alexander Bokovoy

--
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


Re: [Freeipa-devel] [freeipa PR#184][comment] Minor install script fixes

2016-10-24 Thread Simo Sorce
On Mon, 2016-10-24 at 17:38 +0200, abbra wrote:
>   URL: https://github.com/freeipa/freeipa/pull/184
> Title: #184: Minor install script fixes
> 
> abbra commented:
> """
> ACK from my side if you would split the commit into two small ones, please.
> 
> Note that CI integration is currently broken so travis says your commits 
> failed the checks.
> """

Done, and the CI seem happy ?

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York

-- 
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