Yggdrasil icon indicating copy to clipboard operation
Yggdrasil copied to clipboard

Add recipe for pthread-win32

Open sjkelly opened this issue 3 years ago • 6 comments

This adds pthread-win32. The install step is currently broken, and will need to be fixed...

sjkelly avatar Jul 26 '22 16:07 sjkelly

Rebased and fixed; builds locally.

maleadt avatar Nov 15 '22 21:11 maleadt

Should probably rename this to pthreads_win32, note the trailing s. An alternative name used in the upstream repo is pthreads4w. Or, maybe we should just name this pthreads, in the case we'd also want to provide libpthread.so for other platforms?

maleadt avatar Nov 16 '22 15:11 maleadt

The DLL generated here also doesn't contain a single pthread_ API call, so something is wrong here.

maleadt avatar Nov 16 '22 16:11 maleadt

Don't we build pthreads for out mingw cross compiler?

On Wed, Nov 16, 2022, 10:14 Tim Besard @.***> wrote:

The DLL generated here also doesn't contain a single pthread_ API call, so something is wrong here.

— Reply to this email directly, view it on GitHub https://github.com/JuliaPackaging/Yggdrasil/pull/5234#issuecomment-1317273698, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABDO2XTSMFK3ZK65EDRBIDWIUB63ANCNFSM54WTM2LQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

vchuravy avatar Nov 16 '22 16:11 vchuravy

It should be included in a mingw distribution, yes, but which JLL contains that?

maleadt avatar Nov 16 '22 16:11 maleadt

It should be included in a mingw distribution, yes, but which JLL contains that?

None, it's in the compiler shard for windows.

giordano avatar Nov 24 '22 00:11 giordano

Mingw contains winpthreads and this is indeed installed in our compiler shards.

The last comment here was three years ago, and the need of this is unclear - close it?

fingolfin avatar Nov 25 '25 06:11 fingolfin