From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by olra.theworths.org (Postfix) with ESMTP id CFA3B429E21 for ; Wed, 26 Oct 2011 11:29:45 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at olra.theworths.org X-Spam-Flag: NO X-Spam-Score: -0.7 X-Spam-Level: X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5 tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled Received: from olra.theworths.org ([127.0.0.1]) by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id fJNAJW2M6a4o for ; Wed, 26 Oct 2011 11:29:44 -0700 (PDT) Received: from mail-qy0-f181.google.com (mail-qy0-f181.google.com [209.85.216.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by olra.theworths.org (Postfix) with ESMTPS id 646F4431FB6 for ; Wed, 26 Oct 2011 11:29:44 -0700 (PDT) Received: by qyk27 with SMTP id 27so2393986qyk.5 for ; Wed, 26 Oct 2011 11:29:42 -0700 (PDT) Received: by 10.229.19.200 with SMTP id c8mr98117qcb.71.1319653782285; Wed, 26 Oct 2011 11:29:42 -0700 (PDT) Received: from localhost (nikula.org. [92.243.24.172]) by mx.google.com with ESMTPS id gx9sm3307733qab.12.2011.10.26.11.29.39 (version=SSLv3 cipher=OTHER); Wed, 26 Oct 2011 11:29:40 -0700 (PDT) From: Jani Nikula To: Daniel Schoepe , notmuch@notmuchmail.org Subject: Re: Patch review/application process In-Reply-To: <878vo8kdl2.fsf@gilead.invalid> References: <878vo8kdl2.fsf@gilead.invalid> User-Agent: Notmuch/0.5-232-g917e874 (http://notmuchmail.org) Emacs/23.1.1 (i686-pc-linux-gnu) Date: Wed, 26 Oct 2011 18:29:37 +0000 Message-ID: <87mxcnbo8e.fsf@nikula.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: "Use and development of the notmuch mail system." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 26 Oct 2011 18:29:45 -0000 On Tue, 25 Oct 2011 22:42:33 +0200, Daniel Schoepe wrote: > as many of you have probably noticed, the time after which patches are > reviewed and/or applied is considerably higher lately than it was, for > example, earlier this year. My subjective impression is that there is > also a recent increase in contributions and general activity for/about > notmuch. Since long waiting times between sending a patch and receiving > a response will probably deter some (potential) contributors from > working / continuing to work on notmuch, I find this to be an important > issue. There is also a number of patches that have been reviewed by > long-term contributors, but are then seemingly forgotten (I can find > some concrete examples of this, if this claim is in doubt). The good thing is, there are contributions and review. The bad thing is, unless you've hung around long enough, you don't know if the reviewers are people whose comments you should really pay attention to or not, and either way, fixing the patches seems pointless and frustrating if they don't get applied anyway. A MAINTAINERS file might be helpful in identifying some of the key people. AUTHORS could be updated to include people with not insignificant contributions. > - Further delegate responsibility for the various parts, specifically > the emacs UI, which has a large number of outstanding patches. I'd be > in favor (if Carl is okay with it, of course) of giving one or more > people (Jameson and Austin came up as possible candidates when > discussing this on IRC, if they are willing) the authority to apply > patches for the emacs UI, similar to how patches for bindings are > handled. Agreed. I sincerely hope Carl and the candidates are willing. And if not, a favorable review from the long-term contributors (see AUTHORS above) should carry more weight in getting the patches applied in a timely manner. > - (Re)try some patch/issue management software: Since patches are easily > forgotten if they just float around in several months old mails, it > might be prudent to use something to keep track of patches or issues > these patches address. I know that the patchwork instance didn't work > out so well, partly because it didn't recognize new versions of sent > patches. An alternative might be an issue-based system, which would be > comfortably usable if it supported discussing issues via mail instead > of having to use some web interface. I think this is supported by > redmine. If the problem is lack of time, I'm not sure if setting up and maintaining some world facing web service would help things. > A mechanism to share notmuch tags between users could probably also be > adapted for this purpose, but this would make it harder for > non-notmuch users to discuss issues / see existing with the same > comfort. (Package maintainers or people who want to check what > outstanding flaws exist before migrating to notmuch come to mind). Hmm, if there was a way to reference messages in Mailman/pipermail archive using message IDs, I'm sure it would be trivial to export the tags to simple html with links to the mails in the mailing list archive. > - Some kind of "voting system" that gets a patch applied if some > number of "trusted" contributors reviewed a patch and think it is > good. I haven't given this idea much thought and I guess it might > lead to a "lack of direction / guiding principles" in the development > of notmuch. I wouldn't put too much emphasis on creating a voting system or a process. I do have hopes for the tag sharing mechanism helping in tracking the reviewed patches, though. That means figuring out whose tags to trust anyway. > I'm probably overlooking some downsides of those ideas, so I'd like to > hear any responses and/or other approaches to deal with this (Of course, > I'm also open to arguments showing that I'm making too big a deal out of > this :)). Thanks for bringing this up. BR, Jani.