Update clang format to 8 (!872) · Merge requests · Gaudi / Gaudi · GitLab

Snippets Groups Projects

Merged Marco Clemencic requested to merge clemenci/Gaudi:update-clang-format into master 6 years ago

All threads resolved!

The main reason to update from 7 to 8 is, for me, the fix to bug https://bugs.llvm.org/show_bug.cgi?id=35641

Apart from that, the new version only makes minor changes to the current version, which, I think, improve readability.

Ubuntu 18.04 users, clang-8 can be installed via the binary tarball from http://releases.llvm.org/download.html, while CentOS 7 users can pick up the SFT build in /cvmfs/sft.cern.ch/lcg/releases/clang/8.0.0-ed577

Edited 5 years ago by Marco Clemencic

Activity

Marco Clemencic changed milestone to %v32r0 6 years ago

changed milestone to %v32r0
Marco Clemencic added 112 commits 5 years ago
added 112 commits

c2565d95...51763dbb - 110 commits from branch gaudi:master

639dfeb7 - Update clang-format version to 8

cea0f975 - Minor changes to the formatting due to clang-format 8

Compare with previous version
Marco Clemencic added 3 commits 5 years ago
added 3 commits

beff7b98 - Update reference file for produce_consume test

abf215ab - Update clang-format version to 8

5e5102af - Minor changes to the formatting due to clang-format 8

Compare with previous version
Marco Clemencic mentioned in merge request !908 (closed) 5 years ago

mentioned in merge request !908 (closed)
Marco Clemencic added 26 commits 5 years ago
added 26 commits

5e5102af...9d5388da - 24 commits from branch gaudi:master

a3ba5d37 - Update clang-format version to 8

9f75da78 - Minor changes to the formatting due to clang-format 8

Compare with previous version
Marco Clemencic unmarked as a Work In Progress 5 years ago

unmarked as a Work In Progress
Marco Clemencic added 2 commits 5 years ago
added 2 commits

84b93a07 - Use private wrapper for SFT clang-format if not found elsewhere

dff42275 - Minor changes to the formatting due to clang-format 8

Compare with previous version
Marco Clemencic @clemenci started a thread on the diff 5 years ago

Resolved 5 years ago by Marco Clemencic
Last reply by Marco Clemencic 5 years ago

Marco Clemencic resolved all discussions 5 years ago

resolved all discussions

Marco Clemencic mentioned in issue #71 (closed) 5 years ago

mentioned in issue #71 (closed)

Marco Clemencic @clemenci · 5 years ago

Author Owner

I implemented @fwinkl suggestion. This can now go in as we discussed in the meeting.

Marco Clemencic assigned to @leggett 5 years ago

assigned to @leggett

Software for LHCb @lhcbsoft · 5 years ago

[2019-05-16 00:03] Validation started with lhcb-sanitizers#247
[2019-05-16 00:03] Validation started with lhcb-soa-track#79
[2019-05-16 00:03] Validation started with lhcb-lcg-dev4#902
[2019-05-16 00:03] Validation started with lhcb-test-throughput2#9
[2019-05-16 00:04] Validation started with lhcb-dd4hep#179
[2019-05-16 00:05] Validation started with lhcb-lcg-dev3#894
[2019-05-16 00:06] Validation started with lhcb-tdr-test#547
[2019-05-16 00:09] Validation started with lhcb-gaudi-head#2254

Edited 5 years ago by Software for LHCb

Charles Leggett approved this merge request 5 years ago

approved this merge request

Charles Leggett merged 5 years ago

merged

Charles Leggett mentioned in commit 5 years ago

mentioned in commit 47d5e552

Marco Clemencic mentioned in merge request lhcb-core/LbDevTools!59 (merged) 5 years ago

mentioned in merge request lhcb-core/LbDevTools!59 (merged)

Charles Leggett mentioned in merge request atlas/athena!23528 (merged) 5 years ago

mentioned in merge request atlas/athena!23528 (merged)

Please register or sign in to reply