Skip to content

[fix] CmakeLists.txt now works on ubuntu 16.04 without the need to compile oce#34

Merged
aothms merged 1 commit into
IfcOpenShell:masterfrom
yilativs:master
Feb 24, 2016
Merged

[fix] CmakeLists.txt now works on ubuntu 16.04 without the need to compile oce#34
aothms merged 1 commit into
IfcOpenShell:masterfrom
yilativs:master

Conversation

@yilativs

Copy link
Copy Markdown
Contributor

[fix] CmakeLists.txt now works on ubuntu 16.04 without the need to compile oce
instead oce and opencollada dev packages should be installed with the command below:

apt-get install git swig cmake gcc g++ libftgl-dev libtbb2 libtbb-dev libboost-all-dev libgl1-mesa-dev libfreetype6-dev liboce-modeling-dev opencollada-dev liboce-foundation-dev

…mpile oce

instead you can
apt-get install git swig cmake gcc g++ libftgl-dev libtbb2 libtbb-dev libboost-all-dev libgl1-mesa-dev libfreetype6-dev liboce-modeling-dev opencollada-dev liboce-foundation-dev
@aothms

aothms commented Feb 24, 2016

Copy link
Copy Markdown
Member

Hi,

Thanks for this. I think this is an improvement but let me elaborate a bit. The /usr/include/opencascade/ path is the installation path of the "official" Open Cascade repository [1]. Over time I came to prefer the community edition [2] due its convenient cmake build system and more open character (occt didn't even have a public git repo at that time). It would be good if this is reflected in the IfcOpenShell/CMake file. Secondly, note that these are merely defaults. Users can override these with cmake -DOCC_INCLUDE_DIR=/path/ or with env vars. I am happy though that the defaults are set correctly for Debian/Ubuntu.

[1] http://git.dev.opencascade.org/gitweb/?p=occt.git
[2] https://github.com/tpaviot/oce

Kind regards,
Thomas

aothms added a commit that referenced this pull request Feb 24, 2016
Set cmake search paths to ubuntu 16.04 package manager defaults
@aothms aothms merged commit acc4af8 into IfcOpenShell:master Feb 24, 2016
theoryshaw added a commit that referenced this pull request May 21, 2026
Both PR #7798 (ManualDrawingReference) and PR #8083 (parametric_dimensions)
independently added properties after entry #33 in Psets_BBIM_Annotation.ifc
and new BoolProperty declarations after `type_name` in prop.py and workspace.py
UI rows at the same locations.

Resolution: keep #8083's IDs (#34-#40) unchanged, renumber #7798's
IsManualDrawingReference and IsDocumentReference entries to #41 and #42,
update EPset_Annotation reference list accordingly, keep both UI rows,
and combine hotkey_S_A to use #8083's parametric-dimension routing
with #7798's "INVOKE_DEFAULT" argument for add_annotation.
theoryshaw added a commit that referenced this pull request May 22, 2026
Both PR #7965 (inset_section_endpoints) and PR #8083 (parametric_dimensions)
independently added properties after entry #33 in Psets_BBIM_Annotation.ifc
and new BoolProperty declarations after `type_name` in prop.py and workspace.py
UI rows at the same locations.

Resolution: keep #8083's IDs (#34-#40) unchanged, renumber #7965's
IsManualDrawingReference and IsDocumentReference entries to #41 and #42,
update EPset_Annotation reference list accordingly, keep both UI rows,
and combine hotkey_S_A to use #8083's parametric-dimension routing
with #7965's "INVOKE_DEFAULT" argument for add_annotation.
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