| 00:07.56 | *** join/#asterisk-bugs pabelanger (~pabelange@CPE0013f7abc09a-CM0013f7abc096.cpe.net.cable.rogers.com) |
| 00:14.29 | pabelanger | r616 |
| 00:14.33 | MuffinMan | [R616] Dynamiclly generate asterisk.conf (new script) [Pending] Submitted by pabelanger => https://reviewboard.asterisk.org/r/616/ |
| 00:15.22 | pabelanger | So, starting to merge this but ran into an error about a new file the patch creates. Any information about setting mime types for .config.samples? |
| 00:16.46 | pabelanger | Is it safe to follow the output of pre-commit-propcheck |
| 00:17.40 | pabelanger | http://pastebin.com/hgmQ0t85 |
| 00:21.05 | pabelanger | seems correct, just compared the settings on adsi.conf.samples |
| 00:22.34 | Entomologist | *** CLOSED (17027) [Core/BuildSystem] [patch] Makefile update to only build asterisk.conf |
| 00:22.34 | Entomologist | Assigned to: pabelanger |
| 00:22.35 | Entomologist | Reported by: pabelanger Karma: 36.5 |
| 00:22.35 | Entomologist | https://issues.asterisk.org/view.php?id=17027 |
| 00:22.36 | Entomologist | ********************************************************* |
| 00:37.43 | leifmadsen | pabelanger: hey, there is a script for that |
| 00:38.11 | leifmadsen | propset_txt |
| 00:40.29 | pabelanger | leifmadsen: ok |
| 00:40.48 | pabelanger | I managed to set them manually |
| 00:40.54 | leifmadsen | coolio |
| 00:41.01 | leifmadsen | in the future that script makes it nice and easy :) |
| 00:41.21 | pabelanger | yar |
| 00:44.38 | Entomologist | *** CLOSED (16343) [Core/ManagerInterface] [patch] [branch] New CLI command: manager show settings |
| 00:44.39 | Entomologist | Assigned to: pabelanger |
| 00:44.39 | Entomologist | Reported by: pabelanger Karma: 36.5 |
| 00:44.40 | Entomologist | https://issues.asterisk.org/view.php?id=16343 |
| 00:44.40 | Entomologist | ********************************************************* |
| 00:46.04 | pabelanger | pew! 2 merges :) |
| 00:57.47 | leifmadsen | yay! |
| 00:57.52 | leifmadsen | close those issues! |
| 02:41.56 | *** join/#asterisk-bugs Deeewayne (~dwayne@c-71-207-214-190.hsd1.al.comcast.net) |
| 02:41.56 | *** mode/#asterisk-bugs [+o Deeewayne] by ChanServ |
| 05:06.01 | *** join/#asterisk-bugs Corydon76-dig (eleven@c-69-137-80-31.hsd1.tn.comcast.net) |
| 05:06.02 | *** mode/#asterisk-bugs [+o Corydon76-dig] by ChanServ |
| 05:51.38 | *** join/#asterisk-bugs oej (~olle@ns.webway.se) |
| 06:38.06 | *** join/#asterisk-bugs oej (~olle@ns.webway.se) |
| 07:03.38 | *** join/#asterisk-bugs jsmith-away (~jsmith@asterisk/training-and-documentation-guru/jsmith) |
| 07:03.38 | *** mode/#asterisk-bugs [+o jsmith-away] by ChanServ |
| 07:09.19 | *** join/#asterisk-bugs oej (~olle@ns.webway.se) |
| 08:13.00 | *** join/#asterisk-bugs Aurs (~ove@nidaros.netcore.no) |
| 08:13.14 | Aurs | yes! got a new segfault! |
| 08:13.30 | Aurs | https://issues.asterisk.org/view.php?id=17281 |
| 08:13.33 | Aurs | hehe |
| 08:40.25 | oej | :-) |
| 08:40.32 | oej | I don't know if I should say "congratulations" |
| 08:44.26 | Aurs | hehe |
| 08:45.00 | Aurs | at least the crash is reproduceable |
| 09:14.38 | oej | great |
| 11:24.58 | *** join/#asterisk-bugs eliel (~eliels@201.234.94.226) |
| 11:31.03 | *** join/#asterisk-bugs leifmadsen (~Leif@asterisk/documenteur-extraordinaire/blitzrage) |
| 11:31.03 | *** mode/#asterisk-bugs [+o leifmadsen] by ChanServ |
| 12:08.40 | *** join/#asterisk-bugs caio1982 (~caio1982@CAcert-br/caio1982) |
| 12:14.06 | *** join/#asterisk-bugs mvanbaak (~michiel@asterisk/contributor-and-bug-marshal/mvanbaak) |
| 12:51.56 | *** join/#asterisk-bugs mvanbaak (~michiel@asterisk/contributor-and-bug-marshal/mvanbaak) |
| 12:55.24 | *** join/#asterisk-bugs mvanbaak (~michiel@asterisk/contributor-and-bug-marshal/mvanbaak) |
| 13:03.31 | *** join/#asterisk-bugs malcolmd (~malcolmd@pdpc/sponsor/digium/malcolmd) |
| 13:04.09 | *** join/#asterisk-bugs mvanbaak (~michiel@asterisk/contributor-and-bug-marshal/mvanbaak) |
| 13:11.57 | *** join/#asterisk-bugs mvanbaak (~michiel@asterisk/contributor-and-bug-marshal/mvanbaak) |
| 14:26.05 | *** join/#asterisk-bugs Deeewayne (~dwayne@75-150-14-49-Atlanta.hfc.comcastbusiness.net) |
| 14:26.05 | *** mode/#asterisk-bugs [+o Deeewayne] by ChanServ |
| 14:32.23 | *** join/#asterisk-bugs putnopvut (~putnopvut@asterisk/master-of-queues/mmichelson) |
| 14:32.23 | *** mode/#asterisk-bugs [+o putnopvut] by ChanServ |
| 14:32.29 | *** join/#asterisk-bugs The_Boy_Wonder (~vossel@asterisk/batman-developer/dvossel) |
| 14:49.58 | *** join/#asterisk-bugs mvanbaak (~michiel@asterisk/contributor-and-bug-marshal/mvanbaak) |
| 14:49.58 | *** join/#asterisk-bugs seanbright (~sean@asterisk/contributor-and-bug-marshal/seanbright) |
| 14:49.58 | *** mode/#asterisk-bugs [+o seanbright] by anthony.freenode.net |
| 14:51.20 | *** join/#asterisk-bugs eliel (~eliels@201.234.94.226) |
| 14:52.51 | russellb | leifmadsen: ping |
| 14:58.22 | *** join/#asterisk-bugs jpeeler (~jpeeler@asterisk/digium-software-dev/jpeeler) |
| 15:01.46 | leifmadsen | russellb: pong |
| 15:04.35 | russellb | i retract my ping. |
| 15:04.59 | leifmadsen | I acknowledge your retraction without penalty |
| 15:05.03 | russellb | xlnt |
| 15:06.16 | leifmadsen | do you have an ETA on that bitch of an issue you're working on? |
| 15:08.00 | russellb | need to see what comments have been left in the last day |
| 15:08.03 | russellb | i'll probably commit today |
| 15:08.14 | leifmadsen | awesome, keep me posted and I'll look at creating RCs |
| 15:08.18 | russellb | will do |
| 15:08.21 | russellb | is that the last blocker? |
| 15:08.25 | leifmadsen | I believe so |
| 15:08.31 | leifmadsen | Matt's fax commit is already in |
| 15:09.15 | russellb | multiple success reports |
| 15:09.16 | russellb | w00t |
| 15:09.20 | leifmadsen | w00t! |
| 15:11.29 | pabelanger | m17262 |
| 15:11.31 | MuffinMan | [ready for review] [Asterisk] Applications/app_queue 0017262: [patch] "queue reset stats" erroneously clears wrapuptime configuration reported by rain https://issues.asterisk.org/view.php?id=17262 |
| 15:11.42 | pabelanger | safe to start merge process? |
| 15:12.12 | pabelanger | seem to be straight forward |
| 15:14.49 | russellb | note added |
| 15:15.21 | russellb | M17262#121417 |
| 15:15.23 | MuffinMan | [17262#0121417] Commented by russell on 2010-05-05 at 10:14: looks right to me, ship it |
| 15:15.38 | pabelanger | salutes russellb |
| 15:16.13 | russellb | at ease, sir |
| 15:23.36 | russellb | leifmadsen: i'm not going to be able to get that code in before all of these meetings and such ... |
| 15:23.39 | *** join/#asterisk-bugs corretico (~laguilar@201.201.46.106) |
| 15:23.40 | russellb | it may not be until this afternoon |
| 15:24.12 | pabelanger | russellb: same rule applies? If the problem exists in 1.4 and 1.6 branches, merge in the above patch? |
| 15:24.18 | russellb | correct |
| 15:24.22 | pabelanger | k |
| 15:24.24 | russellb | but in this case that feature is new |
| 15:24.26 | russellb | in 1.6.something ... |
| 15:24.33 | pabelanger | yar |
| 15:24.37 | russellb | can't remember exactly |
| 15:24.48 | russellb | the search is part of the fun, heh |
| 15:24.52 | russellb | runs off to his meeting |
| 15:42.09 | Entomologist | *** CLOSED (17262) [Applications/app_queue] [patch] "queue reset stats" erroneously clears wrapuptime configuration |
| 15:42.10 | Entomologist | Assigned to: pabelanger |
| 15:42.10 | Entomologist | Reported by: rain Karma: 2 |
| 15:42.11 | Entomologist | https://issues.asterisk.org/view.php?id=17262 |
| 15:42.11 | Entomologist | ********************************************************* |
| 15:43.13 | pabelanger | leifmadsen: question |
| 15:43.45 | pabelanger | m17262 |
| 15:43.47 | MuffinMan | [closed] [Asterisk] Applications/app_queue 0017262: [patch] "queue reset stats" erroneously clears wrapuptime configuration reported by rain https://issues.asterisk.org/view.php?id=17262 |
| 15:44.02 | pabelanger | should I have made my commit message also reference 16519? |
| 15:44.13 | pabelanger | Since the mantis issues we related. |
| 15:52.54 | Entomologist | *** CLOSED (17288) [Core/ManagerInterface] AMI Originate does not return a "Reason" when called with Async = 0 |
| 15:52.54 | Entomologist | Reported by: leoburd |
| 15:52.55 | Entomologist | https://issues.asterisk.org/view.php?id=17288 |
| 15:52.55 | Entomologist | ********************************************************* |
| 15:54.50 | leifmadsen | pabelanger: probably -- you can just reference issues with "Issue #16519" if you don't want the commit to close it |
| 15:55.47 | pabelanger | leifmadsen: ok, next time. Thanks. |
| 15:56.12 | leifmadsen | you can update the commit message if you really want :) |
| 15:56.44 | pabelanger | is (closes issue #17262, #16519) valid? Will it close both? |
| 15:58.27 | Entomologist | *** CLOSED (17285) [Channels/chan_h323] SIP Anonymous Call: h323 to SIP |
| 15:58.27 | Entomologist | Reported by: Steph63 |
| 15:58.28 | Entomologist | https://issues.asterisk.org/view.php?id=17285 |
| 15:58.28 | Entomologist | ********************************************************* |
| 16:03.41 | pabelanger | leifmadsen: not sure how to change the commit message after I have committed. I know of the future. |
| 16:04.05 | Entomologist | *** CLOSED (16519) [Applications/app_queue] Queue with wrapuptime "call" agent that shouldn't have any call |
| 16:04.05 | Entomologist | Reported by: benngard2 |
| 16:04.06 | Entomologist | https://issues.asterisk.org/view.php?id=16519 |
| 16:04.06 | Entomologist | ********************************************************* |
| 16:04.29 | leifmadsen | pabelanger: umm... not sure if that is valid... I think you're better off with it on different lines... |
| 16:04.35 | leifmadsen | might want to ask russell about that |
| 16:05.12 | pabelanger | leifmadsen: I think you are correct. When he gets back I will ask how to modify commit message, if it is required. |
| 16:07.15 | pabelanger | I have another question though. If I have a patch for 1.4, but it does not apply clean to trunk. Wouldn't you have to block the 1.4 reversion to stop it from merging into trunk? |
| 16:10.24 | leifmadsen | no, because things do not automerge |
| 16:11.27 | leifmadsen | so what you want to do is: merge to 1.4 --> merge4trunk --> fix conflicts --> commit --> mergetrunk6 --> fix conflicts --> commit |
| 16:12.00 | leifmadsen | if something only applies to 1.4, you don't have to block it from trunk |
| 16:12.55 | pabelanger | Ok. makes sense. |
| 16:16.55 | leifmadsen | that may change if the merge path gets switched to 1.4 -> 1.6.2 -> trunk |
| 16:21.44 | pabelanger | Ya, that is fine. I was just curious about how it is done. This next merge will have some conflicts between branches. |
| 16:24.22 | leifmadsen | gotcha |
| 16:24.28 | leifmadsen | ya, you just merge it, fix the comflicts, then commit |
| 16:42.25 | Entomologist | *** CLOSED (17266) [Channels/chan_sip/General] [patch] Failed to register peers from realtime config |
| 16:42.25 | Entomologist | Assigned to: pabelanger |
| 16:42.26 | Entomologist | Reported by: Nick_Lewis Karma: 8.25 |
| 16:42.26 | Entomologist | https://issues.asterisk.org/view.php?id=17266 |
| 16:42.27 | Entomologist | ********************************************************* |
| 16:50.54 | pabelanger | leifmadsen: when you run merge4trunk, how long does it usually take? |
| 16:51.04 | leifmadsen | ummm.... a few seconds? |
| 16:51.20 | pabelanger | heh. I'm pushing 4mins or so |
| 16:51.32 | pabelanger | I must be doing something wrong. |
| 16:52.04 | leifmadsen | should just be: merge4trunk ##### where ##### is the rev number |
| 16:52.11 | leifmadsen | I'd just cancel and try again |
| 16:52.18 | leifmadsen | there isn't much you can do "wrong" .... |
| 16:52.47 | Marquis42 | Famous last words... ;) |
| 16:53.15 | pabelanger | Did that, merge4trunk 261274 |
| 16:58.49 | *** join/#asterisk-bugs caio1982 (~caio1982@CAcert-br/caio1982) |
| 17:28.09 | pabelanger | leifmadsen: I am having some troubles getting this merge into trunk, have a minute? |
| 17:32.59 | leifmadsen | pabelanger: I will in a couple of mins |
| 17:33.35 | pabelanger | leifmadsen: thanks. I'm starting to think I did my previous commits wrong. |
| 18:10.22 | pabelanger | leifmadsen: I managed to get help from Corydon76-dig. Going to creating a mantis issue to update the developers documentation for merging. |
| 18:32.54 | leifmadsen | pabelanger: awesome thanks! |
| 18:53.00 | pabelanger | russellb: Sorry, looks like my commit affected bamboo |
| 18:53.21 | russellb | yes it did :-) |
| 18:53.25 | russellb | broke the world |
| 18:53.29 | russellb | not sure why yet |
| 18:53.39 | russellb | probably my config parser ... |
| 18:57.48 | *** part/#asterisk-bugs philipp64 (~chatzilla@mail.redfish-solutions.com) |
| 19:13.58 | Entomologist | *** CLOSED (17284) [Channels/chan_sip/Transfers] SIP attended transfer broken |
| 19:13.59 | Entomologist | Assigned to: dvossel |
| 19:13.59 | Entomologist | Reported by: dvossel Karma: 0 |
| 19:14.00 | Entomologist | https://issues.asterisk.org/view.php?id=17284 |
| 19:14.00 | Entomologist | ********************************************************* |
| 20:11.51 | pabelanger | leifmadsen: ping |
| 20:12.01 | leifmadsen | pabelanger: pong good sir? |
| 20:12.06 | pabelanger | m15671 |
| 20:12.09 | MuffinMan | [ready for review] [Asterisk] Channels/chan_dahdi 0015671: Stuck channel using FEATD_MF if caller hangs up at the right time reported by jcromes https://issues.asterisk.org/view.php?id=15671 |
| 20:12.27 | pabelanger | ok to downgrade, since we don' |
| 20:12.32 | pabelanger | 't actually have a patch? |
| 20:12.43 | pabelanger | Or ping reporter to create one? |
| 20:13.05 | leifmadsen | huh... how the heck did that happen? :) |
| 20:13.24 | leifmadsen | oh nevermind -- no need for a patch there |
| 20:13.27 | leifmadsen | it's a one line fix |
| 20:13.29 | leifmadsen | and trivial |
| 20:13.32 | pabelanger | k |
| 20:13.37 | leifmadsen | (he states it in the description |
| 20:13.38 | leifmadsen | ) |
| 20:13.43 | leifmadsen | brb, gotta go meet someone out front |
| 20:20.11 | pabelanger | leifmadsen: np. Created a patch, will get reported to retest and move ahead with merging proceedures. |
| 20:20.19 | leifmadsen | nice! |
| 20:25.15 | leifmadsen | pabelanger: ping? |
| 20:25.35 | pabelanger | leifmadsen: wong? |
| 20:25.44 | leifmadsen | :) |
| 20:26.09 | leifmadsen | your live_ast changes -- are those for running Asterisk as non-root generally, or for the test suite only? |
| 20:26.31 | leifmadsen | the reason I ask is because there has been some discussion about making Asterisk run as non-root by default and to make sure the documentation reflects that |
| 20:28.06 | pabelanger | Ya, it was an options for creating an installation of Asterisk for non-root, rather then creating a new script I was going to added it live_ast, but I revert it after talking to tzafrir. |
| 20:28.53 | pabelanger | However, I have been meaning to submit a menuselect option to allow a user to build and install Asterisk as non-root |
| 20:29.23 | pabelanger | Then setting the appropriate permissions for directories, etc. |
| 20:30.09 | pabelanger | Perhaps the best solutions for testsuite is add that functionality to Asterisk first, then have testsuite invoke it |
| 20:32.39 | leifmadsen | pabelanger: perhaps -- testsuite could just invoke it via a flag or something perhaps? |
| 20:33.24 | leifmadsen | pabelanger: I'd be VERY interested in figuring out the best way of installing Asterisk as non-root and having that done sooner rather than later because the documentation I'm working on kind of depends on it |
| 20:33.32 | leifmadsen | so anything I can do to help please let me know |
| 20:34.24 | pabelanger | sure. Let me do that first then, make Asterisk non-root options first. |
| 20:35.06 | leifmadsen | I kind of like the idea of it being in menuselect and having the default be "enabled" |
| 20:35.17 | leifmadsen | then the documentation becomes trivial |
| 20:35.24 | russellb | pabelanger: can we change asterisk.conf back to where the directories aren't commented out, and the section is just a template? |
| 20:35.28 | russellb | please? :-) |
| 20:37.10 | leifmadsen | does it wreak havoc on bamboo? :) |
| 20:37.11 | pabelanger | russellb: sure, if needed. I only make the change because somebody else requested it. |
| 20:37.36 | russellb | well it breaks a lot of tests, as I was using asterisk.conf to discover the install structure |
| 20:38.05 | russellb | i mean, i know the info is still there ... but i'd have to add special handling in the config parser to preserve and parse contents of comments |
| 20:38.07 | russellb | blargh |
| 20:40.47 | russellb | pabelanger: http://pastebin.com/wKFdpwnw |
| 20:40.51 | russellb | that look right to you? |
| 20:42.27 | pabelanger | russellb: testing now |
| 20:47.35 | pabelanger | russellb: looks good |
| 20:47.39 | russellb | kthx |
| 21:12.54 | pabelanger | leifmadsen: What is the policy for back porting feature into 1.4? |
| 21:13.19 | leifmadsen | pabelanger: get russellb to sign off on it, and you can do anything you want :) |
| 21:13.20 | pabelanger | no new features still applies? |
| 21:13.23 | leifmadsen | yes |
| 21:13.27 | leifmadsen | no new features unless it resolves a bug |
| 21:13.52 | pabelanger | that is what I thought. |
| 21:13.54 | pabelanger | m17292 |
| 21:13.56 | MuffinMan | [new] [Asterisk] Applications/app_queue 0017292: [patch] [backport for 1.4] Autopause only pauses member on queue where timeout took place reported by nik600 https://issues.asterisk.org/view.php?id=17292 |
| 21:14.04 | pabelanger | So, it is unlikely we will port this back? |
| 21:15.06 | leifmadsen | correct -- that can probably be closed because putnopvut put it only in trunk for a reason |
| 21:15.09 | russellb | denied. |
| 21:15.10 | russellb | :-) |
| 21:15.17 | leifmadsen | NEXT!!! |
| 21:15.26 | leifmadsen | you may close the issue |
| 21:15.33 | pabelanger | All right! Denied message coming up! ;) |
| 21:16.49 | Entomologist | *** CLOSED (17292) [Applications/app_queue] [patch] [backport for 1.4] Autopause only pauses member on queue where timeout took place |
| 21:16.49 | Entomologist | Reported by: nik600 |
| 21:16.50 | Entomologist | https://issues.asterisk.org/view.php?id=17292 |
| 21:16.50 | Entomologist | ********************************************************* |
| 21:19.40 | pabelanger | Ok, time to head home. I grabbed a few 'ready for reviews', started to clean them up and get sign off for merges. They look to be trivial changes. |
| 21:19.48 | pabelanger | Thanks again for the help today. |
| 21:19.59 | russellb | thank you for the help! |
| 21:20.06 | pabelanger-away | russellb: np[ |
| 23:35.22 | *** join/#asterisk-bugs seanbright (~sean@asterisk/contributor-and-bug-marshal/seanbright) |
| 23:35.22 | *** mode/#asterisk-bugs [+o seanbright] by ChanServ |
| 23:56.31 | *** join/#asterisk-bugs Deeewayne (~dwayne@c-71-207-214-190.hsd1.al.comcast.net) |
| 23:56.31 | *** mode/#asterisk-bugs [+o Deeewayne] by ChanServ |