Re: [devel] [PATCH 0 of 4] Review Request for imm: add support for long DN [#886]

2014-07-23 Thread Neelakanta Reddy
Hi zoran, Forgot to export SA_ENABLE_EXTENDED_NAMES=1. after exporting new immomtest and new immoitest are working fine. Its good to add SA_ENABLE_EXTENDED_NAMES=1 to immnd.conf Ack for patch 4. Thanks, Neel. On Wednesday 23 July 2014 08:19 PM, Zoran Milinkovic wrote: > Hi Neelakanta, > > The

Re: [devel] [PATCH 0 of 4] Review Request for imm: add support for long DN [#886]

2014-07-23 Thread Zoran Milinkovic
Hi Neelakanta, The reason you are getting these errors is that you didn't enable extended names on server side. You must add to immnd.conf: export SA_ENABLE_EXTENDED_NAMES=1 I'll add this line to immnd.conf. Best regards, Zoran -Original Message- From: Neelakanta Reddy [mailto:reddy.n

Re: [devel] [PATCH 0 of 4] Review Request for imm: add support for long DN [#886]

2014-07-23 Thread Neelakanta Reddy
Hi zoran, Reviewed and tested the patch. have the same comments as AndersBj. Regarding the patch 4 "add test cases for long DN" Ack for the first three patches. But for patch 4: Ack, when immomtest and immoitest is fixed. The test cases are not running: I enabled longdDns, Following are the

[devel] [PATCH 0 of 4] Review Request for imm: add support for long DN [#886]

2014-07-14 Thread Zoran Milinkovic
Summary: imm: add support for long DN [#886] Review request for Trac Ticket(s): 886 Peer Reviewer(s): Neelakanta, Anders Pull request to: Zoran Affected branch(es): default(4.5) Development branch: default(4.5) Impacted area Impact y/n ---