Closed Bug 1603436 Opened 8 months ago Closed 8 months ago

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

Categories

(Firefox Build System :: General, task)

task
Not set
normal

Tracking

(firefox73 fixed)

RESOLVED FIXED
mozilla73
Tracking Status
firefox73 --- fixed

People

(Reporter: calixte, Assigned: calixte)

Details

Attachments

(1 file)

Just rename critical_section_on_gcov_flush.patch to rG7e18aeba5062.patch.

ouch, please keep the name explicit :)

(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. =/

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

Pushed by cdenizet@mozilla.com:
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
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla73
You need to log in before you can comment on or make changes to this bug.