* [PATCH 0/5] cgit + WwwHighlight.pm integration
@ 2019-04-24 23:02 7% Eric Wong
2019-04-24 23:02 7% ` [PATCH 1/5] wwwhighlight: do linkification, too Eric Wong
0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2019-04-24 23:02 UTC (permalink / raw)
To: meta
cgit-wwwhighlight-filter.lua can benefit cgit installations by
allowing them to avoid spawning a new process for about-filter
and source-filter in cgitrc.
Instead of spawning highlight(1) or similar every time a cgit
filter is invoked, we can have a long-lived PSGI server running
WwwHighlight.pm(*) and rely on lua-http to access the server
over loopback.
WwwHighlight.pm is also updated to support the same
linkification logic we use for emails.
(*) see examples/highlight.psgi
Eric Wong (5):
wwwhighlight: do linkification, too
examples/cgit-commit-filter: remove unused variable
wwwhighlight: deal with no-op highlight
examples: cgit filter for use with WwwHighlight
examples/cgit-commit-filter.lua: some doc updates
MANIFEST | 1 +
examples/cgit-commit-filter.lua | 10 ++++-
examples/cgit-wwwhighlight-filter.lua | 57 +++++++++++++++++++++++++++
lib/PublicInbox/WwwHighlight.pm | 11 +++++-
4 files changed, 77 insertions(+), 2 deletions(-)
create mode 100644 examples/cgit-wwwhighlight-filter.lua
^ permalink raw reply [relevance 7%]
* [PATCH 1/5] wwwhighlight: do linkification, too
2019-04-24 23:02 7% [PATCH 0/5] cgit + WwwHighlight.pm integration Eric Wong
@ 2019-04-24 23:02 7% ` Eric Wong
0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2019-04-24 23:02 UTC (permalink / raw)
To: meta
Otherwise, there's no reason to use this API over highlight(1).
Maybe this can be an option in the future; but I'm struggling to
find a reason to not do it by default.
---
lib/PublicInbox/WwwHighlight.pm | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/lib/PublicInbox/WwwHighlight.pm b/lib/PublicInbox/WwwHighlight.pm
index 0191640..d8101b7 100644
--- a/lib/PublicInbox/WwwHighlight.pm
+++ b/lib/PublicInbox/WwwHighlight.pm
@@ -24,6 +24,7 @@ use warnings;
use bytes (); # only for bytes::length
use HTTP::Status qw(status_message);
use parent qw(PublicInbox::HlMod);
+use PublicInbox::Linkify qw();
# TODO: support highlight(1) for distros which don't package the
# SWIG extension. Also, there may be admins who don't want to
@@ -64,7 +65,10 @@ sub call {
return r(405) if $req_method ne 'PUT';
my $bref = read_in_full($env) or return r(500);
+ my $l = PublicInbox::Linkify->new;
+ $l->linkify_1($$bref);
$bref = $self->do_hl($bref, $env->{PATH_INFO});
+ $l->linkify_2($$bref);
my $h = [ 'Content-Type', 'text/html; charset=UTF-8' ];
push @$h, 'Content-Length', bytes::length($$bref);
--
EW
^ permalink raw reply related [relevance 7%]
Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2019-04-24 23:02 7% [PATCH 0/5] cgit + WwwHighlight.pm integration Eric Wong
2019-04-24 23:02 7% ` [PATCH 1/5] wwwhighlight: do linkification, too Eric Wong
Code repositories for project(s) associated with this public inbox
https://80x24.org/public-inbox.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).