Summary: | mail-mta/nullmailer with sys-libs/musl - missing include to unistd.h for uid_t | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Fredric Johansson <fredric.miscmail> |
Component: | Current packages | Assignee: | Robin Johnson <robbat2> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | blueness, jlec, net-mail+disabled |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 430702 | ||
Attachments: | Fix for building on musl |
Comment on attachment 395352 [details, diff]
Fix for building on musl
I don't know how you generate these, but entire ChangeLogs should definitely not be included in your patches.
Need a clean patch, otherwise I cannot support this. (In reply to Justin Lecher from comment #2) > Need a clean patch, otherwise I cannot support this. I think he's asking me to apply that to the hadened-dev::musl overlay. Although the changelog is not necessary for sure, I do appreciate the clean application to the overlay rather than piecing stuff in. Anyhow, I already have a pull request upstream/github for nullmailer. https://github.com/bruceg/nullmailer/pull/25 The problem is pretty much a non-brainer. Header stacking in glibc pulls in typedefs of uid_t and gid_t without directly including <unistd.h>. This breaks posix and is not portable. (In reply to Anthony Basile from comment #3) > The problem is pretty much a non-brainer. Header stacking in glibc pulls in > typedefs of uid_t and gid_t without directly including <unistd.h>. This > breaks posix and is not portable. I assume this completely compatible with glibc and other libc implementations? + 02 Feb 2015; Justin Lecher <jlec@gentoo.org> nullmailer-1.13-r5.ebuild, + +files/nullmailer-1.13-unistd.h.patch: + Import header in portable way, #538482 + *** Bug 542484 has been marked as a duplicate of this bug. *** |
Created attachment 395352 [details, diff] Fix for building on musl nullmailer fails to build against musl. missing include to unistd.h for uid_t Attaching patch for hardened-dev:musl