Skip to content

Commit

Permalink
COMPAT: Do not run xargs with -r on OSX
Browse files Browse the repository at this point in the history
OSX xargs has no -r, but it still terminates
when xargs is empty, which was the whole point
of passing in that flag in the first place.

[ci skip]
  • Loading branch information
gfyoung committed Mar 20, 2017
1 parent da57857 commit d1543b5
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions doc/source/contributing.rst
Original file line number Diff line number Diff line change
Expand Up @@ -529,6 +529,11 @@ run this command, though it will take longer::

git diff master --name-only -- '*.py' | grep 'pandas/' | xargs -r flake8

Note that on OSX, the ``-r`` flag is not available, so you have to omit it and
run this slightly modified command::

git diff master --name-only -- '*.py' | grep 'pandas/' | xargs flake8

Backwards Compatibility
~~~~~~~~~~~~~~~~~~~~~~~

Expand Down

0 comments on commit d1543b5

Please sign in to comment.