Bug 3566

Summary: ORBGestalt local together with ORBSvcConfDirective fails
Product: TAO Reporter: Johnny Willemsen <jwillemsen>
Component: ORBAssignee: DOC Center Support List (internal) <tao-support>
Status: NEW ---    
Severity: normal CC: iliyan, vladimir.zykov
Priority: P3    
Version: 1.6.8   
Hardware: All   
OS: Windows NT   
Bug Depends on:    
Bug Blocks: 3565    

Description Johnny Willemsen 2009-02-06 06:45:12 CST
ORBGestalt local together with ORBSvcConfDirective fails
Comment 1 Johnny Willemsen 2009-02-06 06:48:14 CST
regression test is pending for after x.6.8
Comment 2 Johnny Willemsen 2009-02-11 04:07:38 CST
test is in svn
Comment 3 Johnny Willemsen 2009-02-12 02:56:11 CST
Added Iliyan. Regression test is on svn head and fails now there
Comment 4 Johnny Willemsen 2010-03-17 05:02:36 CDT
Vladimir, is this one related to your recent svc.conf changes?
Comment 5 Vladimir Zykov 2010-03-17 06:02:10 CDT
(In reply to comment #4)
> Vladimir, is this one related to your recent svc.conf changes?
> 

Yes, this one is related but the test itself is a bit wrong. I'll check it.
Comment 6 Johnny Willemsen 2010-03-17 06:05:26 CDT
(In reply to comment #5)
> (In reply to comment #4)
> > Vladimir, is this one related to your recent svc.conf changes?
> > 
> 
> Yes, this one is related but the test itself is a bit wrong. I'll check it.
> 

Thanks, 3565 is also related, that test in the repository needs an update also
Comment 7 Vladimir Zykov 2010-03-17 07:40:42 CDT
(In reply to comment #6)
> Thanks, 3565 is also related, that test in the repository needs an update also
> 

I don't see a difference between tests for 3565 and 3566. I think the following code in 3566 is wrong and needs to be fixed.

second_extra[2] = ACE::strnew (ACE_TEXT ("-ORBSvcConfDirective \"static Client_Strategy_Factory '-ORBReplyDispatcherTableSize 666'\""));

Each command-line option has to be a separate element in argv array. So, -ORBSvcConfDirective and \"static Client_Strategy_Factory '-ORBReplyDispatcherTableSize 666'\" has to be 2 elements. Once this is fixed 3565 and 3566 will become equal.
Comment 8 Johnny Willemsen 2010-03-17 07:51:41 CDT
it is a long time we made this test, for 3565, the test in the repository needs an update when it gets fixed, see the bugzilla issue. the quoting in this test should be ok so far as I remember, without -ORBGestalt local, the quoting is not a problem
Comment 9 Vladimir Zykov 2010-03-17 08:18:55 CDT
(In reply to comment #8)
> it is a long time we made this test, for 3565, the test in the repository needs
> an update when it gets fixed, see the bugzilla issue. the quoting in this test
> should be ok so far as I remember, without -ORBGestalt local, the quoting is
> not a problem
> 

Without -ORBGestalt local I see the same error as with it.

(18552|140737353955088) EXCEPTION, Exception caught:
system exception, ID 'IDL:omg.org/CORBA/BAD_PARAM:1.0'
TAO exception, minor code = 11 (ORB Core initialization failed; EINVAL), completed = NO
Comment 10 Johnny Willemsen 2010-03-17 10:14:46 CDT
I can't remember the details, but on the commandline I thought you could quote the directive in such a way that it is just one argument. Not sure about the details, as far as I remember the quoting was not maintained in some places. Feel free to update the test with 2 arrays and than mark this and 3565 as fixed, 
Comment 11 Johnny Willemsen 2010-03-17 14:04:33 CDT
I think the test itself is fine, I will build 1.6.8 on a system to see what I got there