-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add requirements file of non-python dependencies - fix issue #34 #200
Conversation
…es), named "requirements.apt" - fix issue #34
…ependencies) listed on requirements.apt file
…ect (local, test, and production dependencies)
@pydanny @theskumar review me please if you are good, I'll do some documentation |
👍 |
apt-get update | ||
|
||
# Install the basic compilation dependencies and other required libraries of this project | ||
cat requirements.apt | grep -v "#" | xargs sudo apt-get install -y |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These commands will not only install all packages that are necessary, but will also upgrade any packages already installed to the latest version. That may not be a bad thing, but I prefer to err on the side of caution when doing root-level administration of somebody else's system. Having an --(no-)upgrade
option, or at least warning the user in the docs that packages will be updated would be better.
add .idea on .gitignore
@pydanny @theskumar @dupuy I refactored the install_os_dependencies.sh file. Is good now or I missed some more error? |
👍 |
add requirements file of non-python dependencies and helper scripts - fix issue #34
No description provided.