[vlc-devel] [PATCH 1/2] contribs: pthreads: also build the CRT headers

Martin Storsjö martin at martin.st
Thu Oct 4 15:15:24 CEST 2018


On Thu, 4 Oct 2018, Steve Lhomme wrote:

> ---
> contrib/src/pthreads/rules.mak | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/contrib/src/pthreads/rules.mak b/contrib/src/pthreads/rules.mak
> index b7d2b57681e..9d762466286 100644
> --- a/contrib/src/pthreads/rules.mak
> +++ b/contrib/src/pthreads/rules.mak
> @@ -18,6 +18,8 @@ pthreads: mingw-w64-v$(WINPTHREADS_VERSION).tar.bz2 .sum-pthreads
> 	$(MOVE)
> 
> .pthreads: pthreads
> +	cd $</mingw-w64-headers && $(HOSTVARS) ./configure $(HOSTCONF)
> +	cd $< && $(MAKE) -C mingw-w64-headers
> 	cd $</mingw-w64-libraries/winpthreads && $(HOSTVARS) ./configure $(HOSTCONF)
> 	cd $< && $(MAKE) -C mingw-w64-libraries -C winpthreads install
> 	touch $@
> -- 
> 2.19.0

Please no, this isn't the right solution for the issue we're discussion.

For cases where the mingw-w64 headers and import libraries use a different 
default msvcrt (ucrtbsae), this will override that and break everything. 
The mingw-w64 headers and import libraries should really come from the 
same version, otherwise many things will break.

// Martin



More information about the vlc-devel mailing list