Skip to content
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

Fix package name for Ubuntu #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dkiszkiel
Copy link

No description provided.

@jkolb
Copy link
Member

jkolb commented Feb 1, 2018

This library currently using a manually installed version of GLFW because the current version 3.2.1 has bugs on Mac and Linux that were causing issues that are fixed in 3.3 which wasn't available yet through apt. When you install it manually it doesn't have the "-dev" suffix that is added by the apt tool.

There are also short comings in the Swift Package manager that are being worked on around trying to integrate better with system libraries with various paths and names on different operating systems. Supposedly you can use an option that talks to pkgconfig (I think that is what it is called) that is a program that can be installed that helps abstract the names and paths out of the Package.swift file but I don't think it is something that is as easy as it sounds as it is not something that is installed by default especially on macOS.

In the mean time I would like to keep this as is and require a manual install of GLFW. I will be looking at this again when Swift 5 comes out and hopefully there are better options by then or at least GLFW will be fully on version 3.3 and things will be easier.

@jkolb
Copy link
Member

jkolb commented Feb 1, 2018

I just realized the docs are not updated to reflect what I just said, sorry about that. I am helping out with keeping things current with newer Swift versions but have had limited time to make things as polished as they could be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants