Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1r02Q7-002pta-Bw for pgsql-pkg-debian@arkaria.postgresql.org; Mon, 06 Nov 2023 16:25:47 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.94.2) (envelope-from ) id 1r02Q6-00HEZe-25 for pgsql-pkg-debian@arkaria.postgresql.org; Mon, 06 Nov 2023 16:25:46 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1r02Q5-00HEZW-SA for pgsql-pkg-debian@lists.postgresql.org; Mon, 06 Nov 2023 16:25:45 +0000 Received: from feynman.df7cb.de ([195.49.152.168]) by magus.postgresql.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1r02Q3-0051ms-GU for pgsql-pkg-debian@lists.postgresql.org; Mon, 06 Nov 2023 16:25:45 +0000 Received: from msg.df7cb.de (unknown [IPv6:2a02:908:1482:3660:b7d0:f584:a987:bc23]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature ECDSA (P-384) server-digest SHA384) (Client did not present a certificate) by feynman.df7cb.de (Postfix) with ESMTPSA id 4SPGtG3qMDz3Ft8; Mon, 6 Nov 2023 17:25:42 +0100 (CET) Date: Mon, 6 Nov 2023 17:25:42 +0100 From: Christoph Berg To: Bradford Boyle Cc: pgsql-pkg-debian@lists.postgresql.org Subject: Re: Updated packaging for MobilityDB 1.1.0~beta1 Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Archived-At: Precedence: bulk Re: To Bradford Boyle > In postgresql-XX-mobility, we need the XX version of > postgresql-XX-postgis-3, not just any version, so this change isn't > correct. > > The usage of "postgresql-postgis" in the Build-Depends is also fishy, > but the "update debian/control from control.in" machinery doesn't > support using PGVERSION in the Source section yet. I tried > implementing that the other week, but didn't get to finish it yet, > hopefully that will happen over the next days. Hi Bradford, I've done that now, it will be available in postgresql-common 256. I pushed a change that uses it and also loops over the versions that should be targeted for cmake. Ideally, testing the extension could also use this from debian/tests/installcheck: pg_buildext run_installed /usr/bin/cmake -Btest-%v pg_buildext run_installed make -C test-% test ... but that starts by recompiling the package instead of just running the tests. Perhaps you have an idea how to make it work? The existing debian/tests/regress.sh file looks very complex and might break with every little upstream change. (Of course if there's no easier way to get the upstream testsuite to run, we have to go the complex route.) Christoph