[sancov] fix coverage-report-server cannot display coverage detail
ClosedPublic
Actions

Authored by HerrCai0907 on Mon, Mar 13, 11:34 PM.

Details

Reviewers

vitalybuka
klimek
lattner

Commits

rGfb066c4622c8: [sancov] fix coverage-report-server cannot display coverage detail

Summary

This patch make following change for coverage-report-server.py

using uri ./{name} from root in the old version python http.server can be handled as //{name}. But due to https://github.com/python/cpython/pull/93879, it will be handled as /{name} now.

So I want to use a prefix to avoid double slashes issue.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

HerrCai0907 created this revision.Mon, Mar 13, 11:34 PM

Herald added a project: Restricted Project. · View Herald TranscriptMon, Mar 13, 11:34 PM

HerrCai0907 requested review of this revision.Mon, Mar 13, 11:34 PM

Herald added a project: Restricted Project. · View Herald TranscriptMon, Mar 13, 11:34 PM

Herald added a subscriber: llvm-commits. · View Herald Transcript

Harbormaster completed remote builds in B219255: Diff 504951.Tue, Mar 14, 12:20 AM

I guess ut failed is not related with this PR?

HerrCai0907 added a reviewer: klimek.Tue, Mar 14, 4:26 PM

rebase to main

Harbormaster completed remote builds in B219538: Diff 505355.Tue, Mar 14, 9:47 PM

HerrCai0907 added a reviewer: lattner.Thu, Mar 16, 5:50 AM

this motivation makes sense to me, but I'm not a competent reviewer for this area

This revision was not accepted when it landed; it landed in state Needs Review.Sun, Mar 19, 6:10 AM

Closed by commit rGfb066c4622c8: [sancov] fix coverage-report-server cannot display coverage detail (authored by HerrCai0907). · Explain Why

This revision was automatically updated to reflect the committed changes.

HerrCai0907 added a commit: rGfb066c4622c8: [sancov] fix coverage-report-server cannot display coverage detail.

Revision Contents
Changeset List

Path

Size

llvm/

tools/

sancov/

coverage-report-server.py

7 lines

Diff 506383

View Options

llvm/tools/sancov/coverage-report-server.py

[sancov] fix coverage-report-server cannot display coverage detailClosedPublicActions