[vlc-devel] [PATCH] services_discovery: upnp: Fix libupnp header includes

Hugo Beauzée-Luyssen hugo at beauzee.fr
Tue Jul 31 14:25:26 CEST 2018


On Sat, Jul 28, 2018, at 5:56 PM, Marvin Scholz wrote:
> The libupnp .pc file uses -I${includedir}/upnp which means that the
> way the headers were included would not result in them actually being
> found in all cases due to the extraneous "upnp".
> ---
>  modules/services_discovery/upnp-wrapper.hpp | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/modules/services_discovery/upnp-wrapper.hpp b/modules/
> services_discovery/upnp-wrapper.hpp
> index ebae8fc96f..c62749214e 100644
> --- a/modules/services_discovery/upnp-wrapper.hpp
> +++ b/modules/services_discovery/upnp-wrapper.hpp
> @@ -31,8 +31,8 @@
>  #include <algorithm>
>  #include <assert.h>
>  
> -#include <upnp/upnp.h>
> -#include <upnp/upnptools.h>
> +#include <upnp.h>
> +#include <upnptools.h>
>  
>  #if UPNP_VERSION < 10800
>  typedef void* UpnpEventPtr;
> -- 
> 2.15.1 (Apple Git-101)
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel

LGTM!

-- 
  Hugo Beauzée-Luyssen
  hugo at beauzee.fr


More information about the vlc-devel mailing list