Skip to content

Commit

Permalink
Distribution: Fix broken setup_requires
Browse files Browse the repository at this point in the history
The `pyyaml` dependency is needed in `install_requires` but also in
`setup_requires`, because running `setup.py` requires importing
`yamllint`, which itself imports `yaml`.
  • Loading branch information
adrienverge committed Jan 24, 2016
1 parent 7983c66 commit e3ebea6
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,8 @@
scripts=['bin/yamllint'],
package_data={'yamllint': ['conf/*.yml']},
install_requires=['pyyaml'],
setup_requires=['pyyaml'], # importing `yamllint` (for APP_NAME etc.)
# requires importing `yaml`
tests_require=['nose'],
test_suite='nose.collector',
)

0 comments on commit e3ebea6

Please sign in to comment.