Page MenuHomeFreeBSD

Move pft_ping.py and sniffer.py to the common test directory
ClosedPublic

Authored by ahsanb on Aug 15 2019, 4:43 PM.
Tags
None
Referenced Files
F102839201: D21276.diff
Sun, Nov 17, 8:22 PM
Unknown Object (File)
Thu, Nov 14, 9:19 PM
Unknown Object (File)
Mon, Nov 11, 10:36 AM
Unknown Object (File)
Thu, Nov 7, 1:32 AM
Unknown Object (File)
Wed, Nov 6, 3:55 PM
Unknown Object (File)
Wed, Nov 6, 1:11 PM
Unknown Object (File)
Mon, Nov 4, 4:18 PM
Unknown Object (File)
Fri, Nov 1, 3:16 AM

Details

Summary

The pft_ping.py and sniffer.py tool is moved from/tests/sys/netpfil/pf to /tests/sys/netpfil/common directory because these tools are to be used in common for all the firewalls.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

  • Make the pf tests use the sniffer and pft_ping tools from the common directory and hence ensure only one installation of these tools.

This probably also wants this:

diff --git a/ObsoleteFiles.inc b/ObsoleteFiles.inc
index 19adf066b1b..1725b06fa69 100644
--- a/ObsoleteFiles.inc
+++ b/ObsoleteFiles.inc
@@ -38,6 +38,9 @@
 #   xargs -n1 | sort | uniq -d;
 # done

+# 20190816: move common netpfil test files
+OLD_FILES+=usr/tests/sys/netpfil/pf/sniffer.py
+OLD_FILES+=usr/tests/sys/netpfil/pf/pft_ping.py
 # 20190729: gzip'ed a.out support removed
 OLD_FILES+=usr/include/sys/inflate.h
 # 20190722: cap_random(3) removed

That'll allow people to clean up the old version of sniffer.py and pft_ping.py.

tests/sys/netpfil/pf/forward.sh
48 ↗(On Diff #60873)

I'd do ${common_dir}/pft_ping.py, if only for consistency.

  • Add old pft_ping and sniffer.py to ObsoleteFiles.inc
  • replace $common_dir with ${common_dir} for consistency
  • Fix “replace $common_dir with ${common_dir} for consistency”
This revision is now accepted and ready to land.Aug 16 2019, 2:03 PM
  • Since the NAT tests landed before this diff the merge wouldn't be possible because the NAT tests changed the common/Makefile. Created an updated diff from the new upstream.
This revision now requires review to proceed.Aug 17 2019, 2:08 PM
This revision is now accepted and ready to land.Aug 18 2019, 5:57 PM