Return-Path: X-Original-To: notmuch@notmuchmail.org Delivered-To: notmuch@notmuchmail.org Received: from localhost (localhost [127.0.0.1]) by olra.theworths.org (Postfix) with ESMTP id 59FBE431FC0; Wed, 25 Nov 2009 19:24:14 -0800 (PST) X-Virus-Scanned: Debian amavisd-new at olra.theworths.org 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 FDDfVEUmG32E; Wed, 25 Nov 2009 19:24:13 -0800 (PST) Received: from cworth.org (localhost [127.0.0.1]) by olra.theworths.org (Postfix) with ESMTP id 8B2F7431FAE; Wed, 25 Nov 2009 19:24:13 -0800 (PST) From: Carl Worth To: Jameson Graef Rollins , notmuch@notmuchmail.org In-Reply-To: <1258927116-528-1-git-send-email-jrollins@finestructure.net> References: <1258927116-528-1-git-send-email-jrollins@finestructure.net> Date: Wed, 25 Nov 2009 19:23:59 -0800 Message-ID: <87iqcyrmsw.fsf@yoom.home.cworth.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Subject: Re: [notmuch] [PATCH 1/3] remove Makefile.config from source to allow for custom config. X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.12 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: Thu, 26 Nov 2009 03:24:14 -0000 On Sun, 22 Nov 2009 16:58:34 -0500, Jameson Graef Rollins wrote: > Modified ./configure script to generate a Makefile.config if it > doesn't exist. This will allow people to maintain custom > Makefile.config files without causing conflicts with the repository. It's definitely a good idea to get the generated Makefile.config out of the repository so that users can maintain their own versions without conflicts. In order to do this, we'll need to fix the Makefile to have a rule to generate Makefile.config if it doesn't exist by running configure, (and it could also give a message letting the user know that it's running configure without arguments, and the user can run it explicitly with arguments if desired[*]). -Carl [*] Of course, that's not *really* necessary until our configure script actually *does* accept arguments, but we really should add at least --prefix right away.