user/dev discussion of public-inbox itself
 help / color / mirror / code / Atom feed
Search results ordered by [date|relevance]  view[summary|nested|Atom feed]
thread overview below | download mbox.gz: |
* [PATCH 1/4] wwwtext: show multiple infourl values properly
  2020-02-01  9:12  6% [PATCH 0/4] flesh out multi-URL support Eric Wong
@ 2020-02-01  9:12  7% ` Eric Wong
  0 siblings, 0 replies; 2+ results
From: Eric Wong @ 2020-02-01  9:12 UTC (permalink / raw)
  To: meta

This is now an array, so ensure it's shown properly in the
sample config, instead of "ARRAY(0xI8BADBEEF)" or similar.

Fixes: 1988d730c0088e8b "config: support multi-value inbox.*.*url"
---
 lib/PublicInbox/WwwText.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/PublicInbox/WwwText.pm b/lib/PublicInbox/WwwText.pm
index 2e4aeec0..ace7f9d7 100644
--- a/lib/PublicInbox/WwwText.pm
+++ b/lib/PublicInbox/WwwText.pm
@@ -148,12 +148,12 @@ sub inbox_config ($$$) {
 	; instead of "inboxdir", both remain supported after 1.2
 	mainrepo = /path/to/top-level-inbox
 EOS
-	for my $k (qw(address listid)) {
+	for my $k (qw(address listid infourl)) {
 		defined(my $v = $ibx->{$k}) or next;
 		$$txt .= "\t$k = $_\n" for @$v;
 	}
 
-	for my $k (qw(filter infourl newsgroup obfuscate replyto watchheader)) {
+	for my $k (qw(filter newsgroup obfuscate replyto watchheader)) {
 		defined(my $v = $ibx->{$k}) or next;
 		$$txt .= "\t$k = $v\n";
 	}

^ permalink raw reply related	[relevance 7%]

* [PATCH 0/4] flesh out multi-URL support
@ 2020-02-01  9:12  6% Eric Wong
  2020-02-01  9:12  7% ` [PATCH 1/4] wwwtext: show multiple infourl values properly Eric Wong
  0 siblings, 1 reply; 2+ results
From: Eric Wong @ 2020-02-01  9:12 UTC (permalink / raw)
  To: meta

Better support for multiple URLs will hopefully discourage SPOFs
and give users more options when hosts go down.  It also gives
users and admins more agility when it comes to switching domain
names or having .onion mirrors.

Eric Wong (4):
  wwwtext: show multiple infourl values properly
  wwwtext: give "url" examples in sample config
  solver: join multiple URLs with "||"
  config: assume multiple cgit URLs, too

 lib/PublicInbox/Config.pm    |  8 ++++++--
 lib/PublicInbox/SolverGit.pm |  5 +++--
 lib/PublicInbox/WwwText.pm   | 24 +++++++++++++++---------
 3 files changed, 24 insertions(+), 13 deletions(-)

^ permalink raw reply	[relevance 6%]

Results 1-2 of 2 | reverse | options above
-- pct% links below jump to the message on this page, permalinks otherwise --
2020-02-01  9:12  6% [PATCH 0/4] flesh out multi-URL support Eric Wong
2020-02-01  9:12  7% ` [PATCH 1/4] wwwtext: show multiple infourl values properly 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).