シェルスクリプトの書き方

Question

転送するコマンドをシェルに記載したいのですが
元が複数あっても同じところにもっていくわけでなく行先ももちろん
パスを分けるので複数書かないといけないです。

賢い書き方を教えて抱けますでしょうか？
最悪全部'&&'でつなげていってもいいんですが
かっこいい書き方で教えてください

aws s3 cp /var/log/maillog-* s3://{バケット名}/{パス}
aws s3 cp /var/log/spooler-* s3://{バケット名}/{パス}

Comments

[TakaiY]

この情報では、どのようなところに困難を感じているのかわかりません。
ファイルと転送先の具体的な対応についての情報わありますか？
たとえば、maillog-* であらわされるファイルは s3のどういう場所にどのような名前で配置されるのですか？

[ryoma7i]

>この情報では、どのようなところに困難を感じているのかわかりません。
→確かに羅列してもできることなので困難ではありません
　シェルスクリプト記載するにあたりかっこよくなるなら参考にしたいというだけです。

>ファイルと転送先の具体的な対応についての情報わありますか？
→いやこれについてはいまのところこれだけです。S3の設定して決まれば追記するんですが
　決まればこの後ろに入れていくということだけです

ただ伺いたいのですが、コマンドの羅列をシェルに記載するにあたっての質問が
かっこよくしたい(見栄えを良くしたい、他人に見せられるレベルにしたい)ということにおいて
S3の場所を教えてほしいというのは本当に必要な情報なのでしょうか？

[melian]

一応、bash や zsh にある連想配列(hash)を使えば見栄えはよくなる？かもしれませんが、本質的にはコマンドを羅列するのと同じです。まぁ、シェルスクリプトの先頭に set -e を付けるくらいでしょうかね。。。("&&" で繋げる必要はなくなります)

[TakaiY]

例えば /var/log にあるすべてのもしくは特定のディレクトリを、そのディレクトリ名から作られるバケット名やパス名に転送するのであれば、羅列でなくfor文などで効率よく書けでしょう。
ファイルとs3のパスの対応を 別のテキストファイルに入れておいて読むっていうのもありでしょう。

[ryoma7i]

TakaiY様
ありがとうございます。
少しfor文での作成は頑張ってみたいと思います。

別のテキストファイルはちょっと・・・というところですね。
理由としては1つのシェルでシンプルにまとめたいということです。
残念ながらシェルがこれだけではありませんのでそういう考えをしております。

[maisumakun]

> 複数書かないといけないです。

いくつぐらいになることを想定していますか？

[ryoma7i]

ログファイル20個ほどです。
/va/logにあるものだけはありませんね

Answer 1

既に解決済みですが、一例としてヒアドキュメントを使う方法など。(IFS 等、適当に変更してください)

#!/bin/bash

set -eu
shopt -s nullglob

bucket='s3://bucket'
while IFS=, read src dst
do
  declare -a files=($(echo "${src}"))
  [[ ${#files[@]} == 0 ]] && continue
  # aws s3 cp "${files[@]}" "${bucket}/${dst}"
done <<EOT
/var/log/maillog-*,path01
/var/log/spooler-*,path02
EOT

Answer 2

失礼しました
ですが・・・まだ正式に決まってません

ですがパケット名までは同じです。パスがログの目的別のものになる予定です。
s3://{バケット名}/{パス}

Answer 3

複数の物を規則性無く異なる場所にコピーするなら、書き並べるしか無いです。
パスなどに共通部分が多いなら、関数定義すると見やすくなるかもしれません。コピー先とコピー元に規則性があるならその規則を関数内で実装します。
共通部分が少なく、コピー先とコピー元に規則性も無いなら、質問文のように単に並べるしか無いです。

S3の場所を教えてほしいというのは本当に必要な情報なのでしょうか？

「S3の場所を教えてほしい」という質問じゃなくて、
「コピー先とコピー元に規則性があるかどうか。あるならどういう規則性か」という質問ですね。

&&でつなげるかset -eを書くかは「エラーの時どうしたいか」次第です。

関数定義のサンプル：
コピー先とコピー元のパスに、同じ文字列がありそれ以外は共通という一例。

foo(){
  aws s3 cp /var/log/$1-* s3://{バケット名}/backup-$1
}

foo maillog &&
foo spooler