| 02:09.29 | *** part/#asterisk-dev mjordan (~mjordan@user-69-1-6-49.knology.net) |
| 06:38.52 | *** join/#asterisk-dev irroot (~gregory@41.79.221.226) |
| 06:39.38 | *** part/#asterisk-dev irroot (~gregory@41.79.221.226) |
| 07:57.46 | *** join/#asterisk-dev hehol (~hehol@2001:1438:1009:200:39d4:fbbf:85ff:b6ff) |
| 08:39.29 | *** join/#asterisk-dev tzafrir_laptop (~tzafrir@local.xorcom.com) |
| 08:39.29 | *** mode/#asterisk-dev [+o tzafrir_laptop] by ChanServ |
| 10:26.57 | *** join/#asterisk-dev kpfleming (~kpfleming@asterisk/digium-director-of-software-tech/kpfleming) |
| 10:26.58 | *** mode/#asterisk-dev [+o kpfleming] by ChanServ |
| 12:07.35 | *** join/#asterisk-dev pnlarsson (~niklas@fw1.gml.g.icnet.infracom.se) |
| 12:46.44 | *** join/#asterisk-dev mjordan (~mjordan@nat/digium/x-dfqzijiwyqkghdsn) |
| 12:46.44 | *** mode/#asterisk-dev [+o mjordan] by ChanServ |
| 13:19.28 | *** join/#asterisk-dev pabelanger (~pabelange@asterisk/contributor-and-bug-marshal/pabelanger) |
| 13:19.28 | *** mode/#asterisk-dev [+o pabelanger] by ChanServ |
| 13:20.53 | *** join/#asterisk-dev putnopvut (~putnopvut@asterisk/master-of-queues/mmichelson) |
| 13:20.53 | *** mode/#asterisk-dev [+o putnopvut] by ChanServ |
| 13:23.25 | *** join/#asterisk-dev serafie (~erin@nat/digium/x-uyhfzpsfkiohwzhm) |
| 13:53.55 | *** join/#asterisk-dev newtonr (~newtonr@nat/digium/x-hldmlmrbmzqiykbp) |
| 13:57.13 | *** join/#asterisk-dev anonymouz666 (~anonymouz@187-28-37-118.poolip.RJO.embratel.net.br) |
| 13:58.34 | *** join/#asterisk-dev sruffell (~sruffell@asterisk/the-kernel-guy/sruffell) |
| 13:58.34 | *** mode/#asterisk-dev [+o sruffell] by ChanServ |
| 14:01.45 | *** join/#asterisk-dev zerohalo (~zerohalo@74-94-141-204-ma.hfc.comcastbusiness.net) |
| 14:05.22 | *** join/#asterisk-dev kpfleming (~kpfleming@asterisk/digium-director-of-software-tech/kpfleming) |
| 14:05.23 | *** mode/#asterisk-dev [+o kpfleming] by ChanServ |
| 14:22.05 | *** join/#asterisk-dev sruffell (~sruffell@asterisk/the-kernel-guy/sruffell) |
| 14:22.05 | *** mode/#asterisk-dev [+o sruffell] by ChanServ |
| 14:25.16 | *** join/#asterisk-dev sruffell (~sruffell@asterisk/the-kernel-guy/sruffell) |
| 14:25.16 | *** mode/#asterisk-dev [+o sruffell] by ChanServ |
| 14:37.04 | *** join/#asterisk-dev italorossi (~italoross@201.76.151.254) |
| 14:40.52 | italorossi | Hello, someone has information about this problem? (problem with ringinuse=no with queues) https://issues.asterisk.org/jira/browse/ASTERISK-16115 |
| 14:42.47 | mjordan | italorossi: #asterisk-bugs |
| 14:43.42 | italorossi | mjordan: sorry, thanks! |
| 15:47.55 | *** join/#asterisk-dev sgimeno (~sgimeno@163.117.206.10) |
| 16:34.12 | *** join/#asterisk-dev rgagnon (~rgagnon@rrcs-71-42-183-54.sw.biz.rr.com) |
| 16:34.15 | *** part/#asterisk-dev rgagnon (~rgagnon@rrcs-71-42-183-54.sw.biz.rr.com) |
| 16:42.50 | *** join/#asterisk-dev anthm (~anthm@freeswitch/developer/anthm) |
| 16:49.36 | *** join/#asterisk-dev irroot (~gregory@197.106.242.64) |
| 16:50.07 | *** part/#asterisk-dev irroot (~gregory@197.106.242.64) |
| 16:55.52 | *** join/#asterisk-dev tzafrir_laptop (~tzafrir@local.xorcom.com) |
| 16:55.52 | *** mode/#asterisk-dev [+o tzafrir_laptop] by ChanServ |
| 17:18.52 | *** join/#asterisk-dev newtonr (~newtonr@nat/digium/x-vlstwjaaoiooyfus) |
| 17:19.14 | *** join/#asterisk-dev datachomper (~russ@nat/digium/x-xijurjwiskysssni) |
| 17:19.51 | *** join/#asterisk-dev kenhunt (~kenhunt@nat/digium/x-tybltiwyozvprclm) |
| 18:16.56 | *** join/#asterisk-dev sustav (~vpp@nat/digium/x-sbiosxospnjlxebg) |
| 19:02.11 | putnopvut | Your case sensitivity comment about channel variables is intriguing. |
| 19:02.14 | putnopvut | mjordan: ^^ |
| 19:02.51 | putnopvut | pbx_builtin_getvar_helper() does use case sensitivity when checking channel variable names, so this confuses me. |
| 19:07.58 | mjordan | putnopvut: I'm confused as well. I was sure that it wasn't jumping to that priority ... but the function evaluated to true |
| 19:08.26 | mjordan | given that the voip-info article was referencing stuff pre-1.2, I would normally be inclined to disregard it ... but... yeah. |
| 19:16.10 | putnopvut | Okay, I've found where that's occurring. It's in ast_str_retrieve_variables() in main/pbx.c. It checks for certain builtin variables (case-sensitively) and if none of those matches, then it does a case-insensitive search of channel variables. |
| 19:17.13 | mjordan | interesting |
| 19:17.33 | putnopvut | So for variable subsitution in the dialplan, channel variables are case-insensitive, but builtin variables are case sensitive. That means that if you try to retrieve ${CHANNEL} then that's specifically the builtin variable that gets channel name. |
| 19:17.51 | putnopvut | But ${channel} and ${ChAnNeL} are the same thing since they are not ${CHANNEL} |
| 19:18.19 | opticron | wow... |
| 19:18.25 | putnopvut | But on the other hand ${EGGS} and ${eGGs} are the same since there is no builtin ${EGGS} variable. |
| 19:18.58 | mjordan | that is not the behavior I would have expected |
| 19:19.05 | putnopvut | Similarly, like I had said before, pbx_builtin_getvar_helper() does case-sensitive matching. |
| 19:19.21 | putnopvut | So that could lead to some "fun" debugging. |
| 19:19.32 | putnopvut | Someone sets MiXMONITOR_FILENAME instead of MIXMONITOR_FILENAME |
| 19:19.43 | putnopvut | The result is that app_mixmonitor can't get the proper filename. |
| 19:19.54 | kpfleming | yay for creating our own programming language! |
| 19:19.55 | putnopvut | But if you check the value of MIXMONITOR_FILENAME in the dialplan, it will come back with what you set it to. |
| 19:19.56 | mjordan | yup. Which will often work when that variable is accessed through the normal dialplan execution |
| 19:20.06 | mjordan | but not if its accessed from within an application :-( |
| 19:20.17 | putnopvut | Correct |
| 19:20.48 | mjordan | we should make it case sensitive. Sneak it into a 1.8 release and not document the change. |
| 19:20.50 | mjordan | :-D |
| 19:20.54 | putnopvut | ha! Great plan |
| 19:21.05 | mjordan | maybe even change the commit message to be "Ignore this commit" with no further comments. |
| 19:21.16 | putnopvut | "Changed some stuff" |
| 19:21.24 | putnopvut | Or the favorite "Nothing to see here" |
| 19:21.32 | mjordan | except for... panic. |
| 19:21.58 | mjordan | either way, this probably should be documented on the wiki. |
| 19:22.05 | russellb | thinks it was intended to be case sensitive the whole time anyway |
| 19:22.45 | mjordan | new commit message: "Clarified intent." |
| 19:22.50 | russellb | there you go |
| 19:23.30 | putnopvut | Yeah, going back to 1.2, this is how it has always been, apparently. |
| 19:24.07 | jsmith | That's my understanding as well |
| 19:25.25 | russellb | commit message: "make code suck less" |
| 19:25.51 | leifmadsen | I always assumed it was case sensitive.... |
| 19:25.57 | russellb | same |
| 19:26.17 | leifmadsen | I'd consider that a bug fix |
| 19:26.21 | putnopvut | Well the answer is that it is sometimes :) |
| 19:26.29 | leifmadsen | putnopvut: right :) |
| 19:26.52 | leifmadsen | anyways, I'd be ok if someone decided it was a bug and wanted it in 1.8 |
| 19:27.01 | russellb | i say we have quorum :-p |
| 19:27.18 | mjordan | does not look forward to the JIRA bug reports |
| 19:27.29 | leifmadsen | I'd actually be surprised if there were a lot |
| 19:27.30 | putnopvut | predicts about 502304293 issue reports opening after that change |
| 19:27.39 | russellb | yeah, that's when leifmadsen and I will conveniently not be around |
| 19:27.45 | leifmadsen | PTO |
| 19:27.47 | mjordan | there's bound to be a number of subtle bugs caused by that |
| 19:27.50 | mjordan | har har |
| 19:27.53 | leifmadsen | possibly |
| 19:27.59 | putnopvut | The most common type of bug I see reported is "It worked in X version, now it doesn't" |
| 19:28.03 | leifmadsen | could add a config option to asterisk.conf ? |
| 19:28.07 | putnopvut | And that's what we'd be seeing for a while after that change. |
| 19:28.10 | mjordan | leifmadsen: was just thinking that as well |
| 19:28.11 | leifmadsen | then remove it for trunk |
| 19:28.13 | leifmadsen | or whatever |
| 19:28.13 | russellb | ugh please not more of that crap |
| 19:28.20 | putnopvut | Yeah, I don't like the asterisk.conf option |
| 19:28.25 | russellb | 5 trillion micro tweak options |
| 19:28.27 | russellb | drowns |
| 19:28.34 | putnopvut | Just change it in trunk and make a note in UPGRADE.txt |
| 19:28.38 | leifmadsen | well you just leave it "as-is" unless you need the fixed version |
| 19:28.58 | leifmadsen | it not being available in 1.8 though is kind of annoying as I'd expected it to be case sensitive |
| 19:28.59 | mjordan | then again... newtonr is doing most of the ASTERISK triage... so. Patch for 1.8 with vague commit message it is! |
| 19:29.04 | leifmadsen | it might even be documented like that in A:TDG |
| 19:29.07 | newtonr | mjordan: lol |
| 19:29.27 | leifmadsen | so you can point at the docs to say it was intended to work like this! :) |
| 19:29.28 | mjordan | how about we at least file this as a wtf bug |
| 19:29.39 | newtonr | mjordan: i'll make an awesome canned response for that |
| 19:29.48 | kpfleming | we should add an asterisk.conf 'compat' setting for the asterisk 0.2 compiled-in dialplan mode |
| 19:29.57 | leifmadsen | post a bug, write something to -dev, fix in 1.8, and add something to UPGRADE.txt |
| 19:30.00 | leifmadsen | which everyone should be checking anyways |
| 19:30.23 | leifmadsen | then fix it in 1.8 |
| 19:30.26 | mjordan | Changes since 1.8.15: stuff. |
| 19:30.30 | leifmadsen | I've seen that |
| 19:30.34 | leifmadsen | it's not unprecidented |
| 19:30.45 | leifmadsen | unprecedented |
| 19:30.51 | file | depreciated! |
| 19:30.56 | leifmadsen | file: hush! |
| 19:47.29 | *** part/#asterisk-dev newtonr (~newtonr@nat/digium/x-vlstwjaaoiooyfus) |
| 20:18.58 | *** join/#asterisk-dev sustav_ (~vpp@nat/digium/x-raybhoyuegoieoku) |
| 20:34.06 | *** join/#asterisk-dev tzafrir_laptop (~tzafrir@bzq-218-155-146.cablep.bezeqint.net) |
| 20:34.06 | *** mode/#asterisk-dev [+o tzafrir_laptop] by ChanServ |
| 20:46.36 | *** join/#asterisk-dev newtonr (~newtonr@173-25-121-44.client.mchsi.com) |
| 20:47.45 | *** mode/#asterisk-dev [+o newtonr] by ChanServ |
| 20:48.26 | *** join/#asterisk-dev Micc (~Micc@50.125.113.34) |
| 20:50.55 | Micc | I don't think parking should be in the asterisk code base. |
| 20:51.20 | Micc | I think it would make the code much nicer if it was just removed completely and a dialplan based parking was created. |
| 20:51.23 | russellb | hi2u2 |
| 20:51.37 | Micc | hello |
| 20:51.56 | Qwell | Micc: oh, if only we could do things like that |
| 20:52.09 | kpfleming | how much code is actually affected by the presence of parking? not much. |
| 20:52.45 | Micc | @kpfleming, not much, but there is some scary stuff in there that touches a lot of areas that it has no business being. |
| 20:53.12 | Micc | Its overly complex for something that can be done with some dialplan and a database. |
| 20:53.35 | kpfleming | then do it... nobody's stopping you :-) |
| 20:54.19 | Micc | well, I don't know enough to remove it, but I'd be willing to donate the dialplan code to mimic and even do one better with one touch park. |
| 20:54.35 | kpfleming | sounds good |
| 20:55.17 | Micc | I suppose removing it wouldn't need to be done right away. But if there was a dialplan alternative, then over time the parking code could be removed and phased out. |
| 20:55.42 | kpfleming | exactly |
| 20:55.46 | Micc | that parking code was the source of so many crashes for me. |
| 20:56.01 | putnopvut | There are a lot of things that are in Asterisk that fall into the same category. Voicemail for instance. |
| 20:56.18 | opticron | shudders |
| 20:56.24 | Qwell | Can we reimplement masquerades in dialplan? |
| 20:56.30 | Micc | hehe, I still use voicemail. |
| 20:56.39 | kpfleming | can we write dialplan in bash? |
| 20:56.48 | putnopvut | bash agi? |
| 20:56.56 | opticron | BAGI! |
| 21:00.00 | russellb | put bash AGI in A:TDG |
| 21:00.19 | mjordan | noooooooo |
| 21:00.36 | russellb | http://asteriskdocs.org/en/3rd_Edition/asterisk-book-html-chunk/AGI-quickstart.html |
| 21:00.42 | russellb | yeah, i totally did |
| 21:01.14 | Micc | I suppose I should put something up on voip-info, or is there a better place to publish my dialplan parking? |
| 21:02.31 | kpfleming | wiki.asterisk.org |
| 21:02.50 | kpfleming | or you can put it in JIRA and it can be added to the 'contrib' directory of Asterisk |
| 21:11.23 | *** join/#asterisk-dev michael-i (~anonymous@204.11.231.178.static.etheric.net) |
| 21:14.16 | leifmadsen | some of the agent stuff was removed in favour of a dialplan based method, so that has been done before. If it happens again though, we should make sure it is thoroughly documented so people have a drop in replacement |
| 21:16.19 | kpfleming | good grief, the func_shell documentation was awful. |
| 21:17.20 | mjordan | "command - This is the argument to the function, the command you want to pass to the shell." |
| 21:17.27 | kpfleming | yeah |
| 21:17.30 | mjordan | oof. |
| 21:17.34 | kpfleming | self-referential documentation |
| 21:18.05 | mjordan | Did you see Andrew's comment on the wiki page? |
| 21:18.11 | kpfleming | no |
| 21:18.13 | mjordan | https://wiki.asterisk.org/wiki/display/AST/Function_SHELL |
| 21:18.25 | kpfleming | nice |
| 21:18.29 | kpfleming | not any more! |
| 21:21.56 | michael-i | After the dummy channel is populated with custom fields in cdr_sqlite3_custom, I'm trying to read a value from one of those custom fields. Is pbx_builtin_getvar_helper the right way to do that? |
| 21:23.30 | kpfleming | if you use it to call the CDR() dialplan function, yes |
| 21:23.44 | kpfleming | i am going to 'review' tzafrir_laptop's codec_ilbc patch by committing it |
| 21:25.33 | michael-i | kpfleming: that is how I'm setting the value, yes…but I need to record certain records in a secondary table directly from the cdr driver. Determining if this is one of those records is my goal. |
| 21:25.43 | kpfleming | that's tricky |
| 21:25.49 | michael-i | (i know i'm in over my head here...) |
| 21:26.07 | Corydon76-work | kpfleming: func_odbc already supports returning multiple values from a single function... right? |
| 21:26.21 | kpfleming | i don't know, i've never looked at it or used it |
| 21:26.52 | Corydon76-work | Ask Qwell. He's done some radically awful things with it |
| 21:26.59 | michael-i | i can also solve it by adding post-insert trigger on the sqlite db itself but that would also requires a read-newest select |
| 21:27.40 | Qwell | do what now? |
| 21:27.52 | Corydon76-work | Qwell: you and func_odbc |
| 21:28.00 | Qwell | many years ago |
| 21:28.12 | Qwell | I've long since purged all of that information. |
| 21:28.26 | Corydon76-work | Qwell: start stroking your long gray beard |
| 21:28.49 | Qwell | It was like 40 programmeryears ago. |
| 21:29.43 | Corydon76-work | Anyway, since func_odbc supports returning multiple values from a function, and since we're trying to solve the same problem with a system-like dialplan function, might as well use the same mechanism |
| 21:29.59 | kpfleming | if only we had made Set() take a list of variable names to be set. Set((foo,bar)=SHELL(xyz)) |
| 21:30.20 | Corydon76-work | Set(ARRAY(foo,bar)=${SHELL(xyz)}) |
| 21:30.28 | kpfleming | ahh |
| 21:30.44 | kpfleming | similar, i guess |
| 21:30.55 | Corydon76-work | although for backwards compatibility, it can't be SHELL(), because it already exists |
| 21:30.56 | kpfleming | ShellEx(), anyone? |
| 21:31.49 | Corydon76-work | Sounds fine to me |
| 21:32.13 | Corydon76-work | all caps, for obvious reasons |
| 21:32.57 | Corydon76-work | Or, actually... that's one of those conventions we might think about breaking |
| 21:33.32 | Corydon76-work | There's nothing in func_odbc, for example, that prevents you from using mixed case dialplan functions... |
| 21:33.52 | Corydon76-work | although I honestly don't know anybody who has tried it |
| 21:34.36 | Corydon76-work | The vim syntax highlighter might not like it, but that's not stopped anyone before |
| 21:34.50 | kpfleming | i can't come up with a better name, which bothers me |
| 21:35.00 | Corydon76-work | Well, there's SYSTEM() |
| 21:35.20 | kpfleming | that would just make things even more confusing |
| 21:35.28 | Corydon76-work | How about EXEC() ? |
| 21:35.57 | kpfleming | so much similarity is going to result in people not being able to decide what to use. it's not a good situation. |
| 21:36.21 | Corydon76-work | You could deprecate SHELL() at the same time |
| 21:37.07 | kpfleming | exec() really does something very different though |
| 21:37.17 | kpfleming | at least in C land |
| 21:37.19 | Corydon76-work | The dialplan application, yes |
| 21:37.28 | kpfleming | there too |
| 21:37.57 | Corydon76-work | FORKANDEXEC() |
| 21:38.22 | kpfleming | i wonder if there's a practical way to give SHELL() an optional first argument without breaking things. doubtful. |
| 21:38.22 | Corydon76-work | Now I'm just being silly |
| 21:38.39 | Corydon76-work | Well, there's what we did for the flag day in 1.6 |
| 21:39.04 | Corydon76-work | [compat] app_shell=11 in asterisk.conf |
| 21:39.18 | kpfleming | this isn't really worth that much work though |
| 21:39.28 | kpfleming | it's hardly worth this much discussion |
| 21:39.53 | Corydon76-work | The name definitely needs its own bike shed |
| 21:40.36 | Corydon76-work | If we're agreed that ARRAY() is the way to go, the name is really the last thing that needs to be decided |
| 21:40.41 | kpfleming | yep |
| 21:41.13 | Corydon76-work | Do you prefer a flag day solution, or do you prefer a different name? |
| 21:41.29 | Corydon76-work | SHELLEX() is fine by me |
| 21:41.30 | kpfleming | i think our policies at this point strongly prefer a different name |
| 21:43.43 | *** join/#asterisk-dev luckman212 (~irc@2001:470:8abb:0:e424:e4c:eef4:a291) |
| 21:45.29 | leifmadsen | SKRILLEX() ? |
| 21:45.51 | kpfleming | i knew that was coming |
| 21:45.56 | leifmadsen | :) |
| 21:46.02 | leifmadsen | it was obligatory |
| 21:46.23 | russellb | Returns "WUBWUBWUBWUBWUBWUBWUBW...SQUEEEEEEEE...WUBWUBWUBWUB" regardless of arguments |
| 21:46.28 | leifmadsen | lol |
| 21:46.33 | leifmadsen | runs away |
| 21:46.40 | Qwell | russellb: Audiohook function. Do it. |
| 21:46.49 | kpfleming | nice |
| 21:46.50 | russellb | that sounds like effort |
| 21:47.14 | leifmadsen | and we all know what we think of effort |
| 21:47.25 | russellb | absolutely |
| 21:47.29 | russellb | leifmadsen: EFF it |
| 21:47.39 | russellb | see what i did there? |
| 21:47.53 | leifmadsen | Electronic Freedom Foundation? |
| 21:47.57 | russellb | sure |
| 21:48.23 | kpfleming | russellb: working from the new digs? |
| 21:48.32 | russellb | yup |
| 21:48.39 | kpfleming | have the dogs gone into the lake yet? |
| 21:48.44 | russellb | nope :-) |
| 21:48.49 | russellb | fence prevents them from doing so |
| 21:48.52 | kpfleming | lame |
| 21:48.53 | russellb | ... for now, until they dig out |
| 21:49.12 | russellb | one of the dogs barked up a storm at a fake duck that is anchored in the pond right behind our house. |
| 21:49.14 | russellb | i lol'd. |
| 21:49.29 | kpfleming | i think i'm going to do SHELLEXEC() and deprecate SHELL() |
| 21:49.49 | russellb | i think bkruse wrote func_shell way back in the day |
| 21:49.52 | kpfleming | he did |
| 21:49.59 | kpfleming | and he wrote terrible docs (or someone did) |
| 21:50.09 | kpfleming | and whoever was minding the show back then didn't correct them :-) |
| 21:50.38 | michael-i | kpfleming: ast_cdr_getvar got me what I needed…mission accomplished |
| 21:50.38 | russellb | trololol |
| 21:50.47 | kpfleming | michael-i: nice |
| 21:51.08 | michael-i | now i can go safely meander back to php+shell |
| 22:30.39 | *** join/#asterisk-dev italorossi (~italoross@187.111.231.124) |
| 22:41.31 | *** part/#asterisk-dev mjordan (~mjordan@nat/digium/x-dfqzijiwyqkghdsn) |
| 22:44.46 | *** join/#asterisk-dev Juggie (~Juggie@unaffiliated/juggie) |
| 22:57.42 | *** join/#asterisk-dev zerohalo (~zerohalo@74.61.196.236) |
| 23:02.46 | *** join/#asterisk-dev italorossi (~italoross@zezo.digi.com.br) |
| 23:14.16 | *** join/#asterisk-dev Juggie (~Juggie@unaffiliated/juggie) |
| 23:29.18 | *** join/#asterisk-dev Juggie (~Juggie@unaffiliated/juggie) |