Bug 216966
| Summary: | [exp-run] Additon of RLIM_SAVED_MAX and RLIM_SAVED_CUR | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Konstantin Belousov <kib> | ||||||
| Component: | Ports Framework | Assignee: | Port Management Team <portmgr> | ||||||
| Status: | Closed FIXED | ||||||||
| Severity: | Affects Only Me | CC: | ports-bugs, vsevolod | ||||||
| Priority: | --- | Flags: | kib:
exp-run?
|
||||||
| Version: | Latest | ||||||||
| Hardware: | Any | ||||||||
| OS: | Any | ||||||||
| See Also: | https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=209729 | ||||||||
| Attachments: |
|
||||||||
There were no new failure related to RLIM_SAVED_MAX / RLIM_SAVED_CUR. However, there were new failures related to recent addition of DTYPE_NONE: http://package22.nyi.freebsd.org/data/headi386PR216966-default/2017-02-12_23h02m14s/logs/errors/exim-postgresql-4.88_1.log http://package22.nyi.freebsd.org/data/headi386PR216966-default/2017-02-12_23h02m14s/logs/errors/exim-mysql-4.88_1.log http://package22.nyi.freebsd.org/data/headi386PR216966-default/2017-02-12_23h02m14s/logs/errors/exim-ldap2-4.88_1.log http://package22.nyi.freebsd.org/data/headi386PR216966-default/2017-02-12_23h02m14s/logs/errors/exim-monitor-4.88_1.log http://package22.nyi.freebsd.org/data/headi386PR216966-default/2017-02-12_23h02m14s/logs/errors/exim-sa-exim-4.88+4.2_1.log http://package22.nyi.freebsd.org/data/headi386PR216966-default/2017-02-12_23h02m14s/logs/errors/exim-sqlite-4.88_1.log http://package22.nyi.freebsd.org/data/headi386PR216966-default/2017-02-12_23h02m14s/logs/errors/exim-4.88_1.log (In reply to Antoine Brodin from comment #1) Exim includes sys/file.h and gets what it asked for. sys/file.h is non-standard header and has no definitions relevant to a user-mode applications, on FreeBSD. Since exim uses DTYPE_* symbols for its own purposes, surely it clashes with the FreeBSD-internal usage. I am not sure what is the best way to proceed. One route is to rename newly added DTYPE_NONE define in sys/file.h, but it is IMO over-reaction to the app mis-using our headers. Another, more correct route, is remove sys/file.h include usage from exim. The patch is attached, it should be dropped into mail/exim/files. Created attachment 179990 [details]
mail/exim: Do not include sys/file.h
cc: exim maintainer Thank you, Konstantin. I'll add this patch to the port today. A commit references this bug: Author: vsevolod Date: Wed Feb 15 12:39:53 UTC 2017 New revision: 434160 URL: https://svnweb.freebsd.org/changeset/ports/434160 Log: - Fix build after r313495 - Bump revision PR: 216966 Submitted by: kib Changes: head/mail/exim/Makefile head/mail/exim/files/patch-sys-file-include |
Created attachment 179831 [details] Add RLIM_SAVED_MAX and RLIM_SAVED_CUR This is a request to exp-run the patch in PR 209729. Exact version of the change is attached there. We did not provided the symbols for very long time, so I suspect that some ports might grow workarounds which could be incompatible with the change.