10:33 a.m.
I think the other enumeration disadvantages are documented well enough
in the manpage. I'm also thinking we might just say clear that in large
environments, enumeration should be discouraged. Thoughts?
5:20 a.m.
On 01/25/2013 05:33 PM, Jakub Hrozek wrote:
I think the other enumeration disadvantages are documented well
enough
in the manpage. I'm also thinking we might just say clear that in large
environments, enumeration should be discouraged. Thoughts?
+1
The patch itself looks good. But can you amend the commit message to
include link to the ticket please?
7:28 a.m.
On Mon, Jan 28, 2013 at 12:20:22PM +0100, Pavel Březina wrote:
On 01/25/2013 05:33 PM, Jakub Hrozek wrote:
>I think the other enumeration disadvantages are documented well enough
>in the manpage. I'm also thinking we might just say clear that in large
>environments, enumeration should be discouraged. Thoughts?
+1
The patch itself looks good. But can you amend the commit message to
include link to the ticket please?
OK, new patch is attached.
7:36 a.m.
On 01/28/2013 02:28 PM, Jakub Hrozek wrote:
On Mon, Jan 28, 2013 at 12:20:22PM +0100, Pavel Březina wrote:
> On 01/25/2013 05:33 PM, Jakub Hrozek wrote:
>> I think the other enumeration disadvantages are documented well enough
>> in the manpage. I'm also thinking we might just say clear that in large
>> environments, enumeration should be discouraged. Thoughts?
>
> +1
>
> The patch itself looks good. But can you amend the commit message to
> include link to the ticket please?
OK, new patch is attached.
Ack.
8:56 a.m.
On Mon, Jan 28, 2013 at 02:36:23PM +0100, Pavel Březina wrote:
On 01/28/2013 02:28 PM, Jakub Hrozek wrote:
>On Mon, Jan 28, 2013 at 12:20:22PM +0100, Pavel Březina wrote:
>>On 01/25/2013 05:33 PM, Jakub Hrozek wrote:
>>>I think the other enumeration disadvantages are documented well enough
>>>in the manpage. I'm also thinking we might just say clear that in large
>>>environments, enumeration should be discouraged. Thoughts?
>>
>>+1
>>
>>The patch itself looks good. But can you amend the commit message to
>>include link to the ticket please?
>
>OK, new patch is attached.
Ack.
Pushed to master and sssd-1-9
4115
days inactive
4118
days old
sssd-devel@lists.fedorahosted.org
4 comments
2 participants
participants (2)
-
Jakub Hrozek -
Pavel Březina