Return-Path: X-Original-To: notmuch@notmuchmail.org Delivered-To: notmuch@notmuchmail.org Received: from localhost (localhost [127.0.0.1]) by arlo.cworth.org (Postfix) with ESMTP id 4128E6DE00D3 for ; Tue, 7 Jun 2016 07:28:43 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at cworth.org X-Spam-Flag: NO X-Spam-Score: 0.574 X-Spam-Level: X-Spam-Status: No, score=0.574 tagged_above=-999 required=5 tests=[AWL=-0.078, SPF_NEUTRAL=0.652] autolearn=disabled Received: from arlo.cworth.org ([127.0.0.1]) by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id zTlX8OSKqNhv for ; Tue, 7 Jun 2016 07:28:35 -0700 (PDT) Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34]) by arlo.cworth.org (Postfix) with ESMTP id C59FE6DE00BF for ; Tue, 7 Jun 2016 07:28:33 -0700 (PDT) Received: from guru.guru-group.fi (localhost [IPv6:::1]) by guru.guru-group.fi (Postfix) with ESMTP id 47684100104; Tue, 7 Jun 2016 17:28:14 +0300 (EEST) From: Tomi Ollila To: David Bremner , notmuch@notmuchmail.org Subject: Re: [PATCH] test: Tnnn-* script file name renames In-Reply-To: <87k2i19pz7.fsf@zancas.localnet> References: <1465143703-18260-1-git-send-email-tomi.ollila@iki.fi> <87k2i19pz7.fsf@zancas.localnet> User-Agent: Notmuch/0.22+32~gd4854c5 (http://notmuchmail.org) Emacs/24.5.1 (x86_64-unknown-linux-gnu) X-Face: HhBM'cA~ MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.20 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: Tue, 07 Jun 2016 14:28:43 -0000 On Tue, Jun 07 2016, David Bremner wrote: > Tomi Ollila writes: > >> Over time, new test scripts of any particular "group" are added >> in-between of current test scripts by giving them number that is not >> divisible by 10. Eventually the space between groups fill up (i.e. >> all 10 "slots" are used), or it is just hard to recognize suitable >> number for new test. >> >> Instead of trying hard planning work for grouping these and adding >> more space for the groups (manually by hand) or trying to widen the >> space for all, this (automatic) renumbering of the file name takes >> care of the "problem" for a while. >> --- >> >> If this patch is applied, this will make all patches in queue that >> have test script changes fail to merge :( -- so this is partially >> wip, but at least commit message is written... > > I just rebased the retry-lock and message-properties series (separately) > on top of your commit without conflicts, so it's probably not that big > of a deal. > A-ha. it is nice that this works (it somehow notices renames... or it used the source blob hashes to notice what to change) > Do you think it's worth documenting what the supposed groups > of tests are (maybe in tests/README)? The groups could really use some > cleaning up. For example, we have at least two set groups of emacs > tests. Some things will be harder to classify, of course. That's why we [1] > believe in tags not folders ;). Nonetheless, perhaps it will be useful > to have things a bit less chaotic. Eventually we might even want to > support the idea of running a "group" of tests. As a quick thought is is just Simple Matter Of Documenting... I.e. how to categorize tests to groups (what belong where when there are many choices where files could go). But if someone(tm) can make such easily understantable groups (and devices reasonable numbering -- i can adjust my latest script (which does git-am'able patch files directly) to support renumbering (if ever needed) (*) in this case too) it might be worth sending for review... Tomi (*) for reference: everything we tried there was still need to renumber C64 basic programs ;) > > d > > [1]: except those who believe in searchs, not tags. > _______________________________________________ > notmuch mailing list > notmuch@notmuchmail.org > https://notmuchmail.org/mailman/listinfo/notmuch