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

[Christer Holmberg <[e-mail]>]

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]<mailto:[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]<mailto:[e-mail]>>; Christer Holmberg <[e-mail]<mailto:[e-mail]>>; Justin Uberti <[e-mail]<mailto:[e-mail]>>
Cc: [e-mail]<mailto:[e-mail]>; [e-mail]<mailto:[e-mail]>; mmusic <[e-mail]<mailto:[e-mail]>>; [e-mail]<mailto:[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]<tel:+46%2010%20714%2082%2087>
Färögatan 6                 | Mobile [phone]<tel:+46%2073%20094%2090%2079>
SE-164 80 Stockholm, Sweden | mailto: [e-mail]<mailto:[e-mail]>
----------------------------------------------------------------------