[prev in list] [next in list] [prev in thread] [next in thread] 

List:       orocos-dev
Subject:    [Orocos-Dev] [Bug 538] Unable specify log level when starting deployer
From:       peter.soetens () fmtc ! be (Peter Soetens)
Date:       2008-05-31 0:47:55
Message-ID: E1K2DHX-0000uX-N6 () www ! fmtc ! be
[Download RAW message or body]

For more infomation about this bug, visit <https://www.fmtc.be/bugzilla/orocos/show_bug.cgi?id=538>



--- Comment #14 from Peter Soetens <peter.soetens@fmtc.be>  2008-05-31 00:41:08 ---
(In reply to comment #13)
> Created an attachment (id=301)
 --> (https://www.fmtc.be/bugzilla/orocos/attachment.cgi?id=301) [details]
> Refined patch
> 
> Cleaner patch. More capable use of boost program options (now returns variable
> map to caller)
> 
> Consolidates cdeployer and deployer-corba into cdeployer (removes need for
> deployer-corba). 

I'm still thinking about this solution. It's one program less, but the default
behavior of cdeployer changes from 'daemon' (no console) to console based.
That breaks some scripts probably... On the other hand, I like the fact that
only the server side needs an extra option such that 'console' usage causes
less type work (a sane default).

> 
> Adds option to cdeployer to either run corba server (ie original behaviour), or
> run taskbrowser with corba (ala deployer-corba).
> 
> Adds option to turn off console logging

This is not a 'perfect' implementation. I'd prefer to have a method that tells
the logger to disable console logging, then to redirect it to a stringstream...
We could check for 'stdoutput' not being null or an extra flag in
maylogStdOut().

> 
> NB the code to monkey with argc/argv is still not pretty ... :-(
> 

I'm not sure that you can do it better... Does it matter for TAO what argv[0]
contains ? I would just have passed to InitOrb(argc - taoIndex, argv[taoIndex]
)...and drop taoArgc and taoArgv calculations. Just a guess.

Peter

-- 
Configure bugmail: https://www.fmtc.be/bugzilla/orocos/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic