| 02:14.54 | *** join/#asterisk-bugs leifmadsen (n=Leif@asterisk/documenteur-extraordinaire/blitzrage) |
| 02:14.54 | *** mode/#asterisk-bugs [+o leifmadsen] by ChanServ |
| 07:26.24 | *** join/#asterisk-bugs tzafrir__laptop (n=tzafrir@local.xorcom.com) |
| 10:47.44 | *** join/#asterisk-bugs Marquis42 (n=bfhbmw0@65-127-126-34.dia.static.qwest.net) |
| 11:07.25 | *** join/#asterisk-bugs AllGoodNiksTaken (n=sean@160.77-136-217.adsl-dyn.isp.belgacom.be) |
| 11:08.22 | AllGoodNiksTaken | Hello to all Asterisk gurus here :) |
| 11:08.57 | AllGoodNiksTaken | Is it ok if I ask an asterisk question about billsec=0? |
| 11:09.56 | AllGoodNiksTaken | Ah, just read the topic.. will take my Q to #asterisk :) |
| 11:10.09 | *** part/#asterisk-bugs AllGoodNiksTaken (n=sean@160.77-136-217.adsl-dyn.isp.belgacom.be) |
| 11:27.24 | *** join/#asterisk-bugs caio1982 (i=caio1982@CAcert-br/caio1982) |
| 12:21.44 | *** join/#asterisk-bugs oej (n=olle@ns.webway.se) |
| 12:48.36 | *** join/#asterisk-bugs ccesario (n=ccesario@189-19-6-236.dsl.telesp.net.br) |
| 13:22.32 | *** join/#asterisk-bugs oej (n=olle@ns.webway.se) |
| 13:26.45 | *** join/#asterisk-bugs |Cybex| (n=John@atwork-20.r-212.178.82.atwork.nl) |
| 13:50.57 | *** join/#asterisk-bugs leifmadsen (n=Leif@asterisk/documenteur-extraordinaire/blitzrage) |
| 13:50.57 | *** mode/#asterisk-bugs [+o leifmadsen] by ChanServ |
| 13:56.29 | *** join/#asterisk-bugs Deeewayne (n=dwayne@75.76.254.162) |
| 13:56.30 | *** mode/#asterisk-bugs [+o Deeewayne] by ChanServ |
| 14:25.10 | Entomologist | *** CLOSED (16171) [Applications/app_queue] AgentComplete event sent as soon as call is answered in call to queue through local channel |
| 14:25.10 | Entomologist | Reported by: legart Karma: 0 |
| 14:25.10 | Entomologist | https://issues.asterisk.org/view.php?id=16171 |
| 14:25.11 | Entomologist | ********************************************************* |
| 14:31.08 | leifmadsen | M16154 |
| 14:31.10 | MuffinMan | [new] [Asterisk] Features/Parking 0016154: hangup on transfer reported by avinoash (Karma: neutral) https://issues.asterisk.org/view.php?id=16154 |
| 14:31.23 | leifmadsen | This almost sounds like the guy is attempting to do a blonde transfer... |
| 14:35.20 | *** join/#asterisk-bugs jpeeler (n=jpeeler@asterisk/digium-software-dev/jpeeler) |
| 14:35.44 | leifmadsen | jpeeler: I assigned an issue to you this morning so you can review it |
| 14:35.46 | *** join/#asterisk-bugs davidbrillert (n=aragon12@h216-235-8-130.host.egate.net) |
| 14:36.01 | leifmadsen | it looks like you closed an issue, and may have forgotten to merge it into 1.4 |
| 14:36.16 | jpeeler | i already answered it and closed it |
| 14:36.27 | jpeeler | oh another one? |
| 14:36.45 | leifmadsen | I just did it a couple mins ago |
| 14:36.55 | leifmadsen | but it could be the "same issue" |
| 14:37.04 | leifmadsen | issue about IMAP temp greetings |
| 14:37.15 | leifmadsen | M16170 |
| 14:37.17 | MuffinMan | [assigned] [Asterisk] Applications/app_voicemail/IMAP 0016170: temporary greetings can't be erased in 1.4 reported by francesco_r (Karma: +4.25) https://issues.asterisk.org/view.php?id=16170 |
| 14:37.38 | leifmadsen | I thought I had already seen an issue that like opened up, but I couldn't find it again. Perhaps you had closed it. |
| 14:37.58 | jpeeler | well it really shouldn't be assigned to me... i mean yes i fixed the same symptom, but this is another issue |
| 14:38.24 | leifmadsen | it is? ok, it just looked like the same issue to me, and I didn't see a merge down to 1.4, so it just looked like you missed it |
| 14:38.42 | leifmadsen | I only assigned it to you for review; it doesn't mean that you have to resolve it unless it was the same issue |
| 14:38.54 | jpeeler | the temporary greetings in 1.4 are not stored in imap |
| 14:38.59 | jpeeler | so it's sort of the same... |
| 14:39.06 | jpeeler | but on the file system instead |
| 14:39.29 | leifmadsen | ok, I will set it to Ready for Testing and unassign you |
| 14:40.08 | jpeeler | although it's been such a long time |
| 14:40.17 | jpeeler | let me look at this patch, maybe it is so similar i should just handle it |
| 14:41.03 | leifmadsen | jpeeler: ok, go ahead -- I've set it to the appropriate status, so do what you may |
| 14:41.39 | jpeeler | ok well your comment is good enough, although he didn't actually post a patch |
| 14:42.05 | leifmadsen | wtf... |
| 14:42.17 | leifmadsen | I see, I will set it to feedback then |
| 14:43.27 | jpeeler | i guess a piece of the patch might apply, i'd rather just wait for the patch to confirm |
| 14:44.59 | leifmadsen | jpeeler: no worries, it's minor |
| 14:45.25 | leifmadsen | M16141 |
| 14:45.27 | MuffinMan | [acknowledged] [Asterisk] Channels/chan_iax2 0016141: IAX trunk clicks as other calls in the same trunk hang up reported by alecdavis (Karma: +80.50) https://issues.asterisk.org/view.php?id=16141 |
| 14:45.56 | leifmadsen | russellb: I've moved that from Minor to Major, although it is tempting to move it to Block. I'm leaving it at Major unless you tell me different though. |
| 14:55.04 | Entomologist | *** CLOSED (16136) [Applications/app_meetme] Meetme - Quitting time issue |
| 14:55.05 | Entomologist | Reported by: asannucci Karma: 0 |
| 14:55.05 | Entomologist | https://issues.asterisk.org/view.php?id=16136 |
| 14:55.06 | Entomologist | ********************************************************* |
| 15:12.55 | leifmadsen | M16106 |
| 15:12.57 | MuffinMan | [ready for review] [Asterisk] Core/General 0016106: [patch] Hangup extension executed twice in 1.6.2 RC2 reported by ajohnson (Karma: +1.75) https://issues.asterisk.org/view.php?id=16106 |
| 15:13.22 | leifmadsen | russellb: I feel like maybe that issue should try to get resolved before the next round of 1.6.2.0 RCs, especially since the reporter states the patch attached there resolves the issue |
| 15:14.00 | leifmadsen | and since it is a regression from RC2 |
| 15:23.41 | Entomologist | *** CLOSED (15857) [Core/Configuration] [patch] Revision 152765 introduces regression in stdexten |
| 15:23.42 | Entomologist | Assigned to: lmadsen |
| 15:23.42 | Entomologist | Reported by: pprindeville Karma: 5 |
| 15:23.43 | Entomologist | https://issues.asterisk.org/view.php?id=15857 |
| 15:23.43 | Entomologist | ********************************************************* |
| 15:30.44 | *** join/#asterisk-bugs putnopvut (n=putnopvu@asterisk/master-of-queues/mmichelson) |
| 15:30.44 | *** mode/#asterisk-bugs [+o putnopvut] by ChanServ |
| 15:36.19 | Entomologist | *** CLOSED (15909) [Channels/chan_dahdi] When no callerid is recieved, cannot override callerid |
| 15:36.19 | Entomologist | Assigned to: file |
| 15:36.20 | Entomologist | Reported by: kobaz Karma: 2.25 |
| 15:36.20 | Entomologist | https://issues.asterisk.org/view.php?id=15909 |
| 15:36.21 | Entomologist | ********************************************************* |
| 15:47.51 | davidbrillert | hey Leif, this ticket was marked as a blocker but it keeps getting removed from roadmap ? |
| 15:48.01 | leifmadsen | what ticket? |
| 15:48.05 | davidbrillert | Reporter aragon (Karma: +4.25) View Status public |
| 15:48.05 | davidbrillert | Assigned To |
| 15:48.05 | davidbrillert | Priority normal Resolution open |
| 15:48.05 | davidbrillert | Status acknowledged |
| 15:48.05 | davidbrillert | Summary 0016081: utils.c:938 ast_carefulwrite: Timed out trying to write causes corruption to astdb |
| 15:48.46 | leifmadsen | M16081 |
| 15:48.49 | MuffinMan | [acknowledged] [Asterisk] Resources/res_agi 0016081: utils.c:938 ast_carefulwrite: Timed out trying to write causes corruption to astdb reported by aragon (Karma: +4.25) https://issues.asterisk.org/view.php?id=16081 |
| 15:48.56 | davidbrillert | yes |
| 15:50.56 | leifmadsen | I think it got removed because of a script that was run on it |
| 15:51.23 | davidbrillert | yes, probably. something about assigning to internal priority queue |
| 15:51.45 | leifmadsen | yes |
| 15:52.03 | davidbrillert | sorry in advance, but what is the internal priority queue? |
| 15:52.43 | davidbrillert | how does it affect a quicker resolution? |
| 15:52.46 | leifmadsen | I'm going to fix the Block issue |
| 15:52.47 | leifmadsen | it doesn't |
| 15:52.56 | davidbrillert | :( |
| 15:53.04 | leifmadsen | I'm not going to discuss it |
| 15:53.13 | leifmadsen | it's just a method of tagging issues |
| 15:53.44 | leifmadsen | there are only so many resources to go around; issues will be resolved as soon as resources and time permit |
| 15:59.10 | *** join/#asterisk-bugs |Cybex| (n=John@atwork-20.r-212.178.82.atwork.nl) |
| 16:00.46 | *** part/#asterisk-bugs |Cybex| (n=John@atwork-20.r-212.178.82.atwork.nl) |
| 16:08.35 | *** join/#asterisk-bugs |Cybex| (n=John@atwork-20.r-212.178.82.atwork.nl) |
| 16:30.49 | *** join/#asterisk-bugs |Cybex| (n=John@atwork-20.r-212.178.82.atwork.nl) |
| 17:12.57 | Entomologist | *** CLOSED (16120) [Channels/chan_sip/General] SIP peers are not being built from users.conf configuration |
| 17:12.58 | Entomologist | Assigned to: dvossel |
| 17:12.58 | Entomologist | Reported by: jsmith Karma: 30.25 |
| 17:12.59 | Entomologist | https://issues.asterisk.org/view.php?id=16120 |
| 17:12.59 | Entomologist | ********************************************************* |
| 17:19.25 | Entomologist | *** CLOSED (13828) [Channels/chan_dahdi/NewFeature] [patch] !! Unknown IE 50 (cs5, Unknown Information Element) & RTCP SR transmission error, rtcp halted |
| 17:19.26 | Entomologist | Reported by: fdecher Karma: 0 |
| 17:19.26 | Entomologist | https://issues.asterisk.org/view.php?id=13828 |
| 17:19.27 | Entomologist | ********************************************************* |
| 17:41.30 | *** join/#asterisk-bugs albertoandrade (n=albertoa@189.4.48.141) |
| 17:56.23 | Qwell | M16172 |
| 17:56.25 | MuffinMan | [new] [Asterisk] General 0016172: [patch] "make config" creates really wrong runlevels in Debian (includes patch) reported by ibc (Karma: +3.25) https://issues.asterisk.org/view.php?id=16172 |
| 17:56.29 | Qwell | tzafrir: Do you agree with my comments there? |
| 18:31.15 | *** join/#asterisk-bugs philipp64 (n=chatzill@63.224.43.239) |
| 18:31.34 | leifmadsen | philipp64: so you have not made it very clear what is actually wrong in 15858 |
| 18:31.35 | philipp64 | leifmadsen: ok, fine, here we are on #asterisk-bugs instead. |
| 18:31.39 | leifmadsen | your extensions.conf is not useful |
| 18:31.53 | leifmadsen | redfish is not part of extensions.conf.sample afaik |
| 18:32.00 | philipp64 | I use stdexten in two different places, but it's only found in one of those. |
| 18:32.17 | philipp64 | in the other instance, it complains "No such label 'stdexten' in extension 's' in context 'redfish-pstn'" |
| 18:32.19 | leifmadsen | a diff of what actually needs to be change would be much more useful |
| 18:32.44 | philipp64 | I don't know what changes. not all bug reports come with a fix. :-( |
| 18:33.38 | leifmadsen | ok, lets try this a different way -- what part of the extensions.conf.sample file is wrong? I see your context, which is doing something wrong |
| 18:33.54 | leifmadsen | the redfish-pstn context is probably not using GoSub() correctly |
| 18:34.59 | leifmadsen | exten => s,n,Gosub(stdexten(111,${PHILIP},redfish)) <-- this is probably not doing what you expect |
| 18:35.06 | Corydon76-dig | We REALLY REALLY REALLY recommend that you do NOT intermix calls to Gosub and calls to Macro |
| 18:35.20 | philipp64 | ok... well, it used to work until the definition of stdexten changed. |
| 18:35.25 | philipp64 | I'm not using macros. |
| 18:35.43 | leifmadsen | you're calling the stdexten priority label, which I don't see you using |
| 18:36.12 | leifmadsen | as far as I can tell, this is a configuration issue |
| 18:36.13 | Qwell | exten => s,n,Gosub(stdexten(111,${PHILIP},redfish)) |
| 18:36.18 | Qwell | I'm rather confused by that syntax.. |
| 18:36.27 | leifmadsen | Qwell: that is correct syntax for GoSub() |
| 18:36.47 | Qwell | hrm, okay |
| 18:36.50 | *** part/#asterisk-bugs Qwell (i=north@pdpc/sponsor/digium/Qwell) |
| 18:36.54 | *** join/#asterisk-bugs Qwell (i=north@pdpc/sponsor/digium/Qwell) |
| 18:36.54 | *** mode/#asterisk-bugs [+o Qwell] by ChanServ |
| 18:36.58 | Qwell | this isn't nano. |
| 18:37.02 | leifmadsen | GoSub([[context,]extension,]priority(arg1,arg2)) |
| 18:37.02 | Corydon76-dig | assuming that stdexten is a priority, okay |
| 18:38.16 | leifmadsen | so ya... I don't see any issue here... |
| 18:38.36 | Qwell | stdexten priority? O.o |
| 18:39.14 | leifmadsen | GoSub() is looking for the stdexten priority label in [redfish-pstn] context that isn't part of extensions.conf.sample, which means it is an issue local to your dialplan; either that, or I'm totally missing something |
| 18:39.25 | leifmadsen | Qwell: I know what the problem is, but I don't see an issue for us to resolve |
| 18:39.48 | Qwell | I guess I don't understand how gosub works |
| 18:40.01 | leifmadsen | Qwell: GoSub() is basically just a Goto() that remembers where it came from |
| 18:40.05 | leifmadsen | and you can get back to it with Return() |
| 18:40.14 | leifmadsen | plus you can pass arguments to it |
| 18:40.31 | leifmadsen | exten => s,n,Gosub(stdexten(111,${PHILIP},redfish)) <-- trying to Goto() stdexten priority label |
| 18:40.32 | Qwell | oh, I get it |
| 18:40.40 | leifmadsen | then 111 is ${ARG1}, etc... |
| 18:40.42 | Qwell | so, basically.. need to include => stdexten |
| 18:41.24 | philipp64 | So I need to invoke GoSub(stdexten,111,1,(111,${PHILIP},redfish)) instead then... ok, that changed. |
| 18:41.37 | leifmadsen | that has always been the case |
| 18:41.44 | philipp64 | Also, do we really need stdexten to take ${ARG1} then? Why not just use ${EXTEN} in that case? |
| 18:41.48 | leifmadsen | Qwell: not quite -- there is no 'stdexten' priority label |
| 18:41.55 | tzafrir | Qwell, no. 'start' / 'stop' shouldn't be used anyway. Just use 'defaults' |
| 18:42.14 | leifmadsen | oh there it is |
| 18:42.15 | tzafrir | It is also getting increasingly important to set service dependencies |
| 18:42.24 | leifmadsen | exten => _X.,50000(stdexten),NoOp(Start stdexten) |
| 18:42.25 | leifmadsen | wtf.... |
| 18:42.38 | leifmadsen | how do you start stdexten with priority number 50000 ? |
| 18:42.48 | Qwell | leifmadsen: You don't. hence the label |
| 18:42.55 | leifmadsen | there is no priority 1.... |
| 18:43.03 | leifmadsen | oh I guess you can GoSub(), I see |
| 18:43.29 | Qwell | I think the answer is to just include it, like [demo] does |
| 18:43.48 | leifmadsen | Qwell: he does via nested includes |
| 18:44.09 | leifmadsen | [redfish-pstn] includes [redfish-extens] includes [stdexten |
| 18:44.11 | leifmadsen | ] |
| 18:44.44 | leifmadsen | anyways, I don't understand why this isn't a duplicate |
| 18:44.55 | leifmadsen | assuming 15857 fixed all the issues |
| 18:44.59 | leifmadsen | M15857 |
| 18:45.01 | MuffinMan | [closed] [Asterisk] Core/Configuration 0015857: [patch] Revision 152765 introduces regression in stdexten reported by pprindeville (Karma: +5.00) https://issues.asterisk.org/view.php?id=15857 |
| 18:45.34 | leifmadsen | actually it should be fine... I'm looking at the current extensions.conf.sample file and it contains the exten => _X.,50000(stdexten),NoOp(Start stdexten) |
| 18:45.35 | leifmadsen | <PROTECTED> |
| 18:45.45 | leifmadsen | there is no reason that shouldn't work |
| 18:46.47 | Qwell | 147896 was only comment changes in a sample config file O.o |
| 18:47.03 | Qwell | wait, nm |
| 18:47.38 | leifmadsen | Corydon76-dig: can you do a Goto(priority) into nested include statements? |
| 18:48.09 | philipp64 | leifmadsen: so how does one invoke the new form of stdexten? |
| 18:48.21 | leifmadsen | I don't understand what the "new form" is |
| 18:49.55 | leifmadsen | I don't understand why _X. is being used at all there |
| 18:50.00 | Qwell | I assume it's doing a Goto(${EXTEN},stdexten) |
| 18:50.06 | Qwell | which means 's' would make that funky |
| 18:50.15 | Qwell | with, yeah, the _X. |
| 18:50.27 | leifmadsen | if you don't include an extension it doesn't do 's'... |
| 18:50.43 | Qwell | no, it would do ${EXTEN}, which for him would likely be s |
| 18:51.04 | Qwell | Executing [s@redfish-pstn:27] |
| 18:51.52 | leifmadsen | ya... that is the currently executing extension |
| 18:52.13 | leifmadsen | oh wait... ya I see the problem |
| 18:52.29 | leifmadsen | Qwell: you're right |
| 18:52.37 | Qwell | I don't know that _. would work there though |
| 18:52.50 | leifmadsen | it's basically trying to go to 'stdexten' priority of the 's' extension, which doesn't exist anywhere |
| 18:53.18 | Qwell | hrm, priority 50000.. _. might be okay |
| 18:53.52 | jsmith | _. is not considered "best practice" |
| 18:54.02 | Qwell | jsmith: no, but this is a rather special case |
| 18:54.09 | philipp64 | leifmadsen: i didn't get the use of _X. either. |
| 18:54.13 | Qwell | where one would certainly want to catch all |
| 18:54.15 | jsmith | It'll throw a warning on the CLI |
| 18:54.34 | leifmadsen | really, this could be simplified by doing: exten => _X.,n,Set(LOCAL(ext)=${EXTEN}) instead of exten => _X.,n,Set(LOCAL(ext)=${ARG1}) and calling the GoSub() with: GoSub(111,stdexten(${PHILIP},redfish)) and changing the ARG2 and ARG3 to ARG1 and ARG2 respectively |
| 18:55.06 | leifmadsen | there is no reason to pass the extension as ARG1 when you're forcing an extension to be passed anyways |
| 18:55.20 | philipp64 | that's what I was thinking. |
| 18:55.27 | philipp64 | ok, I'll prepare a patch... |
| 18:56.00 | leifmadsen | when did [stdexten] get introduced and how was it being used before? |
| 18:56.21 | Qwell | leifmadsen: see 1.6.2 r152765 |
| 18:56.32 | leifmadsen | how is it being called? |
| 18:56.49 | leifmadsen | is it being called at all in the extensions.conf.sample file? I can't find any use of it |
| 18:57.20 | Qwell | leifmadsen: demo,1234,n=2 |
| 18:57.52 | leifmadsen | and a few other places, thanks |
| 18:58.06 | leifmadsen | philipp64: I'll just prepare the patch since there are a few other places it needs to be updated |
| 18:58.12 | philipp64 | in the "acme" example too. |
| 18:58.26 | leifmadsen | there are several locations that will need to be updated. I've found them all though. |
| 18:58.35 | philipp64 | yup. me too. |
| 18:59.30 | philipp64 | So do we still need to use it as Gosub(stdexten,NNN,stdexten(DEV,CNTX)) ? |
| 18:59.47 | leifmadsen | no, because you're including [stdexten] |
| 18:59.57 | leifmadsen | so it should try to find the extension,50000(stdexten) |
| 19:00.10 | leifmadsen | I'm going to have you test my patch here before I commit it |
| 19:00.26 | philipp64 | having to repeat context/priority is a bit of a hack if we do. |
| 19:00.46 | leifmadsen | huh? |
| 19:00.49 | leifmadsen | you don't need the context |
| 19:01.20 | philipp64 | because it's been included directly into the current context... |
| 19:01.34 | leifmadsen | I doubt it's an issue |
| 19:03.01 | leifmadsen | philipp64: try that patch |
| 19:03.45 | leifmadsen | apply in the 'configs' dir |
| 19:07.25 | philipp64 | minor quibble: I would rewrite as: ;exten => 6275,1,Gosub(6275,stdexten(${MARK})) instead. |
| 19:07.43 | philipp64 | using ${EXTEN} there just seems gratuitous. perhaps if it were a macro... |
| 19:08.25 | philipp64 | otherwise, I came up with the same set of patches... tested them... they ran. |
| 19:15.05 | leifmadsen | well not using ${EXTEN} could cause a typo, so I'm just showing how you would just pass the extension, there are other extensions where I pass the number directly because the extension is actually a DID for example |
| 19:15.35 | philipp64 | ok, either way... I was thinking of people like me that use m4 to generate their dialplans... |
| 19:15.51 | leifmadsen | right, but not everyone does, and these are just examples, so it doesn't really matter which you use |
| 19:16.03 | philipp64 | right. not a big deal. just a thought. |
| 19:16.18 | leifmadsen | coolio, I think I'll just commit it like this then |
| 19:16.29 | philipp64 | excellent. |
| 19:20.01 | philipp64 | I was thinking of filing a defect to cover another example in extensions.conf for a generic freenum context... just like there is for dundi... |
| 19:20.15 | philipp64 | hopefully this one will be less bother. :-) |
| 19:21.19 | philipp64 | leifmadsen: did you look at the new diffs for 16090? |
| 19:22.48 | leifmadsen | eh? |
| 19:23.01 | philipp64 | <PROTECTED> |
| 19:23.10 | leifmadsen | not yet, but I can do that next |
| 19:23.20 | philipp64 | wait... give me 10. |
| 19:23.44 | leifmadsen | no worries, I'm still working on commiting this other issue |
| 19:25.24 | Entomologist | *** CLOSED (15858) [Core/Configuration] [patch] Revision 152765 introduces scoping difficulties in stdexten |
| 19:25.24 | Entomologist | Reported by: pprindeville Karma: 5 |
| 19:25.24 | Entomologist | https://issues.asterisk.org/view.php?id=15858 |
| 19:25.24 | Entomologist | ********************************************************* |
| 19:26.58 | leifmadsen | philipp64: ok, let me know when you're done that as I can review it since I'm done committing 15858 now |
| 19:41.53 | *** join/#asterisk-bugs ccesario (n=ccesario@189-19-6-236.dsl.telesp.net.br) |
| 19:43.33 | philipp64 | leifmadsen: https://issues.asterisk.org/file_download.php?file_id=24328&type=bug |
| 19:48.29 | philipp64 | errr... almost. #3 I think. |
| 19:48.39 | philipp64 | just tested it. works fine. |
| 19:48.55 | Entomologist | *** CLOSED (16071) [Applications/app_controlplayback] [patch] Warning log message with debug info |
| 19:48.56 | Entomologist | Assigned to: lmadsen |
| 19:48.56 | Entomologist | Reported by: atis Karma: 18 |
| 19:48.57 | philipp64 | suffers from the same hack as stdexten, though... using _X. ... |
| 19:48.57 | Entomologist | https://issues.asterisk.org/view.php?id=16071 |
| 19:48.57 | Entomologist | ********************************************************* |
| 19:55.44 | leifmadsen | ya... I'm not sure how much more of these priority number hacks I want to put in... |
| 19:55.51 | leifmadsen | I'm not a big fan of that syntax |
| 19:56.50 | leifmadsen | I'd much prefer something like start,1,NoOp() and calling it via GoSub(time,start,1) |
| 19:57.29 | leifmadsen | I know it looks nicer to call it via GoSub(time) but I can see all sorts of issues creeping in if you're not careful with the priority numbering hack |
| 19:57.32 | leifmadsen | jsmith: thoughts? |
| 19:59.51 | Entomologist | *** CLOSED (14517) [Core/BuildSystem] [patch] signals.h - syntax error before '*' token |
| 19:59.51 | Entomologist | Reported by: asgaroth Karma: 0 |
| 19:59.52 | Entomologist | https://issues.asterisk.org/view.php?id=14517 |
| 19:59.52 | Entomologist | ********************************************************* |
| 20:07.11 | leifmadsen | M15244 |
| 20:07.13 | MuffinMan | [assigned] [Asterisk] PBX/pbx_ael 0015244: [patch] Invalid syntax with two ampersands in if block reported by atis (Karma: +18.00) https://issues.asterisk.org/view.php?id=15244 |
| 20:07.21 | leifmadsen | any opinions on this patch, and whether it is a good idea or not? |
| 20:07.25 | leifmadsen | Corydon76-dig: ? |
| 20:08.03 | russellb | he's busy :-p |
| 20:12.30 | leifmadsen | no worries, just wanted to leave a msg and figured he'd see it at some point |
| 20:12.35 | leifmadsen | jsmith: ping? |
| 20:12.41 | leifmadsen | M15421 |
| 20:12.43 | MuffinMan | [confirmed] [Asterisk] PBX/pbx_config 0015421: [patch] Serious problem in pattern matching reported by jsmith (Karma: +30.25) https://issues.asterisk.org/view.php?id=15421 |
| 20:12.48 | leifmadsen | what versions does that affect? All? |
| 20:13.03 | jsmith | leifmadsen: Last I checked, it affected 1.6.0 and 1.6.1... not sure I checked it on 1.4 |
| 20:13.10 | leifmadsen | ok thanks |
| 20:16.09 | philipp64 | leifmadsen: I agree. it's too bad there's no shorthand for Gosub(context) that would expand to Gosub(context,s,1) |
| 20:16.45 | philipp64 | the options should have been right-to-left, and not left-to-right... |
| 20:16.55 | philipp64 | in terms of the parameters you could leave out. |
| 20:19.00 | Marquis42 | leifmadsen: That seems like a pretty straightforward warning to add. |
| 20:19.17 | Marquis42 | Err... I'm behind... I meant re: 15244 |
| 20:22.19 | leifmadsen | Marquis42: ya, i figured :) I just wasn't sure if we were going down a path we didn't want to in terms of adding syntax warnings |
| 20:23.00 | Marquis42 | Gotcha. Not exactly my call on that, of course. :) |
| 21:16.17 | Entomologist | *** CLOSED (15168) [Applications/app_queue] [patch] Add ability to use extension state as well as device state when adding queue memebers |
| 21:16.18 | Entomologist | Assigned to: file |
| 21:16.18 | Entomologist | Reported by: p_lindheimer Karma: 10.25 |
| 21:16.19 | Entomologist | https://issues.asterisk.org/view.php?id=15168 |
| 21:16.19 | Entomologist | ********************************************************* |
| 21:21.13 | Entomologist | *** CLOSED (15180) [CDR/General] [patch] No unique identifier for CDR |
| 21:21.14 | Entomologist | Assigned to: mnicholson |
| 21:21.14 | Entomologist | Reported by: Nick_Lewis Karma: 6.25 |
| 21:21.15 | Entomologist | https://issues.asterisk.org/view.php?id=15180 |
| 21:21.15 | Entomologist | ********************************************************* |
| 21:26.31 | Entomologist | *** CLOSED (14635) [Core/ManagerInterface/NewFeature] [patch] Changes to manager interface for registering event hooks |
| 21:26.32 | Entomologist | Reported by: jozza Karma: 1.75 |
| 21:26.32 | Entomologist | https://issues.asterisk.org/view.php?id=14635 |
| 21:26.33 | Entomologist | ********************************************************* |
| 22:00.03 | *** join/#asterisk-bugs tzafrir (n=tzafrir@bzq-218-155-147.cablep.bezeqint.net) |
| 22:50.56 | *** join/#asterisk-bugs tzafrir (n=tzafrir@212.179.75.202) |