Skip to content

GitLab

  • Menu
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 120
    • Issues 120
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 39
    • Merge requests 39
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & 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
  • !1683

Draft: Remove unnecessary VCS/Build info makefile variables

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Duncan Macleod requested to merge duncanmmacleod/lalsuite:cleanup-vcs-makefile into master Oct 01, 2021
  • Overview 0
  • Commits 11
  • Pipelines 1
  • Changes 10

Description

This MR attemps to clean up some build infrastructure by removing variables that no longer serve any purpose:

  • vcs_build_info_source: this was defined only in order to declare prerequisites for it, but since in all cases the named file is generated by autoconf it will almost certainly predate everything else in the build by default, so this makes no sense (any more)
  • build_info_dependencies: this list defined prerequisites for the $(build_info_source) target, however that target is so simple that it doesn't make sense for it to depend on the existence of other files (any more)

In all this should have no visible impact on the build.

API Changes and Justification

Backwards Compatible Changes

  • This change introduces no API changes
  • This change adds new API calls

Backwards Incompatible Changes

  • This change modifies an existing API
  • This change removes an existing API

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.

Edited Oct 01, 2021 by Duncan Macleod
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: cleanup-vcs-makefile