Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • L lalsuite
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 114
    • Issues 114
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 40
    • Merge requests 40
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • lscsoft
  • lalsuite
  • Merge requests
  • !2059

Prepend RPATH of install/build directories to AM_LDFLAGS

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Karl Wette requested to merge ANU-CGA/lalsuite:fix-rpath into master Nov 27, 2022
  • Overview 9
  • Commits 5
  • Pipelines 5
  • Changes 50

Description

  • Prevent being overridden by other libraries or environment (e.g. the LDFLAGS set by the default Conda environment at CIT).
  • top-level and *:pkg CI jobs now test building in environment with LDFLAGS=/usr/lib64 and LALSuite packages installed in /usr/lib64.
  • Add installcheck tests of the lal*_version executables; check that these are reporting the correct information, an error would indicate linking against the wrong libraries. (This is easier than directly probing with ldd which libraries are linked to, as that would involve more platform-specific code, e.g. ldd vs otool.)

Closes #633

API Changes and Justification

Backwards Compatible Changes

  • This change does not modify any class/function/struct/type definitions in a public C header file or any Python class/function definitions
  • This change adds new classes/functions/structs/types to a public C header file or Python module

Backwards Incompatible Changes

  • This change modifies an existing class/function/struct/type definition in a public C header file or Python module
  • This change removes an existing class/function/struct/type from a public C header file or Python module

If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.

Review Status

Please provide details on any reviews related to this change and and the associated reviewers.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: fix-rpath