Summary: | sys-libs/tevent : /.../tevent.h: error: unknown type name pid_t | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Toralf Förster <toralf> |
Component: | Current packages | Assignee: | Gentoo's SAMBA Team <samba> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | bkohler, fsvm88, hougelangley1987, sam, samba |
Priority: | Normal | Keywords: | PATCH |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 713786 | ||
Attachments: |
emerge-info.txt
emerge-history.txt environment etc.portage.tbz2 logs.tbz2 sys-libs:ldb-2.1.1:20200325-064357.log temp.tbz2 sys-libs/tevent: include sys/types.h if pid_t undefined |
Description
Toralf Förster
2020-03-25 08:17:03 UTC
Created attachment 625696 [details]
emerge-info.txt
Created attachment 625698 [details]
emerge-history.txt
Created attachment 625700 [details]
environment
Created attachment 625702 [details]
etc.portage.tbz2
Created attachment 625704 [details]
logs.tbz2
Created attachment 625706 [details]
sys-libs:ldb-2.1.1:20200325-064357.log
Created attachment 625708 [details]
temp.tbz2
Created attachment 625880 [details, diff]
sys-libs/tevent: include sys/types.h if pid_t undefined
tevent.h makes use of pid_t without including it on musl.
Fix tevent (the real issue) instead of fixing ldb.
The attached patch makes sys-libs/ldb compile properly.
alpine disable a test but not seemingly this one: https://git.alpinelinux.org/aports/tree/main/ldb/disable-compile-error-test.patch (In reply to Fabio Scaccabarozzi from comment #8) > Created attachment 625880 [details, diff] [details, diff] > sys-libs/tevent: include sys/types.h if pid_t undefined > > tevent.h makes use of pid_t without including it on musl. > Fix tevent (the real issue) instead of fixing ldb. > The attached patch makes sys-libs/ldb compile properly. No need to ifdef, you can just include the header. diff --git a/tevent.h b/tevent.h index 3c3e3cc..011e1ad 100644 --- a/tevent.h +++ b/tevent.h @@ -31,6 +31,7 @@ #include <stdint.h> #include <talloc.h> #include <sys/time.h> +#include <sys/types.h> #include <stdbool.h> struct tevent_context; *** Bug 821691 has been marked as a duplicate of this bug. *** Workaround here too: https://bugs.gentoo.org/821691#c4 Stable tevent seems to already be including types.h as mentioend in comment #10 here, closing |