[asterisk-dev] [Code Review] 3910: Fix test failures introduced by 420934
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3910/ --- Review request for Asterisk Developers, Matt Jordan and rmudgett. Bugs: ASTERISK-24027 https://issues.asterisk.org/jira/browse/ASTERISK-24027 Repository: Asterisk Description --- r420934 changed ast_channel_is_leaving bridge to not care about the unbridge flag (which is no longer a softhangup flag due to that causing the behavior noted in ASTERISK-24027). This patch makes it so that the unbridge flag is evaluated again since that caused an unexpected change in how media would be re-established when a bridge starts to get torn down on hangup. Diffs - /branches/12/main/channel.c 420934 Diff: https://reviewboard.asterisk.org/r/3910/diff/ Testing --- Ran the failing tests from https://issues.asterisk.org/jira/browse/AFS-137. I couldn't get all of them to pass prior to r420934, but the ones that I could run successfully were confirmed broken by r420934 and fixed by this patch. Thanks, Jonathan Rose -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [asterisk-dev] [Code Review] 3910: Fix test failures introduced by 420934
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3910/ --- (Updated Aug. 14, 2014, 1:28 p.m.) Review request for Asterisk Developers, Matt Jordan and rmudgett. Changes --- Oops, copied the wrong link. Bugs: ASTERISK-24027 https://issues.asterisk.org/jira/browse/ASTERISK-24027 Repository: Asterisk Description --- r420934 changed ast_channel_is_leaving bridge to not care about the unbridge flag (which is no longer a softhangup flag due to that causing the behavior noted in ASTERISK-24027). This patch makes it so that the unbridge flag is evaluated again since that caused an unexpected change in how media would be re-established when a bridge starts to get torn down on hangup. Diffs - /branches/12/main/channel.c 420934 Diff: https://reviewboard.asterisk.org/r/3910/diff/ Testing (updated) --- Ran the failing tests from https://bamboo.asterisk.org/bamboo/browse/AST-ATSF-C632TE-336. I couldn't get all of them to pass prior to r420934, but the ones that I could run successfully were confirmed broken by r420934 and fixed by this patch. Thanks, Jonathan Rose -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [asterisk-dev] [Code Review] 3910: Fix test failures introduced by 420934
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3910/#review13086 --- Ship it! Ship It! - rmudgett On Aug. 14, 2014, 1:28 p.m., Jonathan Rose wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3910/ --- (Updated Aug. 14, 2014, 1:28 p.m.) Review request for Asterisk Developers, Matt Jordan and rmudgett. Bugs: ASTERISK-24027 https://issues.asterisk.org/jira/browse/ASTERISK-24027 Repository: Asterisk Description --- r420934 changed ast_channel_is_leaving bridge to not care about the unbridge flag (which is no longer a softhangup flag due to that causing the behavior noted in ASTERISK-24027). This patch makes it so that the unbridge flag is evaluated again since that caused an unexpected change in how media would be re-established when a bridge starts to get torn down on hangup. Diffs - /branches/12/main/channel.c 420934 Diff: https://reviewboard.asterisk.org/r/3910/diff/ Testing --- Ran the failing tests from https://bamboo.asterisk.org/bamboo/browse/AST-ATSF-C632TE-336. I couldn't get all of them to pass prior to r420934, but the ones that I could run successfully were confirmed broken by r420934 and fixed by this patch. Thanks, Jonathan Rose -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [asterisk-dev] [Code Review] 3910: Fix test failures introduced by 420934
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3910/#review13087 --- Ship it! Ship It! - Matt Jordan On Aug. 14, 2014, 1:28 p.m., Jonathan Rose wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3910/ --- (Updated Aug. 14, 2014, 1:28 p.m.) Review request for Asterisk Developers, Matt Jordan and rmudgett. Bugs: ASTERISK-24027 https://issues.asterisk.org/jira/browse/ASTERISK-24027 Repository: Asterisk Description --- r420934 changed ast_channel_is_leaving bridge to not care about the unbridge flag (which is no longer a softhangup flag due to that causing the behavior noted in ASTERISK-24027). This patch makes it so that the unbridge flag is evaluated again since that caused an unexpected change in how media would be re-established when a bridge starts to get torn down on hangup. Diffs - /branches/12/main/channel.c 420934 Diff: https://reviewboard.asterisk.org/r/3910/diff/ Testing --- Ran the failing tests from https://bamboo.asterisk.org/bamboo/browse/AST-ATSF-C632TE-336. I couldn't get all of them to pass prior to r420934, but the ones that I could run successfully were confirmed broken by r420934 and fixed by this patch. Thanks, Jonathan Rose -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
