After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 656030 - Add Configurable Multicast Address Specification
Add Configurable Multicast Address Specification
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-rtsp-server
0.10.x
Other Linux
: Normal normal
: 0.10.9
Assigned To: GStreamer Maintainers
GStreamer Maintainers
: 653317 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-08-05 13:52 UTC by Robert Krakora
Modified: 2011-08-16 12:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add Configurable Multicast Address Specification (14.01 KB, application/octet-stream)
2011-08-05 13:52 UTC, Robert Krakora
  Details
Modified previous patch to account for change needed for SDP... (15.20 KB, patch)
2011-08-08 15:29 UTC, Robert Krakora
none Details | Review
Add Configurable Multicast Address Specification - Patch Revised (15.20 KB, patch)
2011-08-08 19:56 UTC, Robert Krakora
none Details | Review
Modified previous patch to account for bug introduced by last patch... (15.10 KB, patch)
2011-08-09 14:10 UTC, Robert Krakora
none Details | Review

Description Robert Krakora 2011-08-05 13:52:45 UTC
Created attachment 193310 [details]
Add Configurable Multicast Address Specification

I had exchanged e-mails with Wim last year on adding configurable multicast address specification to the RTSP Server.  I got real busy and did not have time to make the change until this week.  I hope this is what he was envisioning.
Comment 1 Robert Krakora 2011-08-08 15:29:40 UTC
Created attachment 193423 [details] [review]
Modified previous patch to account for change needed for SDP...

Modified previous patch to account for change needed for SDP...
Comment 2 Robert Krakora 2011-08-08 19:56:29 UTC
Created attachment 193444 [details] [review]
Add Configurable Multicast Address Specification - Patch Revised

test-launch example had hardcoded IP address left in it from debugging...all should be fine now...
Comment 3 Robert Krakora 2011-08-09 14:10:27 UTC
Created attachment 193486 [details] [review]
Modified previous patch to account for bug introduced by last patch...

A bug was uncovered in the previous patch and has now been fixed.  It had to do with launch lines containing multiple payloads.
Comment 4 Wim Taymans 2011-08-16 10:50:43 UTC
*** Bug 653317 has been marked as a duplicate of this bug. ***
Comment 5 Wim Taymans 2011-08-16 12:57:32 UTC
commit 56a16f9f5a67c05f74b8c2ebf386f598280d652e
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Tue Aug 16 13:43:44 2011 +0200

    client: use media multicast group

commit 2c9701bd73686be01143c73feb6b095ccd03eb20
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Tue Aug 16 13:37:50 2011 +0200

    retab some .h

commit a5e028ba72a195aa35f220b8ba7d49398f5995e0
Author: Robert Krakora <rob.krakora at messagenetsystems.com>
Date:   Tue Aug 16 13:31:52 2011 +0200

    sdp: copy and free the server ip address
    
    Copy and free the server ip address to make memory management easier later.

commit 647e8c7af8fed431f2904e1bd7c0d2612092b995
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Tue Aug 16 13:27:39 2011 +0200

    media-factory: configure multicast in media

commit c0793251698f91982002c6e8d3f450e8c1270822
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Tue Aug 16 13:25:16 2011 +0200

    media: add property for multicast group
    
    Add a property to configure the multicast group in the media.
    
    Based on patches from Marc Leeman and Robert Krakora.

commit 514728864a34459b52a48e8bd805cde0dc2a3685
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Tue Aug 16 13:13:36 2011 +0200

    media-factory: add property for multicast group
    
    Add a property to configure the multicast group in the media factory.
    
    Based on patches from Marc Leeman and Robert Krakora.

commit b881dc66690f81b5928f859e43c202440b85ab97
Author: Wim Taymans <wim.taymans@collabora.co.uk>
Date:   Tue Aug 16 12:51:44 2011 +0200

    client: do configuration of transport in one place
    
    Move the configuration of the transport destination address to where we also
    configure the other bits.