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

[Peter Thatcher <[e-mail]>]

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]
----------------------------------------------------------------------