bashスクリプト内でchown $ USER：$ USERを使用する

私が実行している小さなbashスクリプトで、作成された新しいディレクトリを変更しようとしています。私は追加しました：

sudo chown $USER:$USER /var/www/$sitename
sudo chmod 775 /var/www/$sitename

mkdir（sudo mkdir /var/www/$sitename）の行の後。

何らかの理由で、chownが実行されていません。手動で実行できますが、ファイルに書き込んでも動作しません。「chown」が「mkdir」および「chmod」と同じ色で強調表示されていないことに気付きましたが、問題を理解できません。

ここでchownが機能しないのはなぜですか？

それは問題$USER:$USERですか？

編集これ が完全なスクリプトです。root以外のユーザーがスクリプトを実行した場合、どのようにファイルを変更しますか？

#!/bin/sh
#!/bin/bash
# New Site

cd /etc/apache2/sites-available/
echo "New site name (test.my):"
read sitename
echo "<VirtualHost *:80>

        ServerAdmin admin@$sitename

    ServerName $sitename

        ServerAlias $sitename

    DocumentRoot /var/www/$sitename

        <Directory />
                Options FollowSymLinks
                AllowOverride All
        </Directory>
        <Directory /var/www/$sitename>
                Options Indexes FollowSymLinks MultiViews
                AllowOverride All
                Order allow,deny
                allow from all
        </Directory>

        ErrorLog ${APACHE_LOG_DIR}/error.log
        CustomLog ${APACHE_LOG_DIR}/access.log combined


</VirtualHost>" > $sitename.conf
sudo mkdir /var/www/$sitename
sudo chown $USER:$USER /var/www/$sitename
echo USER is $USER
sudo chmod 775 /var/www/$sitename
sudo a2ensite $sitename.conf
sudo apachectl restart
echo "New site created"

何らかの理由で$USER設定されていない場合は、idコマンドを使用して実際のユーザーのIDを取得できます。したがって、$USER変数を初めて使用するときは、シェル拡張を使用してデフォルト値を指定できます。chownスクリプトの行を次のように変更します。

sudo chown ${USER:=$(/usr/bin/id -run)}:$USER /var/www/$sitename

場合はUSER、これが実行されたときに空または設定解除され、bashが設定されますUSERの出力に変数を/usr/bin/id -run

スクリプトを呼び出すと、ルートにsudo設定さ$USERれます。

$ sudo ./myscript.sh

私は試しましたchown ${USER:=$(/usr/bin/id -run)}:$USER /var/www/$sitenameが、それでもルートを返します。

で使用whoした場合awk、でスクリプトを呼び出した現在のユーザーを取得できましたsudo。

currentuser=$(who | awk '{print $1}')}
chown -R $currentuser:$currentuser /var/www/$sitename`

問題を単純化するために、そして変数sitenameを取得しているので、ユーザー名変数を読み取ってみませんか？

これにより、スクリプトの実行が、スクリプトの実行方法で利用できるようにされた環境変数に依存しないことを確認できます。

ほんの小さな欠陥があると思います。sudoはコマンドを実行するための新しいシェルを開き、sudoの後、ユーザーはrootになります。だから多分あなたはこのようなものを使うべきです：

MYUSER = $ USER

sudo chown $ MYUSER：$ MYUSER

私がMYUSERは上書きされたsystemspezificではなく、うまくいくと思うので。