[SourceTree]Git hooksの改行コードについて

Question

######質問内容
SourceTreeのGit hooksの「pre-push」のフックスクリプトを用いて、".xml"のついたリソースをプッシュした時にサーバ側にもアップするように処理を書いているのですが、
以下のことがあり、今困っています。

######リソース
pre-push：改行コードはUnix（LF）
upload.sh：改行コードはWindows（CR LF）
upload.bat：改行コードはWindows（CR LF）
上記のリソースはD:\Gitwork\hook_test.git\hooks内にある。
OSはWindows10 Pro。

・リソースのコード
pre-push
if [ $? -eq 0 ]; then
MSG=$(git log -1 --name-only --pretty=format:"%h" | grep strings | grep .xml)
cd $(cd $(dirname $0) && pwd)
for i in ${MSG[@]};do
echo "Update"
echo $i
./upload.sh $i
done
fi

######今起きている現象
upload.shとupload.batの改行コードが両方Windows（CR LF）・両方Unix（LF）の場合
→上記の二つのファイルは正常に起動するがpushができない。
エラー内容
./upload.sh: line 37: $'\r': command not found
.git/hooks/pre-push: line 8: english: command not found
error: failed to push some refs to 'Z:\hook_test'

upload.shとupload.batの改行コードが両方Macintosh（CR）の場合
→上記の二つのファイルは正常に起動しないがpushができる。

######解決したいこと
・なぜ、upload.shとupload.batの改行コードがMacintosh（CR）でないと、pushができないのか。その理由・対処法。
・リソースを正常に動作しかつ、pushも正常に動作させるにはどのようにすればよいのか。

是非、ご教授の程よろしくお願いいたします。

Answer 1

この質問に回答してくださった皆様、ありがとうございます。

この質問につきましては、シェルからバッチを呼ぶ形でフックスクリプトを呼ぶと正常に動作しないことを確認いたしましたので、シェルスクリプトでサーバ接続する方法に切り替えます。

ご教授してくださった皆様、本当にありがとうございました。

Answer 2

こんにちは。

gitの改行コード自動変換をオフにしたうえで
改行コードをCR LFかLFに統一するとうまくいくと思います。

windows環境の git で改行コードの自動変換に注意 - Qiita

なお改行コードは下の二種類と思ってもらっていいです（古いMacのみCR : \r）。

Win CR+LF : \r\n
Unix/Mac LF : \n

Comments

[fukkun0412]

gitの改行コード自動変換は最初からオフになっていました。
質問にあったリソースをWindows（CR LF）・Unix（LF）にしましたが、
まだ正常にpushができません。
エラー内容
./upload.sh: line 37: $'\r': command not found
.git/hooks/pre-push: line 8: english: command not found
error: failed to push some refs to 'Z:\hook_test'

更新対象のリソース（上記で言う".xml"のもの：改行コードはWindows（CR LF））の改行コードも関係があるのでしょうか？

[firedfly]

「./upload.sh」の37行目にある「'\r'」はコマンドではない、
というエラーメッセージで、xmlファイルは関係ないです。
upload.shは本当にLFになってるんでしょうか……？

[fukkun0412]

NotePad++の置換を使用して、LFにしています。改行コードを表示する箇所はLFと表示になっており、NotePad++の機能で改行コード表示すると、LFと表示されているので、LFになっていると思われるのですが、実際はそのほかに何か対応しないといけないのでしょうか？

[fu7mu4]

バイナリエディタで確認してはいかがでしょうか?

[fukkun0412]

fu7mu4さん、ご回答ありがとうございます。
バイナリエディタで確認しましたところ、改行がすべてLF（0A）に
なっておりました。
それでも、上記のエラー（$'\r': command not found）が発生しています。
hookスクリプト自体は問題ないことが分かりました。hookスクリプトから呼び出される、upload.shとupload.batの改行コードに関する事案とわかりましたが、なぜそうなってしまうのかわかりません・・・。

Answer 3

pre-push.sh の 8行目でエラーがでているということは、$iまたは./upload.shがおかしいということになります。この$iがおかしい可能性があります。確認してください。

if [ $? -eq 0 ]; then
  MSG=$(git log -1 --name-only --pretty=format:"%h" | grep strings  | grep .xml)
  cd $(cd $(dirname $0) && pwd)
  for i in ${MSG[@]};do
    echo "Update"
    echo $i
    ./upload.sh $i
  done
fi 

Comments

[fukkun0412]

fu7mu4さんご回答ありがとうございます。
この回答に関しましての返答は以下の物になります。
・$iには正常にファイルパスが格納されている。
　→そのため、$iがおかしいというわけではない
・./upload.shがおかしいと表示されるのはupload.sh内で呼び出しているupload.batの動作に不備があり、./upload.shがおかしいと表示される
　→upload.shの処理自体は問題なく動作はしている。（batの呼び出し含め）
　→batの問題に関しては別の質問として掲載を予定。

以上になります。

Answer 4

エラーメッセージにそのまま書いてあるじゃないですか

$'\r': command not found

\rなんてコマンドは知りませんって。
「改行コードがLFの場合」にコレが出てるってことは、carriage returnがおかしな変換されているんじゃないですかね。
すべてのファイルで改行コードを統一して下さい。

Comments

[fukkun0412]

該当するリソースの改行コードを統一しましたが、同じくpushがうまくできませんでした。

更新対象のリソース（上記で言う".xml"のもの：改行コードはWindows（CR LF））の改行コードも関係があるのでしょうか？

[kunai]

うまくいかないとはどういう状況か教えて下さい。

[fukkun0412]

上記に記載されているエラー内容
./upload.sh: line 37: $'\r': command not found
.git/hooks/pre-push: line 8: english: command not found
error: failed to push some refs to 'Z:\hook_test'
これが表記されました。

upload.shとupload.batは正常に動作しているのですが、SourceTreeのpushがうまくいきません。

[kunai]

LFに正しく変換されていれば、そもそも\rが登場してこないと思うんですよ。
LFに統一した、と言うのは、どういう作業でLFにされているんでしょうか。
また、pre-push.shからuploadが呼ばれているんだと思いますが、pre-push側のコードを開示出来ませんか。

[fukkun0412]

pre-pushのコードを記載しました。
LFへの変換に関しては、NotePad++の置換を使用しています。改行コードを表示する箇所はLFと表示になっており、NotePad++の機能で改行コード表示すると、LFと表示されているので、LFになっていると思われるのですが、実際はそのほかに何か対応しないといけないのでしょうか？

[fu7mu4]

.xmlもLFにしてください。

[fukkun0412]

fu7mu4さん
.xmlの改行コードをLFにしてみましたが、変わらずpushができませんでした。