Summary: | emulators/virtualbox-ose-kmod: compile error: error: "offsetof is not defined!" | ||||||
---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | O. Hartmann <ohartmann> | ||||
Component: | Individual Port(s) | Assignee: | Jung-uk Kim <jkim> | ||||
Status: | Closed FIXED | ||||||
Severity: | Affects Many People | CC: | jkim, madpilot, olevole, vbox | ||||
Priority: | --- | ||||||
Version: | Latest | ||||||
Hardware: | Any | ||||||
OS: | Any | ||||||
Attachments: |
|
Description
O. Hartmann
2018-05-23 04:40:02 UTC
Created attachment 193639 [details] Tantive patch I've hit this myself just now. Having a look at the failing part of the source code it looks like what they define as a kludge is not necessary anymore and has been fixed in base r324622. I also suspect the kludge was already not necessary when they switched from including stddef.h to including sys/stddef.h, which has never diverged from types.h in the definition of offsetof. I've made a very simple patch removing the kludge and simply including the required files. It does compile, but I still have to test it at runtime, which I will do once building a bunch of other packages is done and report back. I'm anyway attaching my patch for reference. (In reply to Guido Falsi from comment #1) > Created attachment 193639 [details] > Tantive patch [...] > I'm anyway attaching my patch for reference. With the attached patch it compiles and works fine on head r334087. I'll update the patch to limit it's scope based on __FreeBSD_version, but I still have to identify the correct values for that. A commit references this bug: Author: jkim Date: Fri Jun 1 00:30:51 UTC 2018 New revision: 471248 URL: https://svnweb.freebsd.org/changeset/ports/471248 Log: Remove obsolete kludges. All supported versions do not need these hacks. Furthermore, these hacks break build on head. PR: 228433 Changes: head/emulators/virtualbox-ose/files/patch-include_iprt_types.h Committed, thanks! (In reply to Guido Falsi from comment #2) All supported versions do not need the kludges. Confirmed with Poudriere. (In reply to Jung-uk Kim from comment #5) > (In reply to Guido Falsi from comment #2) > All supported versions do not need the kludges. Confirmed with Poudriere. Thanks for checking this and taking care of it! |