Skip to content
Snippets Groups Projects

A number of minor fixes to build on MacOS

Merged Pere Mato Vila requested to merge mato/Gaudi:mato-fixes-MacOS into master
All threads resolved!
  • project(Gaudi) is just to avoid a warning
  • Forward declaration of std::chrono::duration produces a compilation error: error: reference to 'chrono' is ambiguous
  • std::tuple of a literal string produces the following error: no viable constructor or deduction guide for deduction of template arguments of 'tuple' auto t = std::tuple{500, 1.0, "sec"};
  • The Apple clang version starts with a two digit number.

Merge request reports

Pipeline #1438110 passed

Pipeline passed for d44b70fb on mato:mato-fixes-MacOS

Merged by Marco ClemencicMarco Clemencic 5 years ago (Feb 26, 2020 9:05am UTC)

Merge details

  • Changes merged into master with 8de22c80.
  • Did not delete the source branch.

Pipeline #1442949 passed

Pipeline passed for 8de22c80 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • I implemented the changes I propose to ChronoEntity.cpp in clemenci/Gaudi@194dc619

  • Pere Mato Vila added 1 commit

    added 1 commit

    • 4d512dbf - Apply suggestion to GaudiKernel/src/Lib/ChronoEntity.cpp

    Compare with previous version

  • Pere Mato Vila resolved all threads

    resolved all threads

  • Pere Mato Vila added 2 commits

    added 2 commits

    Compare with previous version

  • Pere Mato Vila resolved all threads

    resolved all threads

  • Pere Mato Vila added 2 commits

    added 2 commits

    • 3f57b80b - Fixed formatting
    • d44b70fb - Make it compatible with older versions of rangesv3

    Compare with previous version

  • Edited by Software for LHCb
  • Marco Clemencic approved this merge request

    approved this merge request

  • Marco Clemencic mentioned in commit 8de22c80

    mentioned in commit 8de22c80

  • Please register or sign in to reply
    Loading