Bug 263041 - emulators/open-vm-tools: fix build after 2587b74c6187 removed NDF_ONLY_PNBUF
Summary: emulators/open-vm-tools: fix build after 2587b74c6187 removed NDF_ONLY_PNBUF
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Renato Botelho
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-04-04 15:25 UTC by Dimitry Andric
Modified: 2022-04-04 18:01 UTC (History)
0 users

See Also:
garga: maintainer-feedback+


Attachments
fix build after 2587b74c6187 removed NDF_ONLY_PNBUF (1.40 KB, patch)
2022-04-04 15:25 UTC, Dimitry Andric
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitry Andric freebsd_committer freebsd_triage 2022-04-04 15:25:17 UTC
After base commit 2587b74c6187, which removed NDF_ONLY_PNBUF, emulators/open-vm-tools fails to build, with:

--- vfsops.o ---
vfsops.c:190:16: error: use of undeclared identifier 'NDF_ONLY_PNBUF'; did you mean 'NDFREE_PNBUF'?
   NDFREE(ndp, NDF_ONLY_PNBUF);
               ^~~~~~~~~~~~~~
               NDFREE_PNBUF
/usr/src/sys/sys/namei.h:291:6: note: 'NDFREE_PNBUF' declared here
void NDFREE_PNBUF(struct nameidata *);
     ^

Indeed, NDFREE_PNBUF() should now be used.
Comment 1 Dimitry Andric freebsd_committer freebsd_triage 2022-04-04 15:25:52 UTC
Created attachment 232936 [details]
fix build after 2587b74c6187 removed NDF_ONLY_PNBUF
Comment 2 Renato Botelho freebsd_committer freebsd_triage 2022-04-04 16:19:12 UTC
I had this change queued here together with version 12.0.0 but it's better to get it fixed earlier.  It's approved, please go ahead and get it committed.

Thanks!
Comment 3 commit-hook freebsd_committer freebsd_triage 2022-04-04 17:58:59 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=c55c6dd6767ceaf37b3d69cff84d1b859e33d30e

commit c55c6dd6767ceaf37b3d69cff84d1b859e33d30e
Author:     Dimitry Andric <dim@FreeBSD.org>
AuthorDate: 2022-04-04 15:21:54 +0000
Commit:     Dimitry Andric <dim@FreeBSD.org>
CommitDate: 2022-04-04 17:54:55 +0000

    emulators/open-vm-tools: fix build after 2587b74c6187 removed NDF_ONLY_PNBUF

    After base commit 2587b74c6187, which removed NDF_ONLY_PNBUF,
    emulators/open-vm-tools fails to build, with:

    --- vfsops.o ---
    vfsops.c:190:16: error: use of undeclared identifier 'NDF_ONLY_PNBUF'; did you mean 'NDFREE_PNBUF'?
       NDFREE(ndp, NDF_ONLY_PNBUF);
                   ^~~~~~~~~~~~~~
                   NDFREE_PNBUF
    /usr/src/sys/sys/namei.h:291:6: note: 'NDFREE_PNBUF' declared here
    void NDFREE_PNBUF(struct nameidata *);
         ^

    Indeed, NDFREE_PNBUF() should now be used.

    PR:             263041
    Approved by:    garga (maintainer)
    MFH:            2022Q2

 .../files/patch-modules_freebsd_vmblock_vfsops.c          | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)