Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
2004293003:
Roll LibFuzzer d05583b..feca8e5. (Closed)
DescriptionRoll LibFuzzer d05583b..feca8e5.
https://chromium.googlesource.com/chromium/llvm-project/llvm/lib/Fuzzer/+log/d05583b..feca8e5
TBR=kcc@chromium.org
BUG=613888
Committed: https://crrev.com/7bdf0a05e83ffc054cc2a966ff108b05d82f8cfd
Cr-Commit-Position: refs/heads/master@{#395722}
Patch Set 1 #Patch Set 2 : Remove deleted file FuzzerInterface.cpp and add a dummy target for afl_fuzzer.cpp. #Patch Set 3 : Update tools/checklicenses/checklicenses.py for afl_driver.cpp. #Patch Set 4 : Update to the next revision. #Messages
Total messages: 25 (10 generated)
Description was changed from ========== Roll LibFuzzer d05583b..b8fa8b1. https://chromium.googlesource.com/chromium/llvm-project/llvm/lib/Fuzzer/+log/... R=kcc@chromium.org BUG=613888 ========== to ========== Roll LibFuzzer d05583b..b8fa8b1. https://chromium.googlesource.com/chromium/llvm-project/llvm/lib/Fuzzer/+log/... TBR=kcc@chromium.org BUG=613888 ==========
On 2016/05/24 10:33:05, mmoroz wrote: TBR
The CQ bit was checked by mmoroz@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2004293003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/2004293003/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by mmoroz@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2004293003/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/2004293003/20001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
On 2016/05/24 13:29:42, commit-bot: I haz the power wrote: > Try jobs failed on following builders: > linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED, > http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...) Ok, we don't need a dummy target for afl_driver there, but need to update '/tools/checklicenses/checklicenses.py' or add a license for afl_driver.
mmoroz@chromium.org changed reviewers: + thestig@chromium.org
On 2016/05/24 13:51:03, mmoroz wrote: > On 2016/05/24 13:29:42, commit-bot: I haz the power wrote: > > Try jobs failed on following builders: > > linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED, > > > http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...) > > Ok, we don't need a dummy target for afl_driver there, but need to update > '/tools/checklicenses/checklicenses.py' or add a license for afl_driver. +Lei for tools/checklicenses/checklicenses.py update.
On 2016/05/24 14:04:56, mmoroz wrote: > +Lei for tools/checklicenses/checklicenses.py update. Can someone add the proper license header upstream to afl_driver.cpp and roll forward to the newer revision?
Done in r270598. On Tue, May 24, 2016 at 10:58 AM, <thestig@chromium.org> wrote: > On 2016/05/24 14:04:56, mmoroz wrote: > > +Lei for tools/checklicenses/checklicenses.py update. > > Can someone add the proper license header upstream to afl_driver.cpp and > roll > forward to the newer revision? > > https://codereview.chromium.org/2004293003/ > -- You received this message because you are subscribed to the Google Groups "Chromium-reviews" group. To unsubscribe from this group and stop receiving emails from it, send an email to chromium-reviews+unsubscribe@chromium.org.
On 2016/05/24 19:11:55, chromium-reviews wrote: > Done in r270598. I guess you can roll to that commit now, and you won't need any checklicenses.py changes, so I'm no longer needed here.
Description was changed from ========== Roll LibFuzzer d05583b..b8fa8b1. https://chromium.googlesource.com/chromium/llvm-project/llvm/lib/Fuzzer/+log/... TBR=kcc@chromium.org BUG=613888 ========== to ========== Roll LibFuzzer d05583b..feca8e5. https://chromium.googlesource.com/chromium/llvm-project/llvm/lib/Fuzzer/+log/... TBR=kcc@chromium.org BUG=613888 ==========
On 2016/05/24 19:17:00, Lei Zhang wrote: > On 2016/05/24 19:11:55, chromium-reviews wrote: > > Done in r270598. > > I guess you can roll to that commit now, and you won't need any checklicenses.py > changes, so I'm no longer needed here. Thanks everyone! Now it should work :)
The CQ bit was checked by mmoroz@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2004293003/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/2004293003/60001
Message was sent while issue was closed.
Description was changed from ========== Roll LibFuzzer d05583b..feca8e5. https://chromium.googlesource.com/chromium/llvm-project/llvm/lib/Fuzzer/+log/... TBR=kcc@chromium.org BUG=613888 ========== to ========== Roll LibFuzzer d05583b..feca8e5. https://chromium.googlesource.com/chromium/llvm-project/llvm/lib/Fuzzer/+log/... TBR=kcc@chromium.org BUG=613888 ==========
Message was sent while issue was closed.
Committed patchset #4 (id:60001)
Message was sent while issue was closed.
Description was changed from ========== Roll LibFuzzer d05583b..feca8e5. https://chromium.googlesource.com/chromium/llvm-project/llvm/lib/Fuzzer/+log/... TBR=kcc@chromium.org BUG=613888 ========== to ========== Roll LibFuzzer d05583b..feca8e5. https://chromium.googlesource.com/chromium/llvm-project/llvm/lib/Fuzzer/+log/... TBR=kcc@chromium.org BUG=613888 Committed: https://crrev.com/7bdf0a05e83ffc054cc2a966ff108b05d82f8cfd Cr-Commit-Position: refs/heads/master@{#395722} ==========
Message was sent while issue was closed.
Patchset 4 (id:??) landed as https://crrev.com/7bdf0a05e83ffc054cc2a966ff108b05d82f8cfd Cr-Commit-Position: refs/heads/master@{#395722} |
