Closed Bug 1900823 Opened 1 year ago Closed 1 year ago

Linkify resource:// and chrome:// URLs in textual occurrences

Categories

(Webtools :: Searchfox, enhancement, P3)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: arai, Assigned: arai)

References

Details

Attachments

(1 file)

chrome:// and resources:// can also appear inside comments etc.
They don't have to be indexed as "use", but still having a link would be nice.
This could be done as part of output step, linkify_comments or somewhere around that.

(In reply to Tooru Fujisawa [:arai] from comment #0)

They don't have to be indexed as "use", but still having a link would be nice.
This could be done as part of output step, linkify_comments or somewhere around that.

The output.sh stage is only like 8 minutes, we could always run it once with linkify_comments having side effects where it emits analysis records before crossref (and perhaps with the output going to /dev/null and maybe blame turned off), then run it again like normal after crossref. It would complicate things slightly during that second stage since the source records would be inside the normally atomic-ish token that likely would start before the source records, but it seems tractable.

Attached file GitHub Pull Request
Assignee: nobody → arai.unmht

(In reply to Andrew Sutherland [:asuth] (he/him) from comment #1)

The output.sh stage is only like 8 minutes, we could always run it once with linkify_comments having side effects where it emits analysis records before crossref (and perhaps with the output going to /dev/null and maybe blame turned off), then run it again like normal after crossref. It would complicate things slightly during that second stage since the source records would be inside the normally atomic-ish token that likely would start before the source records, but it seems tractable.

Yeah, that will also work, but for now I'll go with single-pass way for simplicity.

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
See Also: → 1949604
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: