[ovs-dev] [PATCH] documentation: add missing sudo to intro install

Ben Pfaff blp at ovn.org
Fri Apr 2 19:17:18 UTC 2021


On Fri, Apr 02, 2021 at 07:11:22PM +0000, hunchback wrote:
> the current documentation assumes everything is run from within root
> what is not correct in the general case, the revised install
> documentation makes use of `sudo` for specific commands which require to
> be run under root permissions.
> 
> Signed-off-by: hunchback <aidan.shribman at gmail.com>

Thanks for the patch.

>         $ config_file="/etc/depmod.d/openvswitch.conf"
> -       $ for module in datapath/linux/*.ko; do
> +       $ sudo for module in datapath/linux/*.ko; do
>           modname="$(basename ${module})"
>           echo "override ${modname%.ko} * extra" >> "$config_file"
>           echo "override ${modname%.ko} * weak-updates" >> "$config_file"
>           done

sudo won't work the way you use it above.


More information about the dev mailing list