Skip to content

Bring IfcMax up to date#35

Merged
aothms merged 5 commits into
IfcOpenShell:masterfrom
Tridify:ifcmax
Feb 27, 2016
Merged

Bring IfcMax up to date#35
aothms merged 5 commits into
IfcOpenShell:masterfrom
Tridify:ifcmax

Conversation

@Stinkfist0

Copy link
Copy Markdown
Contributor

Tested with VS 2015 and 3ds Max 2016 + SDK by importing acad2010_objects.ifc. Closes #13
Having CMakeLists.txt complete for all projects makes the old manually crafted VS solution and project files obsolete so I removed them.

screenshot 2016-02-19 21 32 56

@Stinkfist0

Copy link
Copy Markdown
Contributor Author

Thought solution folders might be a nice touch, although the project is not very large yet. Could maybe add Applications folder or something for IfcConvert and IfcGeomServer, if wanted.
solutionfolders

@Stinkfist0

Copy link
Copy Markdown
Contributor Author

If I may suggest, cmake/CMakeLists.txt could be now maybe moved to the project root which seems to be typical for CMake-based projects.

@Stinkfist0

Copy link
Copy Markdown
Contributor Author

Documented GCC 4.7 as the minimum requirement due to OpenCOLLADA requirements.

@aothms aothms mentioned this pull request Feb 27, 2016
6 tasks
@aothms

aothms commented Feb 27, 2016

Copy link
Copy Markdown
Member

Hi, many thanks for this. Sorry for the slow reply. I do have some small points for improvements, but for me they are of low priority as I expect the intersection of 3dsmax users and people compiling their own software rather small. I created a separate issue to track them [1] and will get to it in good time. Major thing is that everything is covered from within cmake, which is a major step forward. Thanks for your efforts.

[1] #40

aothms added a commit that referenced this pull request Feb 27, 2016
Create CMake file for building IfcMax
@aothms aothms merged commit 90f72c4 into IfcOpenShell:master Feb 27, 2016
@Stinkfist0 Stinkfist0 deleted the ifcmax branch March 7, 2016 19:43
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