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

Require Assimp < 3.0~dfsg-4 #455

Closed
wants to merge 1 commit into from

Conversation

mkoval
Copy link
Collaborator

@mkoval mkoval commented Jul 17, 2015

This pull request makes DART depend on the version of Assimp that is missing C++ symbols; see #451 for details. This should stop DART from breaking when Assimp 3.0~dfsg-4 is released on Ubuntu. It would be good if @jslee02 could release a new version of DART with this change to stave off any potential problems.

Next, I will build the Debian packages for the updated version of Assimp and work with @jslee02 to get them in the DART PPA. Once that is done, we can modify the debian/control file to depend on libassimp-dev (>= 3.0~dfsg-4) and release a version of DART without the custom aiScene methods.

@jslee02 jslee02 added this to the DART 5.1.0 milestone Jul 17, 2015
@jslee02
Copy link
Member

jslee02 commented Jul 17, 2015

It would be good if @jslee02 could release a new version of DART with this change to stave off any potential problems.

We have not discussed about the next release schedule yet but I'm thinking to release DART 5.0.1 around Aug 3 that would just include patches. If you want to ship this change in DART 5.0.1, please change the target branch to release-5.0. Current master branch is for DART 5.1.0 (or 6.0.0).

Note that all the patches of DART 5.0.1 will be applied to the next version of DART as well by merging release-5.0 into master.

Beside the target branch 👍

@mkoval
Copy link
Collaborator Author

mkoval commented Jul 17, 2015

Yes, that makes sense. This should be included in 5.0.1. I will make a new pull request against that target.

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