Shell 调用 tar 打包总是报错！

[jalena]

最近在折腾一个脚本，但是总遇到 一个报错！

脚本在 https://github.com/jalena/vps-backup/blob/master/backup.sh （错误地方在#97 ）

每次执行的时候以下代码就包 tar 没有 e 参数！！

function configuration(){
nginxcnf='find / -name nginx.conf |grep -v root'
tar cPf nginx$(date +%Y%m%d).tar.gz $NGINXPATH
echo -e "package nginx$(date +%Y%m%d).tar.gz success!"
tar rPf nginx_$(date +%Y%m%d).tar.gz $nginx_cnf
echo -e "Additional file successfully"
}

求大神解答写

[ooxxcc]

错误提示都不贴出来……

[ooxxcc]

两个 tar 命令前面加 echo ，看看实际执行的指令是什么

[pright]

nginxcnf=`find / -name nginx.conf |grep -v root`
是`不是'

[congeec]

看了下，你 shell 用的很不规范。即使这个地方修复了其他地方肯定也有 bug 。用 shellcheck 过一边吧

[SpicyCat]

set -ex 或者 set -x 看看详细执行信息。

打包是 tar zcPf 吧？你代码在另外一个地方用的是 tar zcPf ，为什么这里用 tar cPf

[Arthur2e5]

能不能先学学 markdown ……
@SpicyCat z 只是要求 gzip 而已。
@pright 干脆不要用反引号用$( ) 大法……
@congeec shellcheck 目前为止还不会提醒 function xxx 这种坏习惯。

[congeec]

@Arthur2e5 funciton 啥的就是看起来碍不碍眼。关键是楼主变量没有加引号噗哈哈。
``` shell
var=
rm -rf /usr/local/$var
```
然后 /usr/local 目录就没了😆😆

[JerningChan]

你的$nginx_cnf?是哪个变量呀?
你帖出来的,就看见个
nginxcnf='find / -name nginx.conf |grep -v root'
你确定你未写错变量?

[Arthur2e5]

@congeec function 并不是碍不碍眼的问题。写顺了写进简陋一点但还算 POSIX 的 shell 例如 dash 和 busybox ash 试试喏。这种最容易避免的 bashism 干脆从最早就不要有。

* 当然，可以养成 `alias function=` 的习惯……

然后管你加不加引号你那个 var 是空的的时候都得死啊。

[congeec]

@Arthur2e5 你说的对哦

[jalena]

@pright 找到问题了,,,,,就是 nginxcnf=`find / -name nginx.conf |grep -v root` ~~~~~~

[jalena]

@congeec function 是故意去掉了,母的技术让他看起来更小..简写也不影响功能!

[jalena]

@JerningChan 变量没错,,只是这个输出的东西和打包用的路径不符,所以报错..用通道就好了..

find / -name nginx.conf |grep -v root | xargs tar rPf nginx_$current_date.tar.gz

[jalena]

@Arthur2e5 加不加 function 还是个人喜好问题吧,~

[jalena]

@SpicyCat cPf...

c 追加
P 按原始路径!

[jalena]

@Arthur2e5 刚开始来 V2EX 哈,我还不知道支持这个语法呢~