Clone/future-install-scripts
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

arch-chroot: split variable declaration and assignment

Browse Source 
As suggested by shellcheck:

In arch-chroot line 164:
  local src=$(resolve_link /etc/resolv.conf)
        ^-^ SC2155 (warning): Declare and assign separately to avoid masking return values.

In arch-chroot line 165:
  local dest=$(resolve_link "$chrootdir/etc/resolv.conf" "$chrootdir")
        ^--^ SC2155 (warning): Declare and assign separately to avoid masking return values.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
This commit is contained in:
Emil Velikov 2023-03-24 20:09:20 +00:00
parent 45a2073e33
commit abef18a432
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
arch-chroot.in
View File

@ -47,8 +47,11 @@ resolve_link() {
chroot_add_resolv_conf() {
local chrootdir=$1
local src=$(resolve_link /etc/resolv.conf)
local dest=$(resolve_link "$chrootdir/etc/resolv.conf" "$chrootdir")
local src
local dest
src=$(resolve_link /etc/resolv.conf)
dest=$(resolve_link "$chrootdir/etc/resolv.conf" "$chrootdir")
# If we don't have a source resolv.conf file, there's nothing useful we can do.
[[ -e $src ]] || return 0