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

closes https://github.com/assimp/assimp/issues/3007 #3008

Closed
wants to merge 2 commits into from

Conversation

kimkulling
Copy link
Member

  • Remove named ELSE and ENDIF statements from cmakefiles.
  • Set required cmake version up to 3.0

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.003%) to 51.708% when pulling 62815b7 on issue_3007 into a4868a9 on master.

@kimkulling kimkulling added this to In progress in Planning for Release V5.1 Feb 21, 2020
@kimkulling kimkulling moved this from In progress to ToDo in Planning for Release V5.1 Feb 21, 2020
@kimkulling kimkulling closed this Feb 25, 2020
@kimkulling kimkulling deleted the issue_3007 branch February 25, 2020 19:44
malortie pushed a commit to malortie/assimp that referenced this pull request Feb 26, 2020
This is a follow-up to PR assimp#3024, in which I explained that I would then change the version of CMake to 3.0 in several files to match the changes that were originally made by OP in PR assimp#3008.

The minimum CMake version have been raised to 3.0 in both assimp_cmd and assimp_view projects.

The same CMake configuration as in PR assimp#3024 was used. All projects from PR assimp#3024 are found in the solution with the current changes. All projects built. All tests were run.
@kimkulling kimkulling moved this from ToDo to Done in Planning for Release V5.1 May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants