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

Depends on gfx-rs? #61

Closed
jasonroelofs opened this issue Sep 15, 2014 · 6 comments
Closed

Depends on gfx-rs? #61

jasonroelofs opened this issue Sep 15, 2014 · 6 comments

Comments

@jasonroelofs
Copy link

I'd like to request that 5bd14ff be reverted. Piston tries to set up an environment where you can choose SDL2 or GLFW separately from the rendering library in question, whether it be opengl_graphics, gfx_graphics, etc, depending on what you want. This commit now forces gfx on everyone who wants to use SDL2 with Piston, breaking this contract Piston is trying to develop.

@jasonroelofs
Copy link
Author

Specifically, in my case, I want to use piston, sdl2_game_window, and opengl_graphics. Because of this change I'm now forced to include gfx, gfx_gl, gfx_macros, device, and render, which is currently putting me unable to work on my project because one of these has a syntax error with the latest Rust build.

@coeuvre
Copy link
Member

coeuvre commented Sep 16, 2014

I agree with you.

Since gfx was broken, I have to drop it in my local sdl2_game_window clone and make every repos point to it to do further development these days.

@coeuvre
Copy link
Member

coeuvre commented Sep 16, 2014

Maybe we should design a new architecture which people can choose between sdl2_game_window and glfw_game_window while underlying the window implementation they can still choose between opengl_graphics and gfx_graphics.

@bvssvni
Copy link
Member

bvssvni commented Sep 16, 2014

We can add a notice to README about how to set up gfx and let the user do this manually.

This was referenced Sep 16, 2014
@bvssvni
Copy link
Member

bvssvni commented Sep 16, 2014

I added notice to both READMEs and removed the Gfx dependency.

@coeuvre Would you mind updating piston-examples? I got a problem with Rust nightlies on mac. PistonDevelopers/piston#647

Opened up a new issue for discussing a better architecture PistonDevelopers/piston#648

Closing.

@bvssvni bvssvni closed this as completed Sep 16, 2014
@coeuvre
Copy link
Member

coeuvre commented Sep 16, 2014

Sure.

You problem must be rust-lang/rust#17192 that you can't get the latest nightly rust on mac. Since it was merged, you can try now.

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

No branches or pull requests

3 participants