Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • LidarView LidarView
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 6
    • Issues 6
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 5
    • Merge requests 5
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • LidarView
  • LidarViewLidarView
  • Merge requests
  • !55

[fix] Instruct OSX to bundle LidarSLam.dylib

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Arnaud Billon requested to merge fix/sb-improvements into master Mar 25, 2021
  • Overview 1
  • Commits 3
  • Pipelines 0
  • Changes 6

LidarSlamPlugin is not self contained in a single shared library, this is somewhat unconventional, it requires an additional install of LidarSlam.dylib, this MR addresses this problem by copying LidarSlam.dylib when needed.

This raises some questions with regards to LidarSlam compilation, format and presence within lidarview's superbuild.

Also moved the OSX-RPATH boost patch into lidarview's cmake, in order for it to be properly triggered by the CI.

  • Required Merge lidarview-superbuild!22 (merged) first
  • Update SHA as required
Edited Mar 30, 2021 by Arnaud Billon
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix/sb-improvements