From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: X-Spam-Status: No, score=-4.0 required=3.0 tests=ALL_TRUSTED,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 844A61F8C6 for ; Sat, 11 Sep 2021 00:19:17 +0000 (UTC) From: Eric Wong To: meta@public-inbox.org Subject: [PATCH 0/3] lei saved-search fixes Date: Sat, 11 Sep 2021 00:19:14 +0000 Message-Id: <20210911001917.1310-1-e@80x24.org> In-Reply-To: <20210910141157.6u5adehpx7wftkor@meerkat.local> References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit List-Id: Konstantin Ryabitsev wrote: > I think it's reasonable to normalize \s+ into a single space for all queries, > right? Yes, 3/3 does it for HTTP(S). Don't think it's needed for local queries. I'm leaving non-ASCII whitespace alone for now since I don't think it'd be a real problem. > Another observation is that when I missed a \ on one of the lines, I managed > to make lei unusable. :) That's a far more important problem fixed in 1/3 :> Eric Wong (3): lei: fix handling of broken lei.saved-search config files lei: pass client stderr to git-config in more places lei: normalize whitespace in remote queries lib/PublicInbox/Config.pm | 10 ++-- lib/PublicInbox/LEI.pm | 23 +++++++-- lib/PublicInbox/LeiEditSearch.pm | 79 +++++++++++++++++++++++++++++-- lib/PublicInbox/LeiMirror.pm | 4 +- lib/PublicInbox/LeiSavedSearch.pm | 77 ++++++------------------------ lib/PublicInbox/LeiXSearch.pm | 4 +- script/lei | 2 + t/lei-q-save.t | 17 +++++++ 8 files changed, 136 insertions(+), 80 deletions(-)