-
Notifications
You must be signed in to change notification settings - Fork 211
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
Added build deps to README.md #304
base: master
Are you sure you want to change the base?
Conversation
@JJK96 AFAICT those are ubuntu/debian specific dependencies, aren't they? |
Yes, but I think people who are smart enough to compile packages are smart enough to find the corresponding package for their system. |
Ok, then we could just say "Depends on FreeImage", right? |
i guess we could, but it also depends on the -dev (development) version of
|
No it doesn't depend on the "development versions". It depends on freeimage and libc6. One dependency is for building, the other one is for running. I believe we should list those necessary for running. Having necessary dev-packages for building should be common sense for users (packagers and coders) using such distros. |
Thank you for your clarification, I'm okay with that.
|
Thanks for the clarification Jay. I think that's way more reasonable, since every distro is different and this is Linux (or *unix) specific. |
@@ -22,6 +22,9 @@ Official binary sources (provided by davidgfnet): | |||
Building | |||
-------- | |||
|
|||
Dependencies: | |||
Libc6-dev, libfreeimage-dev |
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.
Should be something like: "Development headers and libraries for glibc and Freeimage. You'll also need a C/C++ compiler and linker such as GCC".
Possibly add a debian example: "Example (debian): sudo apt-get install build-essential libfreeimage-dev" [untested, not a debian user myself but I do realize that especially debain users need help with this..]
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.
Hahahaha do they? Are Debian users retarded? :P Yeah lots of Ubuntu folks, but come on, they do have a launchpad repo... Let them suffer a bit
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.
A lot of linux users start with an ubuntu based distro, so as a result,
most "beginners" use ubuntu/debian. I think we should start getting a
little more beginner friendly if we want linux to be more popular, and not
ridicule them.
On Aug 13, 2015 1:16 PM, "David G. F." [email protected] wrote:
In README.md
#304 (comment)
:@@ -22,6 +22,9 @@ Official binary sources (provided by davidgfnet):
Building
+Dependencies:
+Libc6-dev, libfreeimage-devHahahaha do they? Are Debian users retarded? :P Yeah lots of Ubuntu folks,
but come on, they do have a launchpad repo... Let them suffer a bit—
Reply to this email directly or view it on GitHub
https://github.com/davidgfnet/whatsapp-purple/pull/304/files#r36963608.
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.
No description provided.