Skip to content

Commit

Permalink
Fix stubs for non-glibc (#59)
Browse files Browse the repository at this point in the history
`caml/signals.h` is not public API; it does not include `caml/config.h`
and so it will not include `signal.h` because it does not know about
`POSIX_SIGNAL`. This issue is visible on musl as "unknown type
sigset_t".

Signed-off-by: Etienne Millon <me@emillon.org>
  • Loading branch information
emillon authored Mar 26, 2024
1 parent fa620a8 commit 784f53c
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions src/spawn_stubs.c
Original file line number Diff line number Diff line change
Expand Up @@ -15,17 +15,15 @@
#endif
#endif

/* for [caml_convert_signal_number] */
#include <caml/signals.h>

#undef CAML_INTERNALS

#include <caml/mlvalues.h>
#include <caml/memory.h>
#include <caml/alloc.h>
#include <caml/unixsupport.h>
#include <caml/fail.h>

/* for [caml_convert_signal_number]; must come after public caml headers */
#include <caml/signals.h>

#include <errno.h>

#if defined(__APPLE__)
Expand Down

0 comments on commit 784f53c

Please sign in to comment.