Comment 5 for bug 2010561

Revision history for this message
Simon Déziel (sdeziel) wrote (last edit ):

@slyon I saw that https://git.launchpad.net/network-manager/commit/?h=netplan/lunar-gu&id=900b2e15bce37363b263a224e60674f804114693 requires the `file` package to be available. Some systems don't have installed, albeit probably not common on desktops but I though I'd mention just in case.

That same commit also seem to have a bogus error handling if `nm-online -qs` fails:

```
nm-online -qs || (echo "SKIP: NetworkManager is not ready ..." 1>&2 && continue)
```

The `continue` will be run in the `()` subshell. I think this could be fixed by using `{}` instead.