Re: [rtcweb] Pull request: JSEP-19 Appendix B text moved to BUNDLE

[Peter Thatcher <[e-mail]>]

SGTM

On Mon, Mar 27, 2017 at 6:47 PM Christer Holmberg <
[e-mail]> wrote:

> Hi,
>
>
>
> My intention is to merge YOUR BUNDLE PR (Option B). I will close my PR.
>
>
>
> But, I don’t think we should work on the JSEP PRs anymore, because they
> will not be automatically reflected in your BUNDLE PR.
>
>
>
> Regards,
>
>
>
> Christer
>
>
>
> *From:* Peter Thatcher [mailto:[e-mail]]
> *Sent:* 28 March 2017 04:33
> *To:* Christer Holmberg <[e-mail]>; Magnus
> Westerlund <[e-mail]>; Justin Uberti <
> [e-mail]>
>
>
> *Cc:* [e-mail]; [e-mail]; mmusic <
> [e-mail]>; [e-mail]
> *Subject:* Re: [rtcweb] Pull request: JSEP-19 Appendix B text moved to
> BUNDLE
>
>
>
> There are a few options for merging:
>
>
>
> Option A:
>
>
>
> 1.  Merge your BUNDLE PR (
> https://github.com/cdh4u/draft-sdp-bundle/pull/28/) without the updates
> about RTCP BYEs and PT demux limitations.
>
> 2.  Create a subsequent PR or two for the updates about RTCP BYEs and PT
> demux limitations.
>
>
>
>
>
> Options B:
>
>
>
> Merge my BUNDLE PR (https://github.com/cdh4u/draft-sdp-bundle/pull/29)
> that has the updates about RTCP BYEs and PT demux.
>
>
>
> Option C:
>
>
>
> 1. Merge the two JSEP PRs (
> https://github.com/rtcweb-wg/jsep/pull/627/files,
> https://github.com/rtcweb-wg/jsep/pull/628/files)
>
>
>
> 2.  Update your BUNDLE PR (
> https://github.com/cdh4u/draft-sdp-bundle/pull/28/) to be include those
> two PRs (at which point it looks just like mine).
>
>
>
>
>
>
>
> I think option A makes the most sense.
>
>
>
>
>
> On Sun, Mar 26, 2017 at 10:34 AM Christer Holmberg <
> [e-mail]> wrote:
>
> Hold on...
>
> Which PR are we going to work on?
>
> The JSEP PR mentioned below, or the BUNDLE PR created by Peter?
>
> https://github.com/cdh4u/draft-sdp-bundle/pull/29/
>
> Since everyone seem to agree that the text shall be moved to BUNDLE, my
> suggestion is to work on the BUNDLE PR.
>
> Regards,
>
> Christer
>
>
> -----Original Message-----
> From: Magnus Westerlund
> Sent: 26 March 2017 18:32
> To: Peter Thatcher <[e-mail]>; Christer Holmberg <
> [e-mail]>; Justin Uberti <[e-mail]>
> Cc: [e-mail]; [e-mail]; mmusic <
> [e-mail]>; [e-mail]
> Subject: Re: [rtcweb] Pull request: JSEP-19 Appendix B text moved to BUNDLE
>
> Den 2017-03-24 kl. 16:43, skrev Peter Thatcher:
> > FYI, there may have been another blocker in that Magnus was asking for
> > the PT-based demux limitation of SSRCs not being able to "move"
> > between m-lines to be more explicit.  If the WG(s) agree(s) that we
> > should address this, I have made JSEP PR to address it:
> >
> > https://github.com/rtcweb-wg/jsep/pull/628
> >
> > Again, if we want, I can make a BUNDLE version of the PR.
> >
>
> To be clear I am all for moving appendix B to BUNDLE spec at this point.
>
> I have commenteded on the pull request.
>
> Cheers
>
> Magnus Westerlund
>
> ----------------------------------------------------------------------
> Media Technologies, Ericsson Research
> ----------------------------------------------------------------------
> Ericsson AB                 | Phone  [phone]
> <+46%2010%20714%2082%2087>
> Färögatan 6                 | Mobile [phone]
> <+46%2073%20094%2090%2079>
> SE-164 80 Stockholm, Sweden | mailto: [e-mail]
> ----------------------------------------------------------------------
>
>