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
Unknown Object (File)
Sat, Mar 30, 8:23 PM
Unknown Object (File)
Jan 26 2024, 1:54 PM
Unknown Object (File)
Jan 26 2024, 1:51 PM
Unknown Object (File)
Jan 26 2024, 1:51 PM
Unknown Object (File)
Jan 26 2024, 1:51 PM
Unknown Object (File)
Jan 26 2024, 1:51 PM
Unknown Object (File)
Jan 26 2024, 1:51 PM
Unknown Object (File)
Jan 26 2024, 1:51 PM

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