1603436 - Rename clang patch to make easier its removal when we'll switch to LLVM 10

Status: RESOLVED FIXED

(Firefox Build System :: General, task)

Description

[Calixte Denizet (:calixte)]

Just rename critical_section_on_gcov_flush.patch to rG7e18aeba5062.patch.

Comment 1

[Sylvestre Ledru [:Sylvestre]]

ouch, please keep the name explicit :)

Comment 2

[Calixte Denizet (:calixte)]

Attachment: Bug 1603436 - Rename clang patch (critical_section_on_gcov_flush.patch) to make easier the switch to LLVM 10

Cherry-picked from: https://github.com/llvm/llvm-project/commit/02ce9d8ef5a84bc884de4105eae5f8736ef67634.

Comment 3

[Nathan Froyd [:froydnj]]

(In reply to Sylvestre Ledru [:Sylvestre] from comment #1)

ouch, please keep the name explicit :)

dmajor has requested that we make the names correspond to upstream revisions somehow, so it's easier to to see when we need to ditch our custom patches. This was somewhat easier when the names corresponded to SVN revisions, though. =/

Comment 4

[Mike Hommey [:glandium]]

It's scriptable, though (as long as you have a local clone of llvm)

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1054b841f0fb
Rename clang patch (critical_section_on_gcov_flush.patch) to make easier the switch to LLVM 10 r=dmajor

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/1054b841f0fb