のMakefileのpep8チェックのみのgit diffを

Question

我々はすべてのpythonファイルにPEP8の準拠を確認するために私たちのレポのMakefileにpep8ターゲットを使用します。

## Check all python files for compatibility with PEP8 
PEP8FLAGS := --ignore=E201,E202,E241,E265,E501,E711,E712,E713,E714,E721,W391,W291,W293,W601,W603 
pep8: 
    pep8 $(PEP8FLAGS) . 

私だけチェックに対して同様のpep8-changes目標を持っているしたいのですが

DIFF_FILES := (git diff --name-only --diff-filter=ACMR ; \ # ACMR: added/copied/modified/renamed 
      git diff --staged --name-only --diff-filter=ACMR ; \ 
      git diff --name-only --diff-filter=ACMR upstream/master...) \ 
| sort | uniq | grep -e "\.py$$" | grep -v '__init__.py' 

## Check all diff python files for compatibility with PEP8 
pep8-changes: 
    pep8 $(PEP8FLAGS) --filename $(DIFF_FILES) 

しかし、私は次のエラーを取得：Pythonのファイルは、プル要求に変更

pep8 --exclude=sandbox,thirdparty --ignore=E201,E202,E241,E265,E501,E711,E712,E713,E714,E721,W391,W291,W293,W601,W603 --filename (git diff --name-only --diff-filter=ACMR ; \ 
/bin/sh: -c: line 0: syntax error near unexpected token `(' 

0を

pep8 docsで私は多くのヘルプを見つけることができません。私は--diffフラグが私がここで探しているものだとは思わない。どんな助けもありがとう！

Answer 1

大丈夫です。問題は、$ charをエスケープしているようです。

DIFF_FILES := git diff --name-only --diff-filter=ACMR upstream/master... \ 
| sort | uniq | grep -e "\.py$$" | grep -v '__init__.py' 

pep8-changes: 
    pep8 $(PEP8FLAGS) `$(DIFF_FILES)`