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-Status: No, score=-2.5 required=3.0 tests=ALL_TRUSTED,BAYES_00, BODY_8BITS 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 AF9A61F9FC; Sun, 28 Mar 2021 22:58:36 +0000 (UTC) Date: Sun, 28 Mar 2021 22:58:36 +0000 From: Eric Wong To: =?utf-8?B?zqPPhM6xz43Pgc6/z4Igzp3PhM6tzr3PhM6/z4I=?= Cc: meta@public-inbox.org Subject: Re: [PATCH v1] git-send-email-reply: Append subject Message-ID: <20210328225836.GA17258@dcvr> References: <20210326213517.GA5730@dcvr> <20210327083946.30726-1-133706+stdedos@users.noreply.github.com> <20210327095431.GA32057@dcvr> <20210327193232.GA11329@dcvr> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: List-Id: Σταύρος Ντέντος wrote: > On Sat, 27 Mar 2021 at 21:32, Eric Wong wrote: > > It does, it clutters up the directory listing ("homepage"). > > Also redirect adds latency + traffic. > > > > If somebody is already looking at public-inbox.org, then "meta/" > > is already the top link in the listing. > > Hmm .... I "skipped" it as some error (why on earth "meta" is first on > a "name-ascending" sorting?!?!). It matches nginx behavior, as well as Apache w/ FoldersFirst option enabled. > > Fwiw, the test suite runs w/o install. > > I wanted to test my change "in reality" (i.e. its browser rendering), > rather than run a testing suite. > Honestly, I blacked out after being unable to see my result, I didn't > think to run tests. > (especially considering the not-so-big changes done). Ah, "perl -I lib script/public-inbox-httpd -l $HOST:$PORT" lets you run on ~/.public-inbox/config, or whatever PI_CONFIG= you want. When I work on WWW changes, I also use: plackup -I lib -o 127.0.0.1 -R lib -r examples/public-inbox.psgi Which does auto-reloading and ensures stuff works on generic PSGI servers. > > Agree. Note public-inbox.git has a new "symlink-install" target > > that allows a minimimum install footprint: > > [Also], it's in Debian sid, now: https://packages.debian.org/public-inbox > > True, but I'd be missing "a backend of existing threads". > But now that I think of it, I could've just cloned any small enough > repo (e.g. meta? :-p) Yes, meta is pretty small, still. There's many smaller v2 ones at lore: lore.kernel.org tools | workflows | dwarves ...