| 00:30.16 | *** join/#asterisk-bugs Marquis42 (n=bfhbmw0@65-127-126-34.dia.static.qwest.net) |
| 03:49.50 | *** join/#asterisk-bugs implicit (n=bayan@unaffiliated/implicit) |
| 03:49.50 | *** mode/#asterisk-bugs [+o implicit] by ChanServ |
| 04:12.19 | *** join/#asterisk-bugs Marquis42 (n=bfhbmw0@65-127-126-34.dia.static.qwest.net) |
| 04:43.37 | *** join/#asterisk-bugs implicit (n=bayan@unaffiliated/implicit) |
| 04:43.37 | *** mode/#asterisk-bugs [+o implicit] by ChanServ |
| 10:43.28 | *** join/#asterisk-bugs telecos (n=sergio@96.167.219.87.dynamic.jazztel.es) |
| 10:51.28 | *** join/#asterisk-bugs caio1982 (i=caio1982@CAcert-br/caio1982) |
| 11:11.41 | *** join/#asterisk-bugs guax (n=guaxinim@unaffiliated/guaxinim) |
| 13:37.16 | *** join/#asterisk-bugs lmadsen (n=Leif@asterisk/documenteur-extraordinaire/blitzrage) |
| 13:37.16 | *** mode/#asterisk-bugs [+o lmadsen] by ChanServ |
| 13:58.31 | lmadsen | morning bug hunters! |
| 14:13.14 | *** join/#asterisk-bugs sysreq (n=sysreq@unaffiliated/sysreq) |
| 14:17.22 | jsmith | lmadsen: Morning bug marshall |
| 14:31.03 | lmadsen | jsmith: morning yourself! |
| 15:02.54 | *** join/#asterisk-bugs putnopvut (n=putnopvu@nat/digium/x-6cee8008bf5380bc) |
| 15:02.54 | *** mode/#asterisk-bugs [+o putnopvut] by ChanServ |
| 15:21.35 | JunK-Y | lo |
| 15:23.37 | lmadsen | hi |
| 15:32.17 | *** join/#asterisk-bugs jpeeler (n=jpeeler@asterisk/digium-software-dev/jpeeler) |
| 15:33.18 | JunK-Y | whats up m. madsen? |
| 15:33.56 | lmadsen | just working on a bug right now |
| 15:39.39 | JunK-Y | which one? |
| 15:40.24 | jsmith | wishes it were 13732... OK, that one is pretty low priority |
| 15:40.37 | lmadsen | M12442 |
| 15:40.39 | MuffinMan | [feedback] [Asterisk] Applications/app_test 0012442: pri loop TestClient/TestServer fails: server SEND DTMF 8 reported by tzafrir (Karma: +119.75) http://bugs.digium.com/view.php?id=12442 |
| 15:40.42 | lmadsen | M13732 |
| 15:40.43 | MuffinMan | [acknowledged] [Asterisk] Channels/chan_iax2 0013732: IAX2 qualify problems between 1.6 and 1.4 boxes reported by jsmith (Karma: +30.25) http://bugs.digium.com/view.php?id=13732 |
| 15:47.37 | *** join/#asterisk-bugs Deeewayne (n=Deeewayn@nat/digium/x-18d0f6b47d94ead3) |
| 15:47.37 | *** mode/#asterisk-bugs [+o Deeewayne] by ChanServ |
| 15:52.31 | *** join/#asterisk-bugs guax (n=guaxinim@unaffiliated/guaxinim) |
| 16:39.21 | *** join/#asterisk-bugs russellb (n=russellb@asterisk/digium-open-source-team-lead/russellb) |
| 16:39.21 | *** mode/#asterisk-bugs [+o russellb] by ChanServ |
| 16:39.45 | lmadsen | Corydon76-dig: ping |
| 16:39.49 | lmadsen | M13718 |
| 16:39.51 | MuffinMan | [assigned] [Asterisk] Documentation 0013718: [patch] Neither CHANGES, nor UPGRADE.txt for 1.6.0 say quotes are no longer stripped reported by davidw (Karma: +5.75) http://bugs.digium.com/view.php?id=13718 |
| 16:39.53 | lmadsen | "The second sentence is OK. The first one does seem to imply that the quotes are optional, when things actually break if you use them." |
| 16:40.08 | Corydon76-dig | I like soft-spoken text |
| 16:40.31 | lmadsen | ok, then I have nothing in addition to add to it, so pls be committing :) |
| 16:41.04 | russellb | lmadsen: you have commit access, you know :) |
| 16:41.19 | lmadsen | lies |
| 16:41.51 | lmadsen | I guess that's the only way I'm going to get the practice I suppose |
| 16:52.37 | lmadsen | weird... that is already committed in UPGRADE-1.6.txt |
| 16:52.53 | Corydon76-dig | lmadsen: you are WAY too slow |
| 16:53.01 | lmadsen | I had to checkout trunk :) |
| 16:53.13 | lmadsen | and that's why I tend to let you guys commit things... better use of time |
| 16:53.20 | Corydon76-dig | You should already have all of those checkouts |
| 16:53.31 | lmadsen | I don't merge things |
| 16:53.31 | Corydon76-dig | and then just 'svn update' |
| 16:53.39 | lmadsen | so I didn't have it from origsvn |
| 16:53.40 | file | putnopvut: ooh nifty @ T.38 one |
| 16:54.06 | putnopvut | Yeah, it's cool when I don't even have to do any work besides just reviewing patches :) |
| 17:08.14 | Corydon76-dig | Anybody feel like commenting on my design proposal on the -dev list? |
| 17:17.02 | putnopvut | Corydon76-dig: I haven't really commented because I don't feel strongly enough to comment on it. It seems fine to me, though. |
| 17:36.51 | Corydon76-dig | putnopvut: I'm fairly certain on the copy one, but I waffled around on the blobupdate one |
| 17:38.05 | Corydon76-dig | I thought lenstore, then lenupdate, then finally blobupdate. I'm not great on the name or the functionality |
| 17:38.57 | Corydon76-dig | The issue is that one realtime operation is going to be divided into 2 operations... a store and a blobupdate |
| 17:39.21 | Corydon76-dig | Since the store can't handle a blob |
| 17:41.02 | Corydon76-dig | I also toyed with the idea of 4 parameters for each storage parameter, name, value, type, length |
| 17:41.44 | Corydon76-dig | I'm not sure if databases act badly if you specify a blob type for a non-blob field |
| 17:42.39 | Corydon76-dig | or I suppose I could use a constant like SQL_NTS for length, for text fields |
| 17:45.22 | Marquis42 | Corydon76-dig: I just re-read the proposal, and it looks fine to me (or at least I can't think of a better way off-hand). I certainly support the addition of the functionality, because it would make adding ODBC support to app_minivm much easier. |
| 17:48.03 | Corydon76-dig | Marquis42: I just want the functionality to be "right" |
| 17:48.33 | Corydon76-dig | and not something I need to recode in the future because somebody figured out a massive problem with it |
| 17:48.52 | Corydon76-dig | BTW, I think this approach, once solidified, negates the ast_storage branch |
| 17:49.02 | Marquis42 | Corydon76-dig: I understand. I apologize for not having any better comments. I just don't see anything immeidately wrong with the proposal. I'll give it some more thought, and get back to you if I come up with anything. |
| 17:49.51 | Corydon76-dig | The only way I could see this might cause a problem is with a multi-server system, where a voicemail is attempted to be retrieved immediately after it is stored |
| 17:50.07 | Corydon76-dig | but before the blobupdate completes |
| 17:51.20 | Corydon76-dig | and I could fix that with a 3-way operation, storing the record to an intermediate value, which can't be immediately retrieved, until after the third operation, which updates those key fields to their correct values |
| 17:53.11 | Marquis42 | That seems a little kludgy, like trying to hack a "lock" of some kind where it doesn't really fit. Not that I have a significantly better idea, I'm just not sure I like that one. |
| 17:53.37 | Corydon76-dig | Well, then I'm back to the 4-parameter store |
| 17:54.08 | Marquis42 | Anyway, I actually need to run off to a meeting (yay....). But I'll definitely give it some more thought. I'm personally less averse to the 4-parameter idea. |
| 17:54.42 | Marquis42 | I'll respond to your most recent post on -dev when I get a chance. |
| 17:55.53 | Corydon76-dig | The require method already uses 3-parameters, so it's not entirely without precedent |
| 17:56.14 | Corydon76-dig | although I may use 3-parameters, with negative values encompassing the non-blob types |
| 17:57.14 | Corydon76-dig | in which case, I either use 'store2' or 'lenstore' for naming |
| 17:57.23 | Corydon76-dig | store2, probably |
| 17:58.38 | Corydon76-dig | Marquis42: You've already been helpful. You've convinced me that a 3-param store function isn't overengineering the solution |
| 18:22.21 | Corydon76-dig | I wonder if I should call it store3 |
| 19:03.15 | *** join/#asterisk-bugs sysreq (n=sysreq@unaffiliated/sysreq) |
| 19:13.09 | lmadsen | M14004 |
| 19:13.11 | MuffinMan | [new] [Asterisk] Core/General 0014004: restart gracefully drops cap_net_admin capability reported by nemo (Karma: neutral) http://bugs.digium.com/view.php?id=14004 |
| 19:13.47 | lmadsen | any opinions on this? Asterisk running as non-root, trying to execute something that has been set to require root privileges doesn't work... do I not understand the issue correctly? |
| 19:14.37 | seanbright | it reads as though he is able to set it on startup but not restart. |
| 19:15.25 | lmadsen | question asked :) |
| 19:15.39 | lmadsen | aha.. reading the topic again makes more sense now |
| 19:18.39 | lmadsen | M14132 |
| 19:18.41 | MuffinMan | [new] [Asterisk] General 0014132: [patch] autosupport script not 100% ready for DAHDI reported by dsedivec (Karma: neutral) http://bugs.digium.com/view.php?id=14132 |
| 19:18.47 | lmadsen | Qwell: could I assign that issue to you? |
| 19:19.29 | seanbright | it's already been resolved |
| 19:19.31 | seanbright | i think |
| 19:19.40 | seanbright | http://svn.digium.com/view/asterisk?view=rev&revision=130042 |
| 19:20.40 | seanbright | hmmm, or not. |
| 19:22.57 | seanbright | lmadsen: i'd also ask 14004 what os/distro he is running on. |
| 19:23.24 | seanbright | lmadsen: we're using a linux specific method of keeping the cap_net_admin privilege around on restart |
| 19:23.57 | lmadsen | gotcha... I asked |
| 19:24.07 | lmadsen | thx! |
| 19:24.11 | seanbright | yup |
| 22:11.39 | lmadsen | mnicholson: yay! assigned an issue to you :) |
| 22:12.18 | *** join/#asterisk-bugs Entomologist (n=nobody@76.164.171.226) |
| 22:12.52 | seanbright | Entomologist: wb! |
| 22:14.15 | lmadsen | I almost closed 13121 until I did a lot more testing, and... *gasp*... looking at code to realize the bug really existed. |
| 22:53.08 | *** part/#asterisk-bugs russellb (n=russellb@asterisk/digium-open-source-team-lead/russellb) |
| 22:53.31 | Corydon76-dig | lmadsen: 14004 is completely valid |
| 22:53.49 | Corydon76-dig | We don't keep around capabilities on restart. That is not by accident. |
| 22:54.16 | Corydon76-dig | It was really to protect against forked processes from obtaining cap_net_admin |
| 22:54.54 | Corydon76-dig | However, I think his expectation is reasonable |
| 22:56.49 | seanbright | but KEEPCAPS is set |
| 22:57.00 | seanbright | or am i reading this wrong |
| 22:57.40 | lmadsen | Qwell: ping retransmit |
| 22:57.50 | Qwell | pong |
| 22:58.03 | lmadsen | [14:18] <MuffinMan> [new] [Asterisk] General 0014132: [patch] autosupport script not 100% ready for DAHDI reported by dsedivec (Karma: neutral) http://bugs.digium.com/view.php?id=14132 |
| 22:58.04 | lmadsen | [14:18] <lmadsen> Qwell: could I assign that issue to you? |
| 22:58.27 | Corydon76-dig | seanbright: the i flag isn't set, which means on exec, the capabilities are cleared. |
| 22:58.33 | Qwell | lmadsen: Might want to poke angler on it |
| 22:58.34 | Corydon76-dig | When we do a restart, we do an exec |
| 22:58.42 | lmadsen | Qwell: okie, will do that |
| 22:58.45 | seanbright | the i flag? |
| 22:58.47 | lmadsen | angler: ping! :) |
| 22:58.54 | Corydon76-dig | seanbright: Inheritable |
| 22:59.41 | seanbright | hm, ok then i am reading something wrong. |
| 23:01.48 | seanbright | Corydon76-dig: ohhhhhhhhh |
| 23:01.53 | seanbright | cap_net_admin=epi |
| 23:01.55 | seanbright | would make it work |
| 23:02.00 | seanbright | yes> |
| 23:02.01 | seanbright | ? |
| 23:03.47 | Corydon76-dig | Correct |
| 23:04.01 | seanbright | just make another flag to asterisk that does a cap_net_admin+i |
| 23:04.02 | Corydon76-dig | eip is what is suggested in the documentation |
| 23:04.04 | seanbright | and we're done |
| 23:04.14 | Corydon76-dig | Well, no |
| 23:04.22 | seanbright | i'd like to be right once |
| 23:04.24 | seanbright | just once. |
| 23:04.25 | Corydon76-dig | We have to avoid that flag being passed on to forked processes |
| 23:04.45 | angler | lmadsen, pong! |
| 23:05.35 | seanbright | runs off |
| 23:58.06 | *** part/#asterisk-bugs Deeewayne (n=Deeewayn@nat/digium/x-18d0f6b47d94ead3) |
| 23:59.20 | lmadsen | angler: pong-ack! |
| 23:59.31 | lmadsen | M14132 |
| 23:59.33 | MuffinMan | [new] [Asterisk] General 0014132: [patch] autosupport script not 100% ready for DAHDI reported by dsedivec (Karma: neutral) http://bugs.digium.com/view.php?id=14132 |
| 23:59.39 | lmadsen | angler: I was wondering if that would be something you could handle? |