Skip to content

Commit

Permalink
Upgrade PostgreSQL to version 15
Browse files Browse the repository at this point in the history
  • Loading branch information
@fbonzon
fbonzon committed Feb 6, 2023
1 parent 8f1f33d commit 05c672d
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions Formula/postgis.rb
View file
Original file line number Diff line number Diff line change
Expand Up @@ -35,15 +35,15 @@ class Postgis < Formula
depends_on "geos"
depends_on "json-c" # for GeoJSON and raster handling
depends_on "pcre2"
depends_on "postgresql@14"
depends_on "postgresql@15"
depends_on "proj"
depends_on "protobuf-c" # for MVT (map vector tiles) support
depends_on "sfcgal" # for advanced 2D/3D functions

fails_with gcc: "5"

def postgresql
Formula["postgresql@14"]
Formula["postgresql@15"]
end

def install
Expand All @@ -65,7 +65,7 @@ def install

system "./autogen.sh" if build.head?
# Fixes config/install-sh: No such file or directory
# This is caused by a misalignment between ./configure in postgres@14 and postgis
# This is caused by a misalignment between ./configure in postgres@15 and postgis
mv "build-aux", "config"
inreplace %w[configure utils/Makefile.in] do |s|
s.gsub! "build-aux", "config"
Expand All @@ -87,7 +87,7 @@ def install
bin.install (buildpath/postgresql_stage_path/"bin").children
doc.install (buildpath/postgresql_stage_path/"share/doc").children

stage_path = File.join("stage", HOMEBREW_PREFIX)
stage_path = File.join("stage", postgresql.prefix)
lib.install (buildpath/stage_path/"lib").children
share.install (buildpath/stage_path/"share").children

Expand Down

2 comments on commit 05c672d

@svoop
Copy link

@svoop svoop commented on 05c672d Feb 26, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fbonzon Hi! Would be nice to have this update in the official core as well, how about creating a PR against Homebrew/homebrew-core? Thanks!!

@fbonzon
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fbonzon Hi! Would be nice to have this update in the official core as well, how about creating a PR against Homebrew/homebrew-core? Thanks!!

@svoop As I wrote in the Homebrew issue, the above changes fix only half of the problem. A PR with only them would probably not be accepted, because it doesn't fully address the issue. I have currently no bandwidth to investigate further.

PostGIS now builds successfully, but is not linked correctly within PostgreSQL. I gave workaround command lines to fix the links. If you want to create a PR with the above changes, you have my permission to copy.

Please sign in to comment.