-
Notifications
You must be signed in to change notification settings - Fork 17
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
Use pkg-config for wld #5
Comments
NOTE: for some reason, this makes the repo unable to compile on Gentoo until I added the WLD links/sources AND the pixman sources (before WLD in the cflags). I think it has to due with the Makefile unable to look up the pixman sources at that point (was getting a lot of undefined references to pixman typedefs). |
I could add WLD as a git submodule maybe. |
I'm not too particular on how it gets included (but a submodule might be better). I was more referring to the fact I'd just rather use my system WLD over the included WLD (via pkg-config) if it is available. Do you foresee any issues with that? |
possibly, I made some changes to WLD a while ago (forgot what they were) so I bundled WLD because the library seemed dead. |
Gotcha. I just wanted to verify if there were customizations. I wanted to
use the native wld on this system (using it with velox), but I'll stick
with the one bundled with the repo.
…On Mar 14, 2018 5:00 AM, "Jeff" ***@***.***> wrote:
Do you foresee any issues with that?
possibly, I made some changes to WLD a while ago (forgot what they were)
so I bundled WLD because the library seemed dead.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#5 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAs_ioqIrJi705vbBrb5AEDvZNqlS-Ysks5teQZ2gaJpZM4SpFi9>
.
|
Is there a specific reason to include the wld sources in the git repository? I guess it means there are less dependencies to download, but I think we'd be better off with pkg-config (or at least prefer the users' library rather than the one included). This way we can update wld separately, plus lots of people might already have wld compiled (velox users).
The text was updated successfully, but these errors were encountered: