Page MenuHomeFreeBSD

net/intel-em-kmod: Update to 7.7.8
ClosedPublic

Authored by erj on Wed, Sep 9, 9:40 PM.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

erj created this revision.Wed, Sep 9, 9:40 PM
erj requested review of this revision.Wed, Sep 9, 9:40 PM
erj added a comment.Wed, Sep 9, 9:42 PM

portlint -AC output:

WARN: Makefile: NETMAP_AUTO is listed in OPTIONS_DEFINE, but no PORT_OPTIONS:MNETMAP_AUTO appears in this slave Makefile.  Make sure it appears in the master's Makefile.
WARN: Makefile: NETMAP_ON is listed in OPTIONS_DEFINE, but no PORT_OPTIONS:MNETMAP_ON appears in this slave Makefile.  Make sure it appears in the master's Makefile.
WARN: Makefile: NETMAP_OFF is listed in OPTIONS_DEFINE, but no PORT_OPTIONS:MNETMAP_OFF appears in this slave Makefile.  Make sure it appears in the master's Makefile.
WARN: Makefile: for new port, make $FreeBSD$ tag in comment section empty, to make SVN happy.
WARN: Makefile: new ports should not set PORTREVISION.
WARN: Makefile: new ports should not set PORTEPOCH.
FATAL: Makefile: extra item "BROKEN_armv6" placed in the LICENSE section.
1 fatal error and 6 warnings found.
erj added a comment.Wed, Sep 9, 9:48 PM

poudriere testport log:

Though, I am annoyed that the log exposes internal information in the environment variable list at the start, like the *_PROXY variables. Is there an automatic way to strip those out?

Thanks for the update :)

This revision is now accepted and ready to land.Thu, Sep 10, 8:03 AM
MAX_NETMAP_OSVERSION=   1199999  # Doesn't build w/NETMAP on 12

@erj, is it still true?

erj added a comment.Fri, Sep 11, 8:14 PM
MAX_NETMAP_OSVERSION=   1199999  # Doesn't build w/NETMAP on 12

@erj, is it still true?

I think so; IIRC it's because FreeBSD 12.0 is missing the driver-specific netmap headers (due to the in-kernel version of em having NETMAP integrated into iflib), and we don't include them in the OOT driver because we don't want to maintain them.

This revision was automatically updated to reflect the committed changes.

Committed, thanks!