Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cgDrU-0007Ij-RL for pgsql-hackers@arkaria.postgresql.org; Tue, 21 Feb 2017 17:04:24 +0000 Received: from localhost ([127.0.0.1] helo=postgresql.org) by malur.postgresql.org with smtp (Exim 4.84_2) (envelope-from ) id 1cgDrU-0008IO-0H for pgsql-hackers@arkaria.postgresql.org; Tue, 21 Feb 2017 17:04:24 +0000 Received: from makus.postgresql.org ([2001:4800:1501:1::229]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1cgDq3-0006gy-MO for pgsql-hackers@postgresql.org; Tue, 21 Feb 2017 17:02:55 +0000 Received: from out4-smtp.messagingengine.com ([66.111.4.28]) by makus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1cgDq1-0006jO-Kb for pgsql-hackers@postgresql.org; Tue, 21 Feb 2017 17:02:54 +0000 Received: from compute7.internal (compute7.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id E022F209A5; Tue, 21 Feb 2017 12:02:52 -0500 (EST) Received: from frontend1 ([10.202.2.160]) by compute7.internal (MEProxy); Tue, 21 Feb 2017 12:02:52 -0500 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-sender:x-me-sender:x-sasl-enc:x-sasl-enc; s= smtpout; bh=zPCMYnlonM0168dW7PmeAIhTqu0=; b=MCexnKmjU6sP3XPB/Q7d 4s+rsJcAkBH8EDwZdiuw1UvPl9mJKuZBFualad6FF7SsiLgdDZYxS4X9W5hWQiCL IL33WeeuK2s6P0UgSL4rS43rYKbc+3bnfM79LOrkYwNKTSrRNLQcfXo+ERbXLKqz l4Vz/hCgA45uTdsNYkIlGxc= X-ME-Sender: X-Sasl-enc: CNFvK9QZm/WpS+TJGu2l3yDgnpmhPXBJCFUZ67OfGxYB 1487696572 Received: from april.local (unknown [8.41.192.198]) by mail.messagingengine.com (Postfix) with ESMTPA id 99A807E2C4; Tue, 21 Feb 2017 12:02:52 -0500 (EST) Subject: Re: drop support for Python 2.3 To: Tom Lane References: <7d5073f6-d9ea-ab09-4805-69a12f23e52f@2ndquadrant.com> <9516.1486529343@sss.pgh.pa.us> <1486568115.5462.50.camel@gunduz.org> <0e2fd256-0a3a-bae8-4ef2-c536139c64cb@2ndquadrant.com> <1487493740.2748.13.camel@gunduz.org> <797.1487517822@sss.pgh.pa.us> <1453.1487518921@sss.pgh.pa.us> <1487527748.2748.37.camel@gunduz.org> <24334.1487530091@sss.pgh.pa.us> <1487535029.3796.1.camel@gunduz.org> <633.1487539228@sss.pgh.pa.us> <1487565212.4910.3.camel@gunduz.org> <664e0d02-20ed-5ae8-7d60-8f779d759072@2ndquadrant.com> <7165.1487690305@sss.pgh.pa.us> Cc: =?UTF-8?B?RGV2cmltIEfDvG5kw7x6?= , pgsql-hackers From: Peter Eisentraut Organization: 2ndQuadrant Message-ID: Date: Tue, 21 Feb 2017 12:02:52 -0500 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <7165.1487690305@sss.pgh.pa.us> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Pg-Spam-Score: -2.6 (--) List-Archive: List-Help: List-ID: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: X-Mailing-List: pgsql-hackers Precedence: bulk Sender: pgsql-hackers-owner@postgresql.org On 2/21/17 10:18, Tom Lane wrote: > Based on some not-fun I had along the way to that, I think it would be > a good idea to do the Python version check a bit earlier than you have > here. The shlib search in PGAC_CHECK_PYTHON_EMBED_SETUP is pretty fragile > and version-dependent, which means that a person could waste a lot of time > trying to get past the "could not find shared library for Python" error > before being told that their Python is too old. I propose to move the > minimum-version check into _PGAC_CHECK_PYTHON_DIRS, say right after the > [Python configuration directory] stanza. Unless there was some specific > reason for not wanting it to happen in python.m4? It's fine to move it. -- Peter Eisentraut http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers