[SOLVED] ip vice /sbin/ip in /etc/rc.d/rc.inet1

andygoth · 08-19-2018, 11:09 PM

In slackware64-current, /etc/rc.d/rc.inet1 contains the following:

Code:

    # Flush the address from the interface:
    ip address flush dev ${1}

"ip" should probably be "/sbin/ip" to avoid dependence on $PATH and to match every other invocation of ip in the script.

abga · 08-20-2018, 06:48 PM

Good catch! Don't know why the full path was missed out (but then I use my own network init scripts):
https://www.linuxquestions.org/quest...ml#post5816535
ftp://ftp.osuosl.org/pub/slackware/s.../ChangeLog.txt
Fri Feb 16 03:19:36 UTC 2018
n/network-scripts-15.0-noarch-5.txz: Rebuilt.
rc.inet1: set the broadcast address, and flush the ip address from interfaces
when shutting them down. Thanks to Markus Wiesner.

tadgy · 08-20-2018, 08:45 PM

Quote:

Originally Posted by andygoth

In slackware64-current, /etc/rc.d/rc.inet1 contains the following:

Code:

    # Flush the address from the interface:
    ip address flush dev ${1}

"ip" should probably be "/sbin/ip" to avoid dependence on $PATH and to match every other invocation of ip in the script.

Robby (rworkman) currently maintains the Slackware network setup scripts in his git repo at: https://git.rlworkman.net/slacknetsetup/

The best thing to do is clone that repository, make the changes and submit a patch to Robby via email - such a simple fix will likely be added immediately.

andygoth · 02-04-2021, 12:42 PM

Quote:

Originally Posted by tadgy

Robby (rworkman) currently maintains the Slackware network setup scripts in his git repo at: https://git.rlworkman.net/slacknetsetup/

The best thing to do is clone that repository, make the changes and submit a patch to Robby via email - such a simple fix will likely be added immediately.

Patch email sent to Robby 26 January 2021. I don't know why I didn't do it right when you suggested it. I haven't gotten a reply yet, but I suppose that's fair since I myself waited more than two years!

Code:

--- rc.inet1~   2021-01-26 10:24:51.132541044 -0600
+++ rc.inet1    2021-01-26 10:25:07.986847728 -0600
@@ -285,7 +285,7 @@
        br_close $i
      fi
      # Flush the address from the interface:
-    ip address flush dev ${1}
+    /sbin/ip address flush dev ${1}
    fi
  }

andygoth · 02-28-2021, 03:29 PM

This is now fixed thanks to rc.inet1 being rewritten for IPv6, etc.