Summary: | emulators/open-vm-tools-nox11: fails to build after r357287 and/or r357403 | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Ruslan Garipov <brigadir15> | ||||||
Component: | Individual Port(s) | Assignee: | Josh Paetzel <jpaetzel> | ||||||
Status: | Closed FIXED | ||||||||
Severity: | Affects Only Me | CC: | mjg | ||||||
Priority: | --- | Flags: | bugzilla:
maintainer-feedback?
(jpaetzel) |
||||||
Version: | Latest | ||||||||
Hardware: | Any | ||||||||
OS: | Any | ||||||||
Attachments: |
|
Description
Ruslan Garipov
2020-02-04 11:05:11 UTC
Sorry for build breakage. You can unconditionally stop handling handling both flags. If my git log is right, FreeBSD *never* set them. They got inherited from the original BSD codebase and not cleaned up. iow no version checks are necessary. Thanks for the quick reply. No apologies needed, such is the way of things for out of tree drivers. Created attachment 211404 [details] Remove the VDESC_NOMAP_VPP and VDESC_VPP_WILLRELE flags (v1) > You can unconditionally stop handling handling both flags I removed define-guards from my original patch. Mateusz, thanks for your hints! Ruslan, The patch you attached contains a change from patches you submitted previously I've already committed, I'm just going to pull those changes out and commit your new changes. As always, thanks for your help with this. A commit references this bug: Author: jpaetzel Date: Thu Feb 6 20:18:10 UTC 2020 New revision: 525444 URL: https://svnweb.freebsd.org/changeset/ports/525444 Log: Unbreak build on HEAD PR: 243864 Submitted by: Ruslan Garipov <brigadir15@gmail.com> Sponsored by: Panzura Changes: head/emulators/open-vm-tools/Makefile head/emulators/open-vm-tools/files/patch-modules_freebsd_vmblock_vnops.c (In reply to Josh Paetzel from comment #4) > The patch you attached contains a change from patches you submitted > previously I've already committed, I'm just going to pull those > changes out and commit your new changes. Josh, I'm sorry I've added an extra work for you. Yes, I recreate patch-modules_freebsd_vmblock_vnops.c with `make makepatch` from the scratch to preserve the previous changes (read: to be able to build the port). I just failed to think that posting "complete" patch would force you to pick some parts from it. Thanks for committing! |