| 00:16.04 | *** join/#asterisk-dev dseeb (~Adium@CPE-120-146-252-149.static.vic.bigpond.net.au) |
| 02:41.19 | *** join/#asterisk-dev Juggie (~Juggie@unaffiliated/juggie) |
| 03:35.18 | *** join/#asterisk-dev seraphie (~erin@75.76.38.159) |
| 04:59.14 | *** join/#asterisk-dev philipp64|laptop (~chatzilla@50-39-153-143.bvtn.or.frontiernet.net) |
| 05:12.24 | *** join/#asterisk-dev mickecarlsson (~Micke@h10n3c1o1101.bredband.skanova.com) |
| 05:13.41 | *** join/#asterisk-dev jsmith-away (~jsmith@fedora/jsmith) |
| 05:13.41 | *** mode/#asterisk-dev [+o jsmith-away] by zelazny.freenode.net |
| 05:14.43 | *** join/#asterisk-dev tzafrir (~tzafrir@212.179.75.202) |
| 05:14.52 | *** mode/#asterisk-dev [+o tzafrir] by ChanServ |
| 06:37.10 | *** join/#asterisk-dev irroot (~irroot@pbx.distrotech.co.za) |
| 06:53.03 | *** join/#asterisk-dev schmidts (~schmidts@Lo20-rt01.lm33.sil.at) |
| 06:53.05 | schmidts | good morning |
| 07:09.34 | *** join/#asterisk-dev hehol (~hehol@2001:1438:1009:200:89d4:def7:67d5:eef2) |
| 07:30.46 | *** join/#asterisk-dev tzafrir_laptop (~tzafrir@local.xorcom.com) |
| 07:30.46 | *** mode/#asterisk-dev [+o tzafrir_laptop] by ChanServ |
| 07:33.48 | *** join/#asterisk-dev tamiel (~tamiel@213.30.183.226) |
| 09:16.47 | *** join/#asterisk-dev kaii (~kh@ciphron.de) |
| 09:19.56 | *** join/#asterisk-dev sgimeno (~sgimeno@163.117.206.10) |
| 11:04.02 | *** join/#asterisk-dev kpfleming (~kpfleming@asterisk/digium-director-of-software-tech/kpfleming) |
| 11:04.02 | *** mode/#asterisk-dev [+o kpfleming] by ChanServ |
| 11:15.05 | *** join/#asterisk-dev Marquis42 (~bfhbmw0@65-127-126-34.dia.static.qwest.net) |
| 11:25.11 | *** join/#asterisk-dev oej (~olle@static-213-115-251-100.sme.bredbandsbolaget.se) |
| 11:25.11 | *** mode/#asterisk-dev [+o oej] by ChanServ |
| 11:54.33 | kpfleming | oej: svn.asterisk.org/svn.digium.com should now be in sync again |
| 12:02.33 | irroot | thx kpfleming helps with my brain strain |
| 12:15.00 | *** join/#asterisk-dev leifmadsen (~Leif@asterisk/documenteur-extraordinaire/blitzrage) |
| 12:15.00 | *** mode/#asterisk-dev [+o leifmadsen] by ChanServ |
| 13:05.21 | *** join/#asterisk-dev serafie (~erin@nat/digium/x-ugwarhxbwiigsqcy) |
| 13:07.09 | *** join/#asterisk-dev oej (~olle@static-213-115-251-100.sme.bredbandsbolaget.se) |
| 13:07.09 | *** mode/#asterisk-dev [+o oej] by ChanServ |
| 13:17.10 | oej | kpfleming: THanks and good morning! |
| 13:18.26 | kpfleming | you are welcome |
| 13:18.53 | kpfleming | still digesting all the emails received over the weekend... will be a long morning |
| 13:19.29 | anonymouz666 | are there many? |
| 13:19.33 | anonymouz666 | :) |
| 13:24.28 | kpfleming | not as many as i had feared... maybe the constant rain has washed some away |
| 13:37.58 | leifmadsen | kpfleming: +1 |
| 13:38.31 | oej | Constant rain will give you more water to canoe in, kpfleming. |
| 13:39.27 | leifmadsen | prefers the kayak |
| 13:41.34 | leifmadsen | although I did get a chance to get out on the canoe this weekend at the cottage! |
| 13:42.24 | kpfleming | the rivers around here are all flooded now, it will take a couple of days before they are safe to paddle on again |
| 13:42.36 | kpfleming | but they tend to drain out pretty quickly too |
| 13:46.40 | *** join/#asterisk-dev moy (~moy@173.239.155.74) |
| 13:58.54 | *** join/#asterisk-dev sruffell (~Adium@asterisk/the-kernel-guy/sruffell) |
| 13:58.54 | *** mode/#asterisk-dev [+o sruffell] by ChanServ |
| 14:02.51 | oej | I really wish that I could fix compilation on OS/X Lion. But I can't. |
| 14:07.48 | *** join/#asterisk-dev irroot (~irroot@196-215-2-62.dynamic.isadsl.co.za) |
| 14:08.33 | *** join/#asterisk-dev jeffg (~jeffg@pdpc/supporter/professional/jeffg) |
| 14:19.54 | *** join/#asterisk-dev otherwiseguy (~terry@cpe-69-76-217-40.kc.res.rr.com) |
| 14:20.45 | *** join/#asterisk-dev otherwiseguy (~terry@cpe-69-76-217-40.kc.res.rr.com) |
| 14:27.36 | *** join/#asterisk-dev putnopvut (~putnopvut@asterisk/master-of-queues/mmichelson) |
| 14:27.36 | *** mode/#asterisk-dev [+o putnopvut] by ChanServ |
| 14:29.39 | *** join/#asterisk-dev kpfleming (~kpfleming@asterisk/digium-director-of-software-tech/kpfleming) |
| 14:29.54 | *** mode/#asterisk-dev [+o kpfleming] by ChanServ |
| 14:35.28 | Corydon76-home | oej: I understand that otherwiseguy has gotten Asterisk to compile on Lion, so you might want to ask him for hints |
| 14:37.50 | *** join/#asterisk-dev The_Boy_Wonder (~vossel@asterisk/batman-developer/dvossel) |
| 14:37.50 | *** mode/#asterisk-dev [+o The_Boy_Wonder] by ChanServ |
| 14:39.39 | *** join/#asterisk-dev Fritz09 (~Adium@pop1-3802.catv.wtnet.de) |
| 14:40.38 | oej | Thanks, Corydon76. Will do. |
| 14:43.47 | otherwiseguy | Corydon76-home: oej: I did, but managed to somehow lose the patch and don't remember what it was I did. Seems like I added CONFIG_LDFLAGS=/usr/lib/bundle1.o to makeopts then had to fix the optional api since lion doesn't hit the same defines/support the same stuff as before. |
| 14:45.46 | *** join/#asterisk-dev mjordan (~mjordan@nat/digium/x-wzihadgerynlpesr) |
| 14:45.46 | *** mode/#asterisk-dev [+o mjordan] by ChanServ |
| 14:46.02 | oej | I fixed that by replacing 10.6 with 10.7 |
| 14:46.05 | oej | in the Makefile |
| 14:46.20 | oej | The issue is something else, a compiler error on ? Alzheimer. Wait. |
| 14:46.52 | *** join/#asterisk-dev irroot (~irroot@pbx.distrotech.co.za) |
| 14:47.17 | oej | Have to enable res_adsi and res_agi for it to show up again. |
| 14:48.00 | oej | We need an "or" statement in the makefile so that we do the same for darwin 10.6 or 10.7. (or and) |
| 14:48.56 | oej | "/usr/local/src/asterisk.svn/asterisk-trunk/include/asterisk/adsi.h:200: error: only weak aliases are supported in this configuration/usr/local/src/asterisk.svn/asterisk-trunk/include/asterisk/adsi.h:200: error: only weak aliases are supported in this configuration" |
| 14:53.02 | otherwiseguy | the weak aliases thing is re: the optional api change that needs to be made. |
| 14:54.40 | kpfleming | i wonder if we should just remove the optional API stuff completely in Asterisk 10, and go back to making people have to build and load everything they are going to use |
| 14:59.24 | oej | Pls define "optional API stuff" |
| 14:59.37 | otherwiseguy | include/asterisk/optional_api.h |
| 14:59.53 | otherwiseguy | I think weak_import is no longer supported, and attribute weak must be used instead. |
| 15:01.11 | otherwiseguy | Looks like Corydon76-home already has a patch for it and I said it worked. Huh. |
| 15:01.22 | otherwiseguy | Hows that for my memory failing? |
| 15:01.27 | otherwiseguy | https://issues.asterisk.org/jira/browse/ASTERISK-17612 |
| 15:02.07 | otherwiseguy | So that would leave just the handling SOLINK for both 10.6 and 10.7 instead of just 10.6. |
| 15:02.40 | Corydon76-home | That's a fairly easy fix (Makefile magic) |
| 15:03.33 | Corydon76-home | otherwiseguy: I was waiting on the weak fix, because tzafrir said the configure test was wrong and was trying to fix it. I don't know what's become of that effort |
| 15:04.11 | Corydon76-home | I do know his initial "repair" actually broke things that work now. |
| 15:04.33 | *** join/#asterisk-dev irroot (~irroot@pbx.distrotech.co.za) |
| 15:05.02 | otherwiseguy | Corydon76-home: autoconf/makefile magic makes me think too much. I try to pawn it off on others. :-) |
| 15:12.39 | otherwiseguy | and now to go back to addressing tilghman's comments on the realtime sqlite3 review. |
| 15:12.51 | otherwiseguy | I hate sql escaping as a general concept. |
| 15:13.46 | Corydon76-home | SQL escaping, shell escaping... lots of places where delimited text needs to stay delimited |
| 15:13.47 | otherwiseguy | although, thinking over it again, just having the single escaping functions for ' and " should work as long as I ensure that those values are pre-quoted. I should be able to use %q for ' and %w for ". |
| 15:14.37 | otherwiseguy | Corydon76-home: and I still think that modifying the API to have alternating varargs of [field, op, value] would be beneficial. |
| 15:14.53 | otherwiseguy | string parsing out the op is justâ¦ugly. |
| 15:15.12 | otherwiseguy | and too sql-centric. |
| 15:15.42 | Corydon76-home | Hey, I'm not the one who came up with that syntax... I just adapted to it... |
| 15:15.59 | otherwiseguy | Corydon76-home: (I'm not sure if you ever commented on that idea as my laptop battery died and I had no scroll back from after I posted it earlier) |
| 15:16.22 | Corydon76-home | I don't think I did |
| 15:17.42 | otherwiseguy | k. basically just have the op be RT_EQ/LT/GT/LTE/GTE/LIKE, etc. and have each backend have a function that returns the appropriate string instead of having to parse for "LIKE" in res_config_ldap/curl, etc. |
| 15:17.47 | Corydon76-home | I'm not familiar with the %w syntax. Where are you seeing this in the SQLite docs? |
| 15:18.30 | otherwiseguy | sqlite3.c :-) It is used fairly heavily internally and has made it into several SQLite books. I just don't think they have the documentation completely up to snuff on it. |
| 15:18.51 | Corydon76-home | Aha |
| 15:19.26 | otherwiseguy | Basically one of those, "I can't believe that doesn't existâ¦ah, it does" kinds of things. |
| 15:19.43 | Corydon76-home | I've thought about using constants for comparison, and I didn't do it only because of backwards compatibility |
| 15:20.15 | Corydon76-home | That's also why I added the update2 API call... to avoid messing with preexisting API |
| 15:22.13 | otherwiseguy | Corydon76-home: yeah. there are lots of things I don't like about that APIâ¦for one, it is impossible to write CLI functions that aren't horrible hacks. |
| 15:22.14 | Corydon76-home | Possibly add the call as a new API and add some magic in the core to translate calls from one to the other, if the corresponding API is missing on the backend |
| 15:23.41 | otherwiseguy | I'd also like to see function "register" the "queries" the will make to allow using a cache of bound parameter queries. |
| 15:23.49 | Corydon76-home | In fact, pretty much everywhere (exactly everywhere?) that we use the LIKE call, we're simply gathering all records... the same as having no WHERE clause at all |
| 15:24.45 | otherwiseguy | Parsing the queries every time is annoying. |
| 15:24.51 | otherwiseguy | (if you don't have to) |
| 15:26.12 | Corydon76-home | Another API call is fine with me. If you want to do the work, I'll happily review it |
| 15:27.13 | Corydon76-home | _multientry is the only one that needs it... so _multientry2, perhaps? |
| 15:30.43 | otherwiseguy | Corydon76-home: Anything with a "where clause" (load, update[12], destroy) *could* use it though. I still try to think of the general case. |
| 15:31.01 | otherwiseguy | (as though we're opening it up to AMI, for example) |
| 15:31.36 | pabelanger | any issues with putting https://reviewboard.asterisk.org/r/1414/ into Asterisk 10? It disables authdebug by default now for chan_iax2 |
| 15:31.42 | Corydon76-home | That's another reason why I chose to avoid it. It's a massive new API |
| 15:31.49 | pabelanger | otherwise, it will just be trunk |
| 15:32.08 | kpfleming | pabelanger: no, go ahead |
| 15:32.09 | Corydon76-home | pabelanger: you have my vote to go ahead |
| 15:32.25 | kpfleming | really, might as well change 1.8 as well. debugging options should never default to 'enabled'. |
| 15:33.45 | Corydon76-home | Is authdebug enabled by default or just enabled in the sample config? |
| 15:33.56 | pabelanger | enabled by default |
| 15:34.10 | Corydon76-home | I'd push on the 1.8, then |
| 15:34.17 | pabelanger | roger |
| 15:34.28 | kpfleming | eh?? what does 'push on' mean? |
| 15:34.35 | Corydon76-home | Don't change it |
| 15:35.12 | Corydon76-home | As in, push back |
| 15:35.22 | kpfleming | i don't see how it is benefitting anyone to have it enabled by default, and if it can cause any harm at all, then it should be disabled |
| 15:35.49 | sruffell | tzafrir_laptop: I saw that you made a commit on the 2.5 branchâ¦that the last one you're expecting? |
| 15:36.03 | tzafrir_laptop | yes |
| 15:36.08 | Corydon76-home | Somebody's API tools may be parsing logging statements through AMI and you'll break their tools if you change the default in the middle of a release |
| 15:37.19 | kpfleming | i suppose |
| 15:37.21 | sruffell | tzafrir_laptop: ok, thanks |
| 15:37.36 | Corydon76-home | This is why we don't change defaults in the middle of a release cycle |
| 15:38.04 | Corydon76-home | Well, other than for a REALLY good reason (like a massive slowdown, or a security issue) |
| 15:38.21 | kpfleming | sure. i'd just be pretty surprised if anyone was paying attention to the authdebug-related stuff at all, but of course it could happen |
| 15:39.17 | pabelanger | Ya, that was my concert about changing it in 1.8 too |
| 15:39.29 | pabelanger | I'm happy with Asterisk 10 and forward |
| 15:39.46 | Corydon76-home | My experience suggests that the things I'd least expect people to be paying attention to are always the ones they scream loudest about. |
| 15:42.11 | *** join/#asterisk-dev bklang (~bklang@2001:470:8:5de:225:ff:fe49:f8cd) |
| 15:44.00 | irroot | pabelanger Corydon76-home kpfleming it does make sense to put it in 1.8 however its too far down the cycle of a LTS to mess with something that may be depended on unwittingly |
| 15:49.53 | pabelanger | oh ho... somebody broke the svn properties for merge10trunk |
| 15:51.28 | irroot | pabelanger i made a commit /me ducks and runs to check if it was me |
| 15:52.02 | leifmadsen | infobot: pabelanger++ |
| 15:52.11 | leifmadsen | pabelanger: you modified CHANGES, yay! :) |
| 15:52.53 | pabelanger | leifmadsen: oh snap |
| 15:53.12 | Qwell | irroot: yeah it was you :p |
| 15:53.21 | irroot | drat how the hell |
| 15:53.27 | irroot | the props are gone |
| 15:53.41 | irroot | i checked b4 commit it was right |
| 15:53.51 | irroot | any ideas ?? |
| 15:53.56 | otherwiseguy | blames leifmadsen |
| 15:54.07 | pabelanger | Qwell: what command did you use to figure it out? |
| 15:54.11 | irroot | yeah i was last |
| 15:54.16 | irroot | svn diff |less |
| 15:54.18 | Qwell | pabelanger: thunderbird |
| 15:54.34 | pabelanger | Oh, ya |
| 15:54.36 | pabelanger | just seen it |
| 15:54.41 | Qwell | but really, svn pl -v |
| 15:54.56 | irroot | did not do any pd's |
| 15:55.12 | Qwell | irroot: the script does as part of it's run, but it should replace them |
| 15:55.51 | irroot | saw it do it thats why i dbl checked |
| 15:56.31 | irroot | im reverting and redoing ?? |
| 15:57.03 | pabelanger | irroot: ya, might as well |
| 15:57.21 | Qwell | irroot: svn merge -c-334472 |
| 15:57.24 | Qwell | irroot: svn merge -c-334472 . |
| 15:57.25 | Qwell | rather |
| 15:57.34 | Qwell | that'll get you to where it was before your merge |
| 15:58.01 | irroot | sorry guys put beers on my tab |
| 16:02.47 | *** join/#asterisk-dev oej (~olle@ns.webway.se) |
| 16:02.47 | *** mode/#asterisk-dev [+o oej] by ChanServ |
| 16:03.01 | jmls | korch: you the guy messing with 0mq ? |
| 16:08.20 | pabelanger | irroot: thanks |
| 16:09.53 | irroot | pabelanger all yours ... |
| 16:17.13 | *** topic/#asterisk-dev by kpfleming -> Asterisk Development Discussion -=- http://www.asterisk.org/developers -=- Tier 2 and 3.14159265 support is in #asterisk -=- Astricon 2011 in Denver, Colorado: AstriDevCon on October 24. Be there! |
| 16:25.35 | *** join/#asterisk-dev hehol (~hehol@2001:1438:1009:200:416:47e6:73f8:aea7) |
| 16:27.19 | *** join/#asterisk-dev toddejohnson (~toddejohn@adsl-68-23-193-222.dsl.spfdil.ameritech.net) |
| 16:27.30 | *** part/#asterisk-dev toddejohnson (~toddejohn@adsl-68-23-193-222.dsl.spfdil.ameritech.net) |
| 16:36.29 | otherwiseguy | Corydon76-home: crap. %w wasn't added to sqlite3_*printf until 2007 sometime and Centos 5 uses 3.3.6 from June 2006. So, back to the drawing board. I guess I'll just have to write my own little escaping function. It is just doubling the particular quote character any time you see it, should be easy enough. |
| 17:21.21 | *** join/#asterisk-dev irroot (~irroot@197.174.72.225) |
| 17:27.37 | *** join/#asterisk-dev anthm (~anthm@freeswitch/developer/anthm) |
| 18:08.52 | kpfleming | mjordan: wow... nice work on that incomplete stuff. you fixed up all the channel drivers :-) |
| 18:10.42 | mjordan | kpfleming: thanks, I hope that I got all the right places. Its either that or I broke them all :-) |
| 18:11.05 | kpfleming | did you catch chan_ooh323 in addons? |
| 18:11.06 | irroot | mjordan i broke trunk so dont feel bad .... |
| 18:12.35 | mjordan | kpfleming: nope, I didn't do addons. Wasn't sure about them, and ooh323 was the only one that required any additions. |
| 18:12.50 | kpfleming | yep, that's the only channel driver in addons |
| 18:12.55 | mjordan | doh |
| 18:13.51 | mjordan | Bah. 1.8 has addons, trunk doesn't. |
| 18:14.12 | mjordan | er, nm, it does so |
| 18:14.17 | mjordan | I need new glasses apparently. |
| 18:28.06 | leifmadsen | :) |
| 18:29.42 | *** part/#asterisk-dev mjordan (~mjordan@nat/digium/x-wzihadgerynlpesr) |
| 18:29.58 | *** join/#asterisk-dev mjordan (~mjordan@nat/digium/x-cremkbnskdnxycyq) |
| 18:29.58 | *** mode/#asterisk-dev [+o mjordan] by ChanServ |
| 18:30.20 | *** part/#asterisk-dev mjordan (~mjordan@nat/digium/x-cremkbnskdnxycyq) |
| 18:31.03 | *** join/#asterisk-dev mjordan (~mjordan@nat/digium/x-fvhsywkiaxhybfog) |
| 18:31.03 | *** mode/#asterisk-dev [+o mjordan] by ChanServ |
| 18:34.08 | pabelanger | r1405 |
| 18:34.25 | pabelanger | misses our asterisk bots |
| 18:34.36 | pabelanger | https://reviewboard.asterisk.org/r/1405/ |
| 18:34.48 | pabelanger | if anybody running the testsuite wants to review^ |
| 18:36.03 | irroot | hey sorry about earlier ps i saw you mention me so the ping is cancled ?? |
| 19:07.26 | *** join/#asterisk-dev tzafrir_laptop (~tzafrir@bzq-218-155-170.cablep.bezeqint.net) |
| 19:07.26 | *** mode/#asterisk-dev [+o tzafrir_laptop] by ChanServ |
| 19:36.30 | otherwiseguy | Corydon76-home: So, since we allow dynamic text outside of the quoted column with the whole "key LIKE" syntax, that means that we have to handle more than just quote escaping for columns as well. |
| 19:36.39 | otherwiseguy | for instance: *CLI> realtime load sipregs "lastms IS NOT NULL;INSERT INTO sipregs (name) VALUES (\"hello!\"); select * from sipregs where lastms like" % |
| 19:37.50 | otherwiseguy | It isn't a security problem now since columns are defined in the code and never by user data (except from the CLI interface, which requires admin privileges anyway), but definitely a problem if we ever open up the api... |
| 19:39.38 | *** join/#asterisk-dev tzafrir_laptop (~tzafrir@bzq-218-155-147.cablep.bezeqint.net) |
| 19:39.38 | *** mode/#asterisk-dev [+o tzafrir_laptop] by ChanServ |
| 19:39.56 | Corydon76-home | otherwiseguy: yep, there is that problem |
| 20:30.12 | *** join/#asterisk-dev Marquis42 (~bfhbmw0@65-127-126-34.dia.static.qwest.net) |
| 20:52.44 | *** join/#asterisk-dev malcolmd (~malcolmd@pdpc/sponsor/digium/malcolmd) |
| 20:52.44 | *** mode/#asterisk-dev [+o malcolmd] by ChanServ |
| 23:01.49 | *** join/#asterisk-dev philipp64|laptop (~chatzilla@50-39-153-143.bvtn.or.frontiernet.net) |