| 00:00.24 | file | aha I found your master pbx_find_extension deadlock issue |
| 00:02.32 | file | there, 2 dupes closed |
| 00:03.41 | file | mjordan, hahahahahahahahahahahahaha |
| 00:03.49 | file | omg |
| 00:03.53 | file | ASTERISK-19091 |
| 00:04.38 | mjordan | yup |
| 00:04.46 | mjordan | ironic, ain't it |
| 00:04.50 | file | isn't that what kharwell is working on right now? |
| 00:04.53 | mjordan | yup |
| 00:05.00 | mjordan | it's ... a small world ... after all |
| 00:07.32 | mjordan | there's a part of me that wants to merge a lot of patches. |
| 00:07.40 | mjordan | and just live with the consequences. |
| 00:08.01 | file | do you keep that part of you at bay with HGTV? |
| 00:08.30 | mjordan | L&O: SVU right now. |
| 00:08.40 | mjordan | Katie's favorite show. |
| 00:08.50 | file | oic |
| 00:08.52 | mjordan | easy doc fix on ASTERISK-19470 |
| 00:11.47 | file | closes a T.38 issue |
| 00:16.59 | mjordan | yay |
| 00:17.01 | mjordan | which one? |
| 00:17.13 | file | ASTERISK-19013 |
| 00:18.06 | file | goes to brush teeth, brb |
| 00:20.10 | mjordan | hm. |
| 00:20.28 | mjordan | I'm curious why they thought it worked in 1.10... |
| 00:20.31 | mjordan | which turned into 10 |
| 00:23.58 | file | blind luck? |
| 00:29.17 | mjordan | I'll bet we fixed ASTERISK-17283 |
| 00:29.43 | file | agreed |
| 00:29.52 | file | thoughts on ASTERISK-17742? |
| 00:31.01 | mjordan | probably similar to my problems in 13 |
| 00:31.06 | mjordan | just different :-P |
| 00:31.08 | file | ha |
| 00:31.21 | mjordan | I am curious why it didn't transcode |
| 00:31.28 | file | indeed |
| 00:31.39 | mjordan | there are logs and conf files there :-P |
| 00:31.49 | mjordan | I'm going to mock up ASTERISK-17283 |
| 00:31.56 | mjordan | if nothing else, it'd be nice to know if it still is broken |
| 00:32.00 | file | it's not a full debug log |
| 00:42.37 | mjordan | huh. |
| 00:42.46 | mjordan | the redirect one is weird. It said it redirected it, then didn't. |
| 00:53.13 | mjordan | ah, no frames. |
| 00:53.19 | mjordan | with frames, things work. YAY. |
| 00:53.27 | mjordan | I think ASTERISK-17742 is probably not a bug |
| 00:53.31 | mjordan | but that's just me |
| 00:53.35 | mjordan | or at least, no where near enough info |
| 00:53.55 | file | nods |
| 00:55.38 | mjordan | I'll bet ASTERISK-19501 is no longer an issue |
| 00:56.51 | mjordan | goes to look |
| 00:58.29 | mjordan | er |
| 00:58.31 | mjordan | maybe not. |
| 00:58.51 | mjordan | masquerades would break the crap out of GROUP. It stores the channel pointer and expects them to be the same. |
| 00:59.04 | mjordan | when they are set, and when the channel is destroyed, that is. |
| 00:59.54 | mjordan | actually should be easy to fix... but... MASQUERADES. |
| 01:00.07 | file | smirks |
| 01:01.41 | mjordan | what do you think of the patch on https://issues.asterisk.org/jira/browse/ASTERISK-21765 ? |
| 01:04.37 | file | um |
| 01:04.39 | file | huh |
| 01:05.35 | mjordan | TIL: SPYGROUP is a thing |
| 01:05.47 | file | it.. seems.. sane |
| 01:07.05 | mjordan | if you don't account for the offset, you'll end up some where funky |
| 01:09.14 | mjordan | I'll test and commit if it fixes Rusty's sample |
| 01:12.08 | file | ASTERISK-21038 is cool |
| 01:15.11 | mjordan | works for me |
| 01:15.32 | mjordan | uh |
| 01:15.38 | mjordan | wtf? |
| 01:16.04 | mjordan | wtf. |
| 01:16.20 | mjordan | that's the registered files, i'll bet. |
| 01:16.24 | mjordan | what I don't know is why. |
| 01:16.29 | file | quite |
| 01:21.18 | file | omg |
| 01:21.30 | file | it's because of core set debug |
| 01:22.10 | mjordan | huh |
| 01:22.13 | mjordan | that's weird |
| 01:23.40 | mjordan | alas, it wouldn't surprise me if https://issues.asterisk.org/jira/browse/ASTERISK-16516 was still real |
| 01:25.17 | file | well, I have a fix |
| 01:26.02 | file | - } else if ((a->pos == 4 && !atleast && strcasecmp(argv3, "off")) |
| 01:26.02 | file | + } else if ((a->pos == 4 && !atleast && strcasecmp(argv3, "off") && strcasecmp(argv3, "channel")) |
| 01:27.28 | file | no joke MAN |
| 01:33.57 | mjordan | yay |
| 01:33.59 | mjordan | SHIP IT |
| 01:34.36 | file | working on it but SOMEONE did commits before me |
| 01:36.05 | mjordan | I'm probably going to ship a patch to app_sms. |
| 01:36.07 | mjordan | Because why not. |
| 01:37.18 | mjordan | good lord it is only 8:30 |
| 01:37.32 | mjordan | we should do this once a week. |
| 01:37.41 | mjordan | or maybe once every two weeks. |
| 01:37.46 | file | GAH |
| 01:37.53 | file | TOO MANY TABS OPEN |
| 01:38.02 | mjordan | you're not john bigelow |
| 01:38.18 | mjordan | ever seen that guy's browser? Holy crap. The tabs only have the width of a single pixel. |
| 01:38.26 | mjordan | and yet, he knows where *everything* is |
| 01:38.44 | file | yes, yes I have |
| 01:40.58 | file | there, fixed |
| 01:41.46 | mjordan | might have to talk to Rusty about getting a 'hit list' of easy to fix issues. |
| 01:41.46 | file | I'm tempted to finally fix ASTERISK-18708 |
| 01:42.08 | mjordan | of course, if we move to git, hopefully these things get fixed easier... |
| 01:42.10 | mjordan | hopes |
| 01:42.28 | file | I think my gmail is being stupid |
| 01:42.36 | file | checks |
| 01:42.37 | mjordan | er |
| 01:42.44 | mjordan | hm. |
| 01:43.08 | file | yeah, it is, I'm missing my own commit emails |
| 01:43.10 | file | cute |
| 01:43.54 | mjordan | yes |
| 01:44.04 | file | and yes - I agree on making this a regular thing |
| 01:44.06 | mjordan | curl_easy_setopt should be dup'ing the strings |
| 01:44.27 | mjordan | you know what afternoons are pointless? the ones after sprint planning. |
| 01:44.31 | mjordan | Maybe we just do this after that. |
| 01:44.43 | file | your afternoon is my evening |
| 01:44.47 | file | lawl :P |
| 01:44.52 | mjordan | er |
| 01:44.54 | mjordan | well, true |
| 01:45.00 | mjordan | but right now is definitely your evening |
| 01:45.05 | file | truth |
| 01:45.16 | mjordan | most of sprinting day is kind of a wash, since the meetings sort of disrupt the day |
| 01:45.21 | mjordan | we could do this around the meeting |
| 01:45.27 | file | yup |
| 01:46.14 | mjordan | I'd say make the change - hashcompat is an int (assigned a cast as a long? WTF?) and curl_easy_setopt *should* copy strings |
| 01:46.21 | mjordan | er |
| 01:46.24 | mjordan | except those aren't strings. |
| 01:46.25 | mjordan | HM> |
| 01:47.53 | file | hum |
| 01:47.59 | mjordan | hm. |
| 01:48.04 | mjordan | strings are dup'd by the library. |
| 01:48.07 | file | yes |
| 01:48.16 | mjordan | http://curl.haxx.se/libcurl/c/curl_easy_setopt.html |
| 01:49.07 | file | um |
| 01:49.23 | file | you know, there is an alternative... |
| 01:49.25 | file | rwlock |
| 01:50.23 | file | although wait what |
| 01:50.25 | file | reads code |
| 01:50.50 | file | ah |
| 01:53.45 | mjordan | I actually think it is okay |
| 01:53.59 | mjordan | since the file pointers and such things are not being provided by the global curl stuff |
| 01:54.13 | mjordan | er, but even then, it'd probably be okay... |
| 01:54.18 | mjordan | I'd say risk it ;-) |
| 01:54.22 | file | yeah |
| 01:57.48 | mjordan | bbiab |
| 01:58.49 | file | I'll probably be ASLEEP |
| 02:03.45 | mjordan | go then! go to sleep |
| 02:03.57 | file | pfft |
| 02:05.39 | mjordan | .....and today I learned about regexten |
| 02:05.42 | file | I'm finding more stuff |
| 02:05.48 | mjordan | that's actually not a terrible idea. |
| 02:06.37 | file | ALERT: Dog fart. |
| 02:07.26 | mjordan | lol |
| 02:07.33 | mjordan | ASTERISK-19553 - probably not a bug |
| 02:08.34 | file | want an easy one? ASTERISK-16779 |
| 02:09.34 | mjordan | yeah, that one is too annoying. Masquerades again. |
| 02:10.58 | mjordan | well... are you saying ASTERISK-16779 is not a bug? |
| 02:11.15 | file | no |
| 02:11.18 | file | well |
| 02:11.21 | file | it's an annoyance |
| 02:11.39 | mjordan | it is an annoyance |
| 02:11.53 | mjordan | and we could just do a !ast_strlen_zero on the value |
| 02:11.59 | file | yes |
| 02:12.03 | mjordan | would have been nice if they provided a patch. |
| 02:12.18 | mjordan | allow=all |
| 02:12.18 | mjordan | disallow= |
| 02:12.18 | mjordan | Should never be allowed in static conf files... |
| 02:12.31 | file | theirs came from realtime |
| 02:12.35 | mjordan | yeah |
| 02:12.46 | mjordan | but since the parsing is in the same place, we would hide a problem |
| 02:12.50 | mjordan | not sure if I care though |
| 02:13.28 | mjordan | the annoying part though is that forces you to use the disallow column (if it is in your schema), when generally using allow is all that is needed |
| 02:15.04 | mjordan | and of course, I really should do it in every channel driver |
| 02:15.06 | mjordan | EASY YOU SAID |
| 02:15.19 | file | I did |
| 02:15.25 | file | UNTIL YOU MADE IT HARD, WITH YOUR LOGIC |
| 02:16.30 | mjordan | http://pastebin.com/QMpBVsi4 |
| 02:17.07 | file | wth there was a gun in this music video and they blurred it out |
| 02:18.25 | file | why not for allow as well? |
| 02:19.49 | mjordan | because not providing a value for allow should always be an error |
| 02:20.04 | mjordan | that is, disallow=all with no allow is an error |
| 02:20.30 | mjordan | and disallow=ulaw,alaw and no allow=all is an error |
| 02:20.35 | mjordan | er |
| 02:20.42 | file | true |
| 02:20.50 | mjordan | nuts. Actually, you could be stupid and say disallow=all,!ulaw,!alaw |
| 02:21.05 | file | there are many ways to shoot yourself in the foot |
| 02:21.16 | mjordan | yeah |
| 02:21.24 | mjordan | I'd say this is the safest way that does alleviate a valid config |
| 02:21.27 | mjordan | that is, no disallow |
| 02:21.31 | mjordan | and allow=!all,ulaw,g722 |
| 02:22.05 | mjordan | I may end up posting a 'can't reproduce, try on a supported branch' for ASTERISK-19779 |
| 02:22.10 | mjordan | because that backtrace is silly |
| 02:22.26 | mjordan | seg faulting on locking a valid mutex... with DEBUG_THREADS enabled. |
| 02:22.28 | mjordan | uh. |
| 02:22.53 | file | oh that, yeah |
| 02:26.31 | file | geez @ ASTERISK-17402 |
| 02:26.56 | file | no error foo you |
| 02:29.43 | mjordan | OBE |
| 02:29.48 | mjordan | MeetMe. That exists? :-P |
| 02:29.55 | file | it does not |
| 02:30.17 | mjordan | you'd think that would be easy to fix ... |
| 02:31.33 | file | ASTERISK-14181 came up recently in #freepbx - Grandstream added their own state to convey offline |
| 02:32.27 | mjordan | yeah, I saw that |
| 02:32.35 | mjordan | but that really needs (yet another) config option |
| 02:32.40 | file | yes |
| 02:32.45 | mjordan | it'd be trivial, but we don't want to convey terminated to everyone |
| 02:32.46 | file | I'm not saying to touch it :P |
| 02:32.48 | mjordan | wtf Grandstream |
| 02:33.04 | mjordan | somewhere, malcolmd felt a disturbance in the force |
| 02:33.09 | file | ha |
| 02:33.14 | mjordan | ya know, since they are using Asterisk for their PBX, they could push a patch upstream |
| 02:33.25 | mjordan | so yeah.... I'm going to let that one sit. |
| 02:34.13 | file | diving... whatever it may take I keep on trying... |
| 02:36.02 | mjordan | I know that song... |
| 02:37.24 | file | okay, I've gone through my filter |
| 02:37.25 | file | I'M DONE |
| 02:38.40 | file | on that bombshell as the dust settles in JIRA I'm off to bed, cheers |
| 02:41.55 | mjordan | https://issues.asterisk.org/jira/browse/ASTERISK-17944 |
| 02:41.57 | mjordan | damn! |
| 02:41.59 | mjordan | oh well. |
| 02:43.29 | file | what |
| 02:43.31 | file | :P |
| 02:49.19 | file | ew 340p video |
| 02:52.09 | mjordan | yeah, I'm done as well. The MeetMe conference issue isn't terribly hard, but not enough that I want to test it |
| 02:53.20 | file | I don't even have dahdi on my stuff |
| 02:56.17 | mjordan | once we get things on git |
| 02:56.31 | mjordan | I'm going to hit every improvement/new feature/patch issue and put a remark asking for review |
| 02:56.33 | mjordan | wait two months |
| 02:56.40 | mjordan | maybe three |
| 02:56.44 | mjordan | and then suspend the rest. |
| 02:56.58 | file | oic |
| 03:48.57 | *** join/#asterisk-bugs darkbasic (~quassel@niko.linuxsystems.it) |
| 13:19.51 | *** join/#asterisk-bugs mjordan (mjordan@nat/digium/x-qfmtwwkvvvhksnwe) |
| 13:19.51 | *** mode/#asterisk-bugs [+o mjordan] by ChanServ |
| 14:58.43 | file | mjordan, hi |
| 14:58.49 | mjordan | hello again |
| 14:59.02 | file | it's day #2 of issue tracker attack |
| 14:59.53 | mjordan | I may throw a few patches up |
| 15:00.00 | mjordan | there are a few that are in pretty good shape |
| 15:01.07 | mjordan | ASTERISK-24780 for example |
| 15:01.16 | file | mjordan, I closed a great T.38 one earlier - a provider sending a re-invite before the initial INVITE was completed |
| 15:01.21 | mjordan | excellent! |
| 15:01.23 | mjordan | I saw that :-) |
| 15:01.39 | file | ah that one |
| 15:01.51 | file | yup yup |
| 15:02.10 | file | why does ASTERISK-17856 sound familiar? |
| 15:02.58 | mjordan | er |
| 15:03.02 | mjordan | didn't we do that patch? |
| 15:03.13 | file | we did something there... |
| 15:03.34 | mjordan | or is it just that ASTERISK-21894 got some traffic... |
| 15:04.51 | file | ASTERISK-20559 might have been what I was thinking of |
| 15:05.39 | mjordan | potentially |
| 15:05.53 | mjordan | that's rather old |
| 15:06.03 | mjordan | ASTERISK-20360 is funny |
| 15:06.26 | file | I think I'll close ASTERISK-17856 out in favor of ASTERISK-21894 then, thoughts? |
| 15:07.01 | file | oh that's fun |
| 15:07.35 | mjordan | works for me |
| 15:10.54 | file | erm |
| 15:10.58 | file | ASTERISK-18703 |
| 15:11.26 | mjordan | wha |
| 15:12.04 | mjordan | not a ... proxy |
| 15:17.34 | file | yeah I'm closing that out |
| 15:22.40 | file | ASTERISK-17790 |
| 15:26.23 | mjordan | uh |
| 15:26.25 | file | mjordan, you should fix new pattern matching |
| 15:27.17 | mjordan | lawl |
| 15:27.20 | mjordan | aka, remove it? |
| 15:27.26 | file | yes. |
| 15:27.33 | mjordan | yes, we should. |
| 15:27.44 | mjordan | I'd say ASTERISK-17790 should go |
| 15:28.11 | file | agreed |
| 15:29.44 | mjordan | I wish I'd gotten the config for ASTERISK-20983 |
| 15:30.08 | file | dream the dream |
| 15:30.24 | file | wow, there's really not a lot in that issue at all |
| 15:33.51 | file | ASTERISK-20599 reporter never got back |
| 15:37.47 | file | ugh this wifi is so overloaded |
| 15:38.20 | mjordan | hm |
| 15:38.33 | mjordan | I'm not sure how I feel about that |
| 15:38.40 | mjordan | it may not be a patch, but it is a completely new module |
| 15:39.04 | mjordan | honestly, wouldn't take much to clean it up, throw it in extended. |
| 15:39.14 | mjordan | otoh, if he's already checked out, do we really want it in extended? |
| 15:39.17 | mjordan | conundrum that is |
| 15:39.54 | file | BURN IT |
| 15:42.15 | mjordan | ASTERISK-19794 |
| 15:42.20 | mjordan | pretty sure rmudgett fixed that |
| 15:42.28 | mjordan | and on that issue, I have to go grocery shopping |
| 15:42.50 | file | don't buy more JIRA issues |
| 15:43.52 | mjordan | ASTERISK-15973 is suspect. BUT I SHALL NOT LOOK AT IT RIGHT NOW |
| 15:43.57 | mjordan | be back in awhile! |
| 15:44.36 | file | KK |
| 16:38.57 | file | wanders home |