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

Update russimp-sys to use new build system. #50

Closed
wants to merge 3 commits into from

Conversation

tomheaton
Copy link

@tomheaton tomheaton commented Jun 29, 2023

Update russimp-sys to use new build system as well as strum and strum_macros.

Note: due to the new build system, this fixes issues on windows as well as link issues with other binding libs

@jkvargas
Copy link
Owner

jkvargas commented Jul 2, 2023

hi Tom, thanks so much for your contribution.
I didn't update russimp to use russimp-sys 2.0.0 yet because it requires you to have an assimp header file available.
right now russimp only requires the assimp binary available.
I am still thinking on a solution since this change will cause problems for some users.

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