Add ifc open shell recipe

[EstebanDugueperoux2]

Summary

Changes to recipe: lib/[version]

Motivation

Details

---

• Read the contributing guidelines
• Checked that this PR is not a duplicate: list of PRs by recipe
• Tested locally with at least one configuration using a recent version of Conan

[ecoezen]

Hey. A hint: don't forget to patch add_debug_variants(...) calls ( line 370, 471 and 587 on CMakeLists) as they cause failure with Visual Studio generators. I'll try to support you whenever I find some time (and, ofc, if you need :) )

Great effort, thanks in advance!

[EstebanDugueperoux2]

Hi @ecoezen , thanks for your feedback, are you a IfcOpenShell user?
Currently I have random error for Windows platform about cpp syntax errors on IfcOffsetCurveByDistance (https://c3i.jfrog.io/artifactory/cci-build-logs/cci/prod/PR-27623/17/package_build_logs/build_log_ifcopenshell_0_8_2_cci_20250407_67c5eb8d395510ea87b1de553fde0400_4f41b621d54c4fb83dd818c90e0b754f4d378c3f.txt)

which I don't explain and personaly my workstation is on Linux.

Also for Mac platform I have a "Undefined symbols for architecture arm64" error which I don't explain no more.

Regards.

[ecoezen]

Hey. I actually sent you a pr yesterday—explains exactly that: EstebanDugueperoux2#12

"/permissive- fixes a build failure in MSVC when using alternate operators (&&:and - ||:or) (in IfcOffsetCurveByDistance.cpp). Could be worked around by including in C++17, but it’s removed in C++20+"

So, there are 3 ways to fix the error. Replace and/or keywords with classical operators in IfcOffsetCurveByDistance.cpp as source patch. Second, patch IfcOffsetCurveByDistance.cpp to include ciso646 header. Third, set conformance mode permissive- flag. But all these workarounds are only required when below cpp20.

You may also receive a compilation error about exception handling, set eHsc to observe solution.

[ecoezen]

Hi @ecoezen , thanks for your feedback, are you a IfcOpenShell user?
Currently I have random error for Windows platform about cpp syntax errors on IfcOffsetCurveByDistance (https://c3i.jfrog.io/artifactory/cci-build-logs/cci/prod/PR-27623/17/package_build_logs/build_log_ifcopenshell_0_8_2_cci_20250407_67c5eb8d395510ea87b1de553fde0400_4f41b621d54c4fb83dd818c90e0b754f4d378c3f.txt)

which I don't explain and personaly my workstation is on Linux.

Also for Mac platform I have a "Undefined symbols for architecture arm64" error which I don't explain no more.

Regards.

It seems the same error appears on macos ci as well (and probably only when building with cpp17 and below). If so, the best solution would be patch IfcOffsetCurveByDistance.cpp to include ciso646 header when building with cpp17 and below.

[ecoezen]

hey again @EstebanDugueperoux2 . seems like it resolves failures on windows build. you might need to update test_package.cpp (you may use the one in EstebanDugueperoux2#12)

also, would you be okay me to contribute and push to directly on your branch later today--when i got some time?

[EstebanDugueperoux2]

Hi @ecoezen,
yes you can propose PR to target my branch but it seems there are too many changes on EstebanDugueperoux2#12.
I'll open a issue on ifcopenshell project to talk about conan recipe contribution and propose changes directly to ifcopenshell to avoid patch in our recipe.

Regards.

[EstebanDugueperoux2]

Hi @AbrilRBS,

Do you know why Linux builds take such time?
We have no visibility on logs.

Regards.

[AbrilRBS]

@EstebanDugueperoux2 the builds were stopped after the failures in the other configurations, but Jenkins was a bit too eager and did not wait for the jobs to submit the update that they were being cancelled to the github check suite