From 0a6ec03bdf7f083124d081b78fb34780b49c1a18 Mon Sep 17 00:00:00 2001 From: Christoph Willing Date: Wed, 10 Jan 2018 18:49:47 +1000 Subject: multimedia/vlc: Patched to support libupnp-1.8 Signed-off-by: Christoph Willing --- multimedia/vlc/patch-libupnp.diff | 194 ++++++++++++++++++++++++++++++++++++++ multimedia/vlc/vlc.SlackBuild | 3 +- 2 files changed, 196 insertions(+), 1 deletion(-) create mode 100644 multimedia/vlc/patch-libupnp.diff (limited to 'multimedia/vlc') diff --git a/multimedia/vlc/patch-libupnp.diff b/multimedia/vlc/patch-libupnp.diff new file mode 100644 index 0000000000..2e0580b810 --- /dev/null +++ b/multimedia/vlc/patch-libupnp.diff @@ -0,0 +1,194 @@ +From 3eb4e03512f45c1fa27c7f9a6759e8e7d3905720 Mon Sep 17 00:00:00 2001 +From: Sebastian Ramacher +Date: Tue, 29 Aug 2017 23:10:15 +0200 +Subject: [PATCH 1/1] upnp: Add support for libupnp 1.8 + +Callbacks now take const void* as second argument and some members can +only be accessed via getter functions. + +Signed-off-by: Sebastian Ramacher +Signed-off-by: Jean-Baptiste Kempf +--- + modules/services_discovery/upnp.cpp | 48 +++++++++++++++++++++++++++---------- + modules/services_discovery/upnp.hpp | 14 +++++++---- + 2 files changed, 46 insertions(+), 16 deletions(-) + +diff --git a/modules/services_discovery/upnp.cpp b/modules/services_discovery/upnp.cpp +index c21b2b1e37..bdd3c55ee5 100644 +--- a/modules/services_discovery/upnp.cpp ++++ b/modules/services_discovery/upnp.cpp +@@ -38,6 +38,30 @@ + #include + #include + ++#if UPNP_VERSION < 10800 ++/* ++ * Compat functions and typedefs for libupnp prior to 1.8 ++ */ ++ ++typedef Upnp_Discovery UpnpDiscovery; ++typedef Upnp_Action_Complete UpnpActionComplete; ++ ++static const char* UpnpDiscovery_get_Location_cstr( const UpnpDiscovery* p_discovery ) ++{ ++ return p_discovery->Location; ++} ++ ++static const char* UpnpDiscovery_get_DeviceID_cstr( const UpnpDiscovery* p_discovery ) ++{ ++ return p_discovery->DeviceId; ++} ++ ++static IXML_Document* UpnpActionComplete_get_ActionResult( const UpnpActionComplete* p_result ) ++{ ++ return p_result->ActionResult; ++} ++#endif ++ + /* + * Constants + */ +@@ -679,19 +703,19 @@ void MediaServerList::removeServer( const std::string& udn ) + /* + * Handles servers listing UPnP events + */ +-int MediaServerList::Callback( Upnp_EventType event_type, void* p_event ) ++int MediaServerList::Callback( Upnp_EventType event_type, UpnpEventPtr p_event ) + { + switch( event_type ) + { + case UPNP_DISCOVERY_ADVERTISEMENT_ALIVE: + case UPNP_DISCOVERY_SEARCH_RESULT: + { +- struct Upnp_Discovery* p_discovery = ( struct Upnp_Discovery* )p_event; ++ const UpnpDiscovery* p_discovery = ( const UpnpDiscovery* )p_event; + + IXML_Document *p_description_doc = NULL; + + int i_res; +- i_res = UpnpDownloadXmlDoc( p_discovery->Location, &p_description_doc ); ++ i_res = UpnpDownloadXmlDoc( UpnpDiscovery_get_Location_cstr( p_discovery ), &p_description_doc ); + + MediaServerList *self = UpnpInstanceWrapper::lockMediaServerList(); + if ( !self ) +@@ -704,11 +728,11 @@ int MediaServerList::Callback( Upnp_EventType event_type, void* p_event ) + { + msg_Warn( self->m_sd, "Could not download device description! " + "Fetching data from %s failed: %s", +- p_discovery->Location, UpnpGetErrorMessage( i_res ) ); ++ UpnpDiscovery_get_Location_cstr( p_discovery ), UpnpGetErrorMessage( i_res ) ); + UpnpInstanceWrapper::unlockMediaServerList(); + return i_res; + } +- self->parseNewServer( p_description_doc, p_discovery->Location ); ++ self->parseNewServer( p_description_doc, UpnpDiscovery_get_Location_cstr( p_discovery ) ); + UpnpInstanceWrapper::unlockMediaServerList(); + ixmlDocument_free( p_description_doc ); + } +@@ -716,11 +740,11 @@ int MediaServerList::Callback( Upnp_EventType event_type, void* p_event ) + + case UPNP_DISCOVERY_ADVERTISEMENT_BYEBYE: + { +- struct Upnp_Discovery* p_discovery = ( struct Upnp_Discovery* )p_event; ++ const UpnpDiscovery* p_discovery = ( const UpnpDiscovery* )p_event; + + MediaServerList *self = UpnpInstanceWrapper::lockMediaServerList(); + if ( self ) +- self->removeServer( p_discovery->DeviceId ); ++ self->removeServer( UpnpDiscovery_get_DeviceID_cstr( p_discovery ) ); + UpnpInstanceWrapper::unlockMediaServerList(); + } + break; +@@ -953,7 +977,7 @@ void Upnp_i11e_cb::waitAndRelease( void ) + } + } + +-int Upnp_i11e_cb::run( Upnp_EventType eventType, void *p_event, void *p_cookie ) ++int Upnp_i11e_cb::run( Upnp_EventType eventType, UpnpEventPtr p_event, void *p_cookie ) + { + Upnp_i11e_cb *self = static_cast( p_cookie ); + +@@ -1082,15 +1106,15 @@ bool MediaServer::addItem( IXML_Element* itemElement ) + } + + int MediaServer::sendActionCb( Upnp_EventType eventType, +- void *p_event, void *p_cookie ) ++ UpnpEventPtr p_event, void *p_cookie ) + { + if( eventType != UPNP_CONTROL_ACTION_COMPLETE ) + return 0; + IXML_Document** pp_sendActionResult = (IXML_Document** )p_cookie; +- Upnp_Action_Complete *p_result = (Upnp_Action_Complete *)p_event; ++ const UpnpActionComplete *p_result = (const UpnpActionComplete *)p_event; + + /* The only way to dup the result is to print it and parse it again */ +- DOMString tmpStr = ixmlPrintNode( ( IXML_Node * ) p_result->ActionResult ); ++ DOMString tmpStr = ixmlPrintNode( ( IXML_Node * ) UpnpActionComplete_get_ActionResult( p_result ) ); + if (tmpStr == NULL) + return 0; + +@@ -1616,7 +1640,7 @@ UpnpClient_Handle UpnpInstanceWrapper::handle() const + return m_handle; + } + +-int UpnpInstanceWrapper::Callback(Upnp_EventType event_type, void *p_event, void *p_user_data) ++int UpnpInstanceWrapper::Callback(Upnp_EventType event_type, UpnpEventPtr p_event, void *p_user_data) + { + VLC_UNUSED(p_user_data); + vlc_mutex_lock( &s_lock ); +diff --git a/modules/services_discovery/upnp.hpp b/modules/services_discovery/upnp.hpp +index 033e191aea..39e19cbd56 100644 +--- a/modules/services_discovery/upnp.hpp ++++ b/modules/services_discovery/upnp.hpp +@@ -37,6 +37,12 @@ + #include + #include + ++#if UPNP_VERSION < 10800 ++typedef void* UpnpEventPtr; ++#else ++typedef const void* UpnpEventPtr; ++#endif ++ + namespace SD + { + class MediaServerList; +@@ -63,7 +69,7 @@ public: + static void unlockMediaServerList(); + + private: +- static int Callback( Upnp_EventType event_type, void* p_event, void* p_user_data ); ++ static int Callback( Upnp_EventType event_type, UpnpEventPtr p_event, void* p_user_data ); + + UpnpInstanceWrapper(); + ~UpnpInstanceWrapper(); +@@ -104,7 +110,7 @@ public: + bool addServer(MediaServerDesc *desc ); + void removeServer(const std::string &udn ); + MediaServerDesc* getServer( const std::string& udn ); +- static int Callback( Upnp_EventType event_type, void* p_event ); ++ static int Callback( Upnp_EventType event_type, UpnpEventPtr p_event ); + + private: + void parseNewServer( IXML_Document* doc, const std::string& location ); +@@ -126,7 +132,7 @@ public: + Upnp_i11e_cb( Upnp_FunPtr callback, void *cookie ); + ~Upnp_i11e_cb(); + void waitAndRelease( void ); +- static int run( Upnp_EventType, void *, void *); ++ static int run( Upnp_EventType, UpnpEventPtr, void *); + + private: + vlc_sem_t m_sem; +@@ -152,7 +158,7 @@ private: + + IXML_Document* _browseAction(const char*, const char*, + const char*, const char*, const char* ); +- static int sendActionCb( Upnp_EventType, void *, void *); ++ static int sendActionCb( Upnp_EventType, UpnpEventPtr, void *); + + private: + char* m_psz_root; +-- +2.11.0 + diff --git a/multimedia/vlc/vlc.SlackBuild b/multimedia/vlc/vlc.SlackBuild index 8b6cd299f4..2b86dde423 100644 --- a/multimedia/vlc/vlc.SlackBuild +++ b/multimedia/vlc/vlc.SlackBuild @@ -29,7 +29,7 @@ PRGNAM=vlc VERSION=${VERSION:-20170523_bec643e} -BUILD=${BUILD:-1} +BUILD=${BUILD:-2} TAG=${TAG:-_SBo} if [ -z "$ARCH" ]; then @@ -82,6 +82,7 @@ cd $PRGNAM-$VERSION patch -p0 < $CWD/patch-projectM-fontpath.diff patch -p0 < $CWD/patch_vlc_cache_gen.diff +patch -p1 < $CWD/patch-libupnp.diff autoreconf -fiv -- cgit v1.2.3