From 7036a617b2f6183c11051ab7039347058bcc6cef Mon Sep 17 00:00:00 2001 From: Christoph Berg Date: Mon, 10 Jan 2022 13:15:13 +0100 Subject: [PATCH 1/2] Remove remaining PostGIS traces PostGIS support was removed in 01126bf, but README.md and the RPM spec file were not updated. --- README.md | 8 +------- rpms/postgres-decoderbufs.spec | 3 --- 2 files changed, 1 insertion(+), 10 deletions(-) diff --git a/README.md b/README.md index 115d1e3..218aa2d 100644 --- a/README.md +++ b/README.md @@ -18,21 +18,17 @@ This code depends on the following libraries and requires them for compilation: * [PostgreSQL](http://www.postgresql.org) 9.6+ * [Protobuf-c](https://github.com/protobuf-c/protobuf-c) 1.2+ - used for data serialization -* [PostGIS](http://www.postgis.net/) 2.1+ - used for Postgres geometric types support ## Building `postgres-decoderbufs` has to be built from source after installing required dependencies. The required dependencies are first PostgreSQL -(for pg_config), PostgreSQL server development packages, protobuf-c for the Protocol Buffer support and some PostGIS development packages. +(for pg_config), PostgreSQL server development packages, and protobuf-c for the Protocol Buffer support. ### Installing Dependencies #### Debian # Core build utilities apt-get update && apt-get install -f -y software-properties-common build-essential pkg-config git postgresql-server-dev-9.6 - - # PostGIS dependency - apt-get install -f -y libproj-dev liblwgeom-dev # Protobuf-c dependency (requires a non-stable Debian repo) add-apt-repository "deb http://ftp.debian.org/debian testing main contrib" && apt-get update @@ -139,8 +135,6 @@ The following table shows how current PostgreSQL type OIDs are mapped to which d | TIMESTAMPTZOID | datum_string | | BYTEAOID | datum_bytes | | POINTOID | datum_point | -| PostGIS geometry | datum_point | -| PostGIS geography | datum_point | ## Support diff --git a/rpms/postgres-decoderbufs.spec b/rpms/postgres-decoderbufs.spec index 98bf647..3a6f138 100644 --- a/rpms/postgres-decoderbufs.spec +++ b/rpms/postgres-decoderbufs.spec @@ -12,14 +12,11 @@ Source0: https://github.com/debezium/%{name}/archive/v%{full_version}.tar.gz BuildRequires: gcc BuildRequires: postgresql-devel >= 9.6, postgresql-server-devel >= 9.6 -BuildRequires: postgis-devel >= 2 BuildRequires: protobuf-c-devel Requires: protobuf-c %{?postgresql_module_requires} -Recommends: postgis - %description A PostgreSQL logical decoder output plugin to deliver data as Protocol Buffers messages. From b98c98a70a5ceb24c65d3235bf5e2cc5bf9354c7 Mon Sep 17 00:00:00 2001 From: Christoph Berg Date: Mon, 10 Jan 2022 13:17:20 +0100 Subject: [PATCH 2/2] Modernize Debian install instructions libprotobuf-c-dev 1.2.1 is part of Debian stretch, which is oldoldstable now, so remove the special instructions now. While at it, replace postgresql-server-dev-9.6 with version-agnostic postgresql-server-dev-all. --- README.md | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/README.md b/README.md index 218aa2d..7ba89c0 100644 --- a/README.md +++ b/README.md @@ -27,12 +27,9 @@ This code depends on the following libraries and requires them for compilation: ### Installing Dependencies #### Debian - # Core build utilities - apt-get update && apt-get install -f -y software-properties-common build-essential pkg-config git postgresql-server-dev-9.6 - - # Protobuf-c dependency (requires a non-stable Debian repo) - add-apt-repository "deb http://ftp.debian.org/debian testing main contrib" && apt-get update - apt-get install -y libprotobuf-c-dev=1.2.1-1+b1 + # Core build utilities + apt-get update + apt-get install -f -y build-essential pkg-config git libprotobuf-c-dev postgresql-server-dev-all When updating the ProtoBuf definition, also install the ProtoBuf C compiler: @@ -43,7 +40,6 @@ The above are taken from the Debezium [container images](https://github.com/debe #### Other Linux distributions You just need to make sure the above software packages (_or some flavour thereof_) are installed for your distro. -Note that the last step from the above sequence is only required for Debian to be able to install `libprotobuf-c-dev:1.2.1` ### Getting the source code