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

Remove the gl_common crate #359

Merged
merged 1 commit into from
Nov 4, 2015
Merged

Conversation

tomaka
Copy link
Collaborator

@tomaka tomaka commented Nov 4, 2015

So I was the one who proposed this crate. This was in fact a very stupid idea because if we ever do a breaking change the code in gl_common, everything will break (just like libc).

Crates.io show this: https://crates.io/crates/gl_common/reverse_dependencies

Glutin imports the crate to implement the trait in it, and glium imports the trait to call the load function. All other crates only import gl_common because it is required for gl_generator to compile.

Therefore this PR would only break glutin and glium, which I can easily fix.

@tomaka
Copy link
Collaborator Author

tomaka commented Nov 4, 2015

In fact glium doesn't even call load. gl_common is imported but is used only to make gl_generator compile.

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.

None yet

2 participants