Re: [PATCH v4 1/1] dts: Remove XML-RPC server for Scapy TG and instead use PythonShell

2024-09-27 Thread Juraj Linkeš
+Note that the order of inheritance is important for this class. In order to instantiate this +class, the abstract methods of :class:`~.capturing_traffic_generator.CapturingTrafficGenerator` +must be implemented. Since some of these methods are implemented in the underlying inter

Re: [PATCH v4 1/1] dts: Remove XML-RPC server for Scapy TG and instead use PythonShell

2024-09-26 Thread Jeremy Spewock
On Thu, Sep 26, 2024 at 5:12 AM Juraj Linkeš wrote: > > The subject line should start with a lowercase character and it's also > over the character limit. This fits and basically says the same: > use PythonShell for Scapy instead of XML-RPC > > Scapy TG didn't fit but I think the above still works

Re: [PATCH v4 1/1] dts: Remove XML-RPC server for Scapy TG and instead use PythonShell

2024-09-26 Thread Jeremy Spewock
On Thu, Sep 26, 2024 at 5:12 AM Juraj Linkeš wrote: > > The subject line should start with a lowercase character and it's also > over the character limit. This fits and basically says the same: > use PythonShell for Scapy instead of XML-RPC > > Scapy TG didn't fit but I think the above still works

Re: [PATCH v4 1/1] dts: Remove XML-RPC server for Scapy TG and instead use PythonShell

2024-09-26 Thread Juraj Linkeš
The subject line should start with a lowercase character and it's also over the character limit. This fits and basically says the same: use PythonShell for Scapy instead of XML-RPC Scapy TG didn't fit but I think the above still works. diff --git a/dts/framework/remote_session/single_active_in

[PATCH v4 1/1] dts: Remove XML-RPC server for Scapy TG and instead use PythonShell

2024-09-25 Thread jspewock
From: Jeremy Spewock Previously all scapy commands were handled using an XML-RPC server that ran on the TGNode. This unnecessarily enforces a minimum Python version of 3.10 on the server that is being used as a traffic generator and complicates the implementation of scapy methods. This patch remo