[OpenSER-Devel] [ openser-Patches-1707527 ] add dbtext support via openserctl

[SourceForge.net]

Patches item #1707527, was opened at 2007-04-25 16:01
Message generated for change (Comment added) made by henningw
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1707527&group_id=139143

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: tools
Group: ver devel
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Ovidiu Sas (osas)
Assigned to: Nobody/Anonymous (nobody)
Summary: add dbtext support via openserctl

Initial Comment:
Hi,

This patch integrates dbtext support into the openserctl script:
 - openserctl.patch - patch for openserctl script
 - openserctl.dbtext - new file to be installed

This aims to add proper support for dbtext out of the box and hopefully better maintenance for dbtext.

Todo: fix the Makefile to properly install the new openserctl.dbtext file (not included in the patch)

----------------------------------------------------------------------

Comment By: Henning Westerholt (henningw)
Date: 2007-07-05 08:52

Message:
Logged In: YES 
user_id=337916
Originator: NO

Hello Ovidiu,

i've some questions regarding your patch for openserctl:

In the domain() part, you output the message "..trying to force OpenSER to
reload it" but the necessary fifo_cmd domain_reload is not implemented.
Should i remove this part of user information? 

Or perhaps we can use the same approach like the other engines in this
part, output msg: "..use domain reload.."? Does the command "$CTLCMD
domain_reload" work for dbtext?

The same issue exist in the lcr() part of the patch. 
What do you think?

Henning

----------------------------------------------------------------------

Comment By: Ovidiu Sas (osas)
Date: 2007-05-24 14:48

Message:
Logged In: YES 
user_id=1395524
Originator: YES

Hi Henning,

Please use the patch to update the openserctl.  I forgot that I updated
the openserctl in the pactch.

----------------------------------------------------------------------

Comment By: Henning Westerholt (henningw)
Date: 2007-05-24 08:30

Message:
Logged In: YES 
user_id=337916
Originator: NO

Hello Osas,

i have not looked into this database openserctl specific stuff yet, i
think i can use at least parts of your patch.

Perhaps Bogdan confuses this patch with your previous tracker entry
"sc.dbtext is not up to date for location/alias", this is allready fixed by
the new schema generation stuff. 

Henning

----------------------------------------------------------------------

Comment By: Ovidiu Sas (osas)
Date: 2007-05-23 18:11

Message:
Logged In: YES 
user_id=1395524
Originator: YES

This patch is no longer needed (in light of Henning Westerholt effort to
integrate all databases under the xml schema)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=1707527&group_id=139143