Requirements.txt install fails on non-windows platforms

TheresAFewConors / Sooty

The SOC Analysts all-in-one CLI tool to automate and speed up workflow.

GNU General Public License v3.0

1.34k stars 209 forks source link

Requirements.txt install fails on non-windows platforms #28

Closed ekelson-bcove closed 4 years ago

ekelson-bcove commented 4 years ago

requirements.txt file specifies pywin32 as a requirement, however this only applies if you are using windows. It will fail if you are not. Changing to pywin32==224; sys_platform == ‘windows’ will fix issue.

TheresAFewConors commented 4 years ago

Issue relates to #29 'Fixed Requirements File'.

TheresAFewConors commented 4 years ago

PR approved and merged with main branch.

Good fix, adding to contributors section

TheresAFewConors commented 4 years ago

Added to contributors. Closing issue.