Differential D19418

Fix compilation WITHOUT_{INET,INET6}_SUPPORT or both.
ClosedPublic
Actions

Authored by bz on Mar 1 2019, 2:30 PM.

Details

Reviewers

ngie
hrs
ume

Commits

rS344740: Fix compilation of world with WITHOUT_{INET,INET6}_SUPPORT or both set.

Summary

buildworld failed when both WITHOUT_INET6_SUPPORT and INET
equivalent were set. Fix netstat and syslogd to make world
compile again.

Test Plan

amd64 buildworlds with one and both options set done.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bz created this revision.Mar 1 2019, 2:30 PM

Herald added a subscriber: imp. · View Herald TranscriptMar 1 2019, 2:30 PM

Harbormaster completed remote builds in B22827: Diff 54579.Mar 1 2019, 2:30 PM

rgrimes added a subscriber: rgrimes.Mar 1 2019, 3:14 PM

Mmmm... I love ifdef soup.

This revision is now accepted and ready to land.Mar 1 2019, 7:22 PM

Thank you bz@!

This clearly obsoletes part of my review where I conflated a few and I have work to do: https://reviews.freebsd.org/D17040

Are you planning to MFC this?

In D19418#415948, @editor_callfortesting.org wrote:

This clearly obsoletes part of my review where I conflated a few and I have work to do: https://reviews.freebsd.org/D17040

Oh I forgot about this one. We should sort the remaining parts of yours out as well.

Are you planning to MFC this?

I guess I will eventually at least to 12.

It looks good to me.

hrs accepted this revision.Mar 3 2019, 3:13 AM

Closed by commit rS344740: Fix compilation of world with WITHOUT_{INET,INET6}_SUPPORT or both set. (authored by bz). · Explain WhyMar 3 2019, 10:00 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

usr.bin/

netstat/

inet.c

29 lines

usr.sbin/

syslogd/

syslogd.c

8 lines

Diff 54633

View Options

head/usr.bin/netstat/inet.c