Fix DTrace test tst_inet_ntop_d: remove definitions which are already in libdtrace
ClosedPublic
Actions

Authored by lwhsu on Sep 12 2017, 8:47 AM.

Details

Reviewers

gnn
markj

Commits

rS323499: Fix DTrace test tst_inet_ntop_d: remove definitions are already in libdtrace

Summary

We have D definitions for the named values in socket.h after r323253, remove
them in test script to prevent compiling failure.

Ref: https://ci.freebsd.org/job/FreeBSD-head-amd64-dtrace_test/767/testReport/common.funcs/t_dtrace_contrib/tst_inet_ntop_d/

Test Plan

/usr/tests/cddl/usr.sbin/dtrace/common/funcs/../../dtest /usr/tests/cddl/usr.sbin/dtrace/common/funcs/tst.inet_ntop.d

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lwhsu created this revision.Sep 12 2017, 8:47 AM

Herald added 1 blocking reviewer(s): gnn. · View Herald TranscriptSep 12 2017, 8:47 AM

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B11510: Diff 32952.Sep 12 2017, 8:47 AM

LGTM, thanks Li-Wen.

LGMT too. Thanks!

This revision is now accepted and ready to land.Sep 12 2017, 2:13 PM

Closed by commit rS323499: Fix DTrace test tst_inet_ntop_d: remove definitions are already in libdtrace (authored by lwhsu). · Explain WhySep 12 2017, 4:01 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

cddl/

contrib/

opensolaris/

cmd/

dtrace/

test/

tst/

common/

funcs/

tst.inet_ntop.d

3 lines

Diff 32977

View Options

head/cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/funcs/tst.inet_ntop.d

Fix DTrace test tst_inet_ntop_d: remove definitions which are already in libdtraceClosedPublicActions