tasklwIP - A Lightweight TCP/IP stack - Tasks: task #16543, LwIP unix port use deprecated tool

 
 

You are not allowed to post comments on this tracker with your current authentication level.

task #16543: LwIP unix port use deprecated tool

Submitter:  Andrey Vinogradov <andreyvinogradov>
Submitted:  Tue 21 May 2024 08:46:13 AM UTC
   
 
Category:  Contrib Should Start On:  Tue 21 May 2024 12:00:00 AM UTC
Should be Finished on:  Fri 21 Jun 2024 12:00:00 AM UTC Priority:  3 - Low
Status:  None Privacy:  Public
Assigned to:  None Percent Complete:  0%
Open/Closed:  Open Planned Release:  None
Effort:  0.00

Wed 22 May 2024 07:36:27 PM UTC, comment #2: 

contrib/ports/unix/setup-tapif:

@@ -1,7 +1,5 @@
 #!/bin/bash
 
-# This script needs bridge-util debian package or similar
-# for other distros.
 
 # Run using "source setup-tapif" to get exported PRECONFIGURED_TAPIF variable
 # Alternatively, add "export PRECONFIGURED_TAPIF=tap0" to ~/.bashrc
@@ -16,7 +14,7 @@ export PRECONFIGURED_TAPIF=tap0
 
 sudo ip tuntap add dev $PRECONFIGURED_TAPIF mode tap user `whoami`
 sudo ip link set $PRECONFIGURED_TAPIF up
-sudo brctl addbr lwipbridge
-sudo brctl addif lwipbridge $PRECONFIGURED_TAPIF
+sudo ip link add dev lwipbridge type bridge
+sudo ip link set dev $PRECONFIGURED_TAPIF master lwipbridge
 sudo ip addr add 192.168.1.1/24 dev lwipbridge
 sudo ip link set dev lwipbridge up

-------------------------------------------

this patch works fine for my app (Ubuntu)

Andrey Vinogradov <andreyvinogradov>
Tue 21 May 2024 09:15:45 AM UTC, comment #1: 

I you are able to test, you can submit a patch of what needs to change.

Hints of another project that replaced the usage of brctl:
https://opendev.org/openstack/os-ken/commit/1f751b2d7d385a75d4255934694177d6fd6f012f

Stian Sebastian Skjelstad <mywave>
Tue 21 May 2024 08:46:13 AM UTC, original submission:  

Hi!

lwip/contrib/ports/unix/setup-tapif script use bridge-util

but "The use of brctl is deprecated and is considered obsolete".

Can you update this script to use some other tool?


Andrey Vinogradov <andreyvinogradov>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by mywave (Posted a comment)
  • -email is unavailable- added by andreyvinogradov (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    No changes have been made to this item

    Back to the top

    Powered by Savane 3.14-8eb0.
    Corresponding source code