Re: [sr-dev] [kamailio/kamailio] kamctl: enable kamctl test from shell variable. (#1977)

2019-07-29 Thread Daniel-Constantin Mierla
Closed #1977. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/1977#event-2516023186___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] kamctl: enable kamctl test from shell variable. (#1977)

2019-07-29 Thread Daniel-Constantin Mierla
I applied a slightly different patch for enabling test mode from command line, enclosing the variable in double quotes - 4968e8994157f1152bf7461b16049d67e5c9cc59 . For the bash related problems, I will open a dedicated issue on tracker to decide on a proper solution. -- You are receiving

Re: [sr-dev] [kamailio/kamailio] kamctl: enable kamctl test from shell variable. (#1977)

2019-06-27 Thread Daniel-Constantin Mierla
It seems that the issue is expanding the variable when getting the answer for y/n question: * https://github.com/kamailio/kamailio/blob/master/utils/kamctl/kamdbctl.base#L216-L218 Such expression seems to be specific for bash: * https://mywiki.wooledge.org/Bashism We should decide what to

Re: [sr-dev] [kamailio/kamailio] kamctl: enable kamctl test from shell variable. (#1977)

2019-06-26 Thread Henning Westerholt
I fear that applying 2c8fe55 will break compatibility for embedded systems again (issue #1815). Can you execute the kamdbctl create with 'bash -x' prefixed and see where exactly it stops? Maybe it is possible to just make this substitution to confirm to POSIX standard as well. -- You are

Re: [sr-dev] [kamailio/kamailio] kamctl: enable kamctl test from shell variable. (#1977)

2019-06-15 Thread Ali Shirvani
@miconda Here is the output of `kamdbctl create`: ```bash MySQL password for root: -e \E[37;33mINFO: creating database kamailio_simple_db ... -e \E[37;33mINFO: granting privileges to database kamailio_simple_db ... -e \E[37;33mINFO: creating standard tables into kamailio_simple_db ... -e

Re: [sr-dev] [kamailio/kamailio] kamctl: enable kamctl test from shell variable. (#1977)

2019-06-13 Thread Ali Shirvani
Unfortunately I have no access to that system now, I will revert the change and test again. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] kamctl: enable kamctl test from shell variable. (#1977)

2019-06-13 Thread Daniel-Constantin Mierla
Can you paste the the command you run and the entire error message printed in the terminal to try to track it properly? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] kamctl: enable kamctl test from shell variable. (#1977)

2019-06-13 Thread Ali Shirvani
@miconda It causes `Substitution Error`. I search for that error and I found that the cause is that `sh` links to `dash` in Debian Stretch. I change the `/bin/sh` to `/bin/bash` and error resolved. -- You are receiving this because you are subscribed to this thread. Reply to this email

Re: [sr-dev] [kamailio/kamailio] kamctl: enable kamctl test from shell variable. (#1977)

2019-06-13 Thread Daniel-Constantin Mierla
@alishir - what were the errors related to commit 2c8fe55? Stretch is out for long time, nobody reported the errors so far. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] kamctl: enable kamctl test from shell variable. (#1977)

2019-06-10 Thread Ali Shirvani
@alishir pushed 1 commit. 2c8fe55a2617892e129e613a5950bb8c0228f59e kamdbctl: change /bin/sh to /bin/bash -- You are receiving this because you are subscribed to this thread. View it on GitHub: