为什么 sudo 会更改创建新目录的目录？

Question

这是因为你正在使用$USERNAME.您可以轻松地看到它的实际效果：

$ bash -c 'echo "USERNAME is $USERNAME"'
USERNAME is terdon
$ sudo bash -c 'echo "USERNAME is $USERNAME"'
[sudo] password for terdon: 
USERNAME is

正如您在上面看到的，$USERNAME使用时未定义该变量sudo，这意味着这些行导致了问题：

if grep -q "Arch" $linux_file
then
    # The host is an Arch based distribution
    media_dir=/run/media/$USERNAME/

elif grep -q "Ubuntu" $linux_file || grep -q "Pop" $linux_file
then
    # The host is an Ubuntu or Pop OS based distribution
    media_dir=/media/$USERNAME/

else
    # The host is not a compatible distribution
    echo "Linux distribution not supported, exiting"
    exit 1
fi

由于$USERNAME尚未设置，您只需获取/run/media/Arch 和/media/其他两个。一个简单的解决方案是使用$SUDO_USER：

$ bash -c 'echo "USERNAME is $SUDO_USER"'
USERNAME is 
$ sudo bash -c 'echo "USERNAME is $SUDO_USER"'
USERNAME is terdon

正如您在上面看到的，$SUDO_USER保存调用命令的用户的名称sudo。因此，如果您希望脚本仅以这种方式运行，则可以使用它。如果您仍然想允许不以 root 身份运行的选项，您可以添加一个测试：

if [ -n "$SUDO_USER" ]; then
  username="$SUDO_USER"
else
  username="$USERNAME"
fi

然后，在脚本的其余部分中，$username无论您使用过什么地方，都使用$USERNAME.

或者，您也可以使用$USER即使在使用时设置的标准变量sudo：

$ bash -c 'echo "USERNAME is $USER"'
USERNAME is terdon
$ sudo bash -c 'echo "USERNAME is $USER"'
USERNAME is root

root但是当运行为时，这会返回sudo，所以无论如何你最好使用该SUDO_USER方法。

最后，关于您的脚本的一些注释：

您有很多未加引号的变量，这通常是一个坏主意，如果您正在处理任意路径和文件名，则尤其糟糕。看为什么我的 shell 脚本会因为空格或其他特殊字符而卡住？和忘记在 bash/POSIX shell 中引用变量的安全隐患。
您没有进行任何错误检查，并假设所有命令都有效。在运行包含sudo rm命令的脚本时，这非常非常危险。如果变量未设置或cd失败（例如由于$USERNAME未设置或类似原因），导致您进入cd不存在的目录，那么您很容易破坏整个系统。

这确实有可能发生。这是几年前的一个著名案例：适用于 Linux 的可怕 Steam 错误会删除您 PC 上的所有个人文件。因此，请确保添加|| exit到各种重要命令，甚至添加到set -x脚本的开头。

您有几个正在解析的实例ls。这是也是一个非常糟糕的主意并且很容易破裂。所以，而不是这个：

potential_dirs=($(ls -l $media_dir | awk '/^d/ {print $9}'))

用这个：

potential_dirs=( "$media_dir"/*/ )

您使用的下一部分ls存在各种问题：

# Change to backup directory
cd $base_dir

# Delete the oldest directories if necessary
if [ $num_delete -gt 0 ] ; then
    dirs=`ls -d */ | cut -f1 -d'/' | head -n $num_delete`
    for variable in $dirs
    do
        echo "Removing $variable directory"
        sudo rm -r $variable
    done
fi

首先，这不是删除最旧的目录，而是按字母顺序删除第一个目录。接下来，如果任何目录名称包含换行符，ls则和都会失败，并且由于您没有引用，因此甚至会因空格而失败。更重要的是，您实际上没有任何错误检查，因此如果前一个由于某种原因失败，您将从其他地方删除文件！rmrm"$variable"cd $base_dir

因此，如果您想删除，请将上面的内容更改为类似的内容$num_delete 最老的目录：


# Delete the oldest directories if necessary
if [ $num_delete -gt 0 ] ; then
    readarray -d '' dirs < <(stat --printf='%Y %n\0' "$base_dir"/*/ |
                              sort -znk1,1 | head -zn "$num_delete" | 
                              cut -z -d ' ' -f2-)
    for variable in "${dirs[@]}"
    do
        echo "Removing $variable directory"
        sudo rm -r -- "$variable"
    done
fi

注意这里没有必要cd。

对于名称中包含换行符的情况，您计算目录的方法将失败：

dir_num=`$nfind $base_dir -mindepth 1 -maxdepth 1 -type d | wc -l`

使用这个代替它可以工作任何姓名：

dir_num=$($nfind $base_dir -mindepth 1 -maxdepth 1 -type d -printf '.\n' | wc -l)

避免使用反引号 ( `command`)，它们已被弃用$(command)。无论哪里有var=`command`，都可以替换为var=$(command).
不需要cur_dir=`pwd`，当前目录已经保存在特殊$PWD变量中，因此您可以简单地使用$PWD.
在脚本的最后，你有这样的内容：
```
cd `pwd`
```
这基本上是一个空操作，因为您说的是“移动到命令返回的任何目录pwd”，所以您实际上永远不会移动到任何地方，因为`pwd`只会打印当前目录。我想你的意思是cd "$cur_dir"，但即使这样也是不必要的。正如您在上面第 3 点中看到的，cd首先没有必要。

Answer 1