Ticket #9479 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Update code to ready for latest version of OpenCascade (6.7.1)

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: critical Milestone: Release 3.2
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Parker

Description (last modified by Martyn Gigg) (diff)

Some unit tests fail when running the code built on Windows 8. They all relate to using the OpenCascade function BRepMesh::Mesh(Result,0.001);. The OpenCascade changelogs suggest that there was a bug on 64-bit platforms in functions that the above method calls.

Upgrading Opencascade locally seems to solve the crashing issue on Windows 8.

This ticket will be about putting backwards-compatible changes into the code that will allow third party to be updated.

Change History

comment:1 Changed 6 years ago by Martyn Gigg

Note: We need to upgrade the Mac version too as we only have a single repository of include files.

comment:2 Changed 6 years ago by Martyn Gigg

  • Status changed from new to assigned
  • Description modified (diff)

comment:3 Changed 6 years ago by Martyn Gigg

  • Status changed from assigned to inprogress

comment:4 Changed 6 years ago by Martyn Gigg

Minor updates for OpenCascade 6.7.1 on Windows.

All changes will be compatible with older versions on other systems. Refs #9479

Changeset: 6e41e0638b37a76fa77fc944056801b310a5db41

comment:5 Changed 6 years ago by Martyn Gigg

Kill OpenCascade gcc warnings about const casts

Refs #9479

Changeset: 97bc5454d9551e78f66910aa3b78c158ad8facf2

comment:6 Changed 6 years ago by Martyn Gigg

  • Status changed from inprogress to verify
  • Resolution set to fixed
  • Description modified (diff)
  • Summary changed from Update OpenCascade on Windows/Mac to latest version (6.7.1) to Update code to ready for latest version of OpenCascade (6.7.1)

Branch: ​feature/9479_update_opencascade_libs

Tester: Check by code review and ensuring that all tests/static checks are passing on the develop branch. There were no libraries updated here. This will be done once this ticket is merged to master

comment:7 Changed 6 years ago by Peter Parker

  • Status changed from verify to verifying
  • Tester set to Peter Parker

comment:8 Changed 6 years ago by Peter Parker

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/9479_update_opencascade_libs'

Full changeset: ​9b9109e17491032e1775180cdef13b83625a9bf7

comment:9 Changed 6 years ago by Peter Parker

Tests and static checks are passing on develop branch.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​10322

Note: See TracTickets for help on using tickets.