Skip to content

Conversation

@nilason
Copy link
Contributor

@nilason nilason commented Oct 31, 2025

Description

postgis:

postgis2:

  • update to last version 2.5.11
  • remove PostgreSQL <10 variants
  • remove proj7 variant, make the dependency of proj7 the default
Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 14.7.2 23H311 arm64
Xcode 16.2 16C5032a

Verification

Have you

  • followed our Commit Message Guidelines?
  • squashed and minimized your commits?
  • checked that there aren't other open pull requests for the same change?
  • referenced existing tickets on Trac with full URL in commit message?
  • checked your Portfile with port lint?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?
  • checked that the Portfile's most important variants haven't been broken?

@macportsbot
Copy link

Notifying maintainers:
@Veence for port postgis2.

@reneeotten
Copy link
Contributor

@nilason I would personally prefer to have the postgis port follow the latest upstream version and, if needed, in the future to split off something and add a version number to it. That's how it's done for many/most packages. Any reason why not to do that here?

@nilason
Copy link
Contributor Author

nilason commented Oct 31, 2025

@nilason I would personally prefer to have the postgis port follow the latest upstream version and, if needed, in the future to split off something and add a version number to it. That's how it's done for many/most packages. Any reason why not to do that here?

Very good point, I absolutely agree, I will replace the postgis with the currentpostgis3 (which I will deprecate for now in case of future need).

@nilason nilason force-pushed the postgis branch 4 times, most recently from 3529f5b to d3d927e Compare November 2, 2025 21:56
@nilason
Copy link
Contributor Author

nilason commented Nov 3, 2025

Update to mapnik 4.1.3 fails on Boost bug, addressed upstream with boostorg/gil@fcfce66. Dropping that update here as this PR is cascading too much.

@nilason
Copy link
Contributor Author

nilason commented Nov 3, 2025

This is now ready for review. I made the following changes:

  • replaced postgis with current postgis3
  • made postgis3 obsolete
  • updated postgis2 to latest (EOL) version, and made the port deprecated
  • updated (the dependant) osm2pgrouting to 3.0.0

@nilason
Copy link
Contributor Author

nilason commented Nov 3, 2025

The failing CI runner on macOS 26 is caused by missing dependency (mapnik) of a postgis dependent port (mod_tile). That should be fixed in a separate PR.

@nilason
Copy link
Contributor Author

nilason commented Nov 17, 2025

The failing CI runner on macOS 26 is caused by missing dependency (mapnik) of a postgis dependent port (mod_tile). That should be fixed in a separate PR.

Now the macOS 15 also fails for the same reason. Fixing that requires #30024 . There are no response from maintainers there, so who knows when that will be moving. mod_tile and mapnik are broken irrespective of this PR and shouldn’t stop this from getting merged, @reneeotten ?

@nilason
Copy link
Contributor Author

nilason commented Nov 18, 2025

Added update to mapnik, and accompanying changes to mod_tile.

@nilason
Copy link
Contributor Author

nilason commented Nov 18, 2025

Notifying maintainers:
@stromnov for port mapnik.
@frankdean for port mod_tile.

Copy link
Contributor

@frankdean frankdean left a comment

Choose a reason for hiding this comment

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

The mod_tile port includes a shell script at ${prefix}/share/mod_tile/osm_setup_db.sh which creates an initial database, importing OpenStreetMap map tile data, using osm2pgsql-lua, so that script will fail if osm2pgsql-lua has not been installed.

@nilason
Copy link
Contributor Author

nilason commented Nov 18, 2025

The mod_tile port includes a shell script at ${prefix}/share/mod_tile/osm_setup_db.sh which creates an initial database, importing OpenStreetMap map tile data, using osm2pgsql-lua, so that script will fail if osm2pgsql-lua has not been installed.

Lua is since be175bc part of the port osm2pgsql by default. That’s when the lua subport was removed.

@nilason
Copy link
Contributor Author

nilason commented Nov 18, 2025

The mod_tile port includes a shell script at ${prefix}/share/mod_tile/osm_setup_db.sh which creates an initial database, importing OpenStreetMap map tile data, using osm2pgsql-lua, so that script will fail if osm2pgsql-lua has not been installed.

Lua is since be175bc part of the port osm2pgsql by default. That’s when the lua subport was removed.

It is possible the script needs to be updated for the changes with version 2+, but I believe that is out of scope for this PR.

@frankdean
Copy link
Contributor

The mod_tile port includes a shell script at ${prefix}/share/mod_tile/osm_setup_db.sh which creates an initial database, importing OpenStreetMap map tile data, using osm2pgsql-lua, so that script will fail if osm2pgsql-lua has not been installed.

Lua is since be175bc part of the port osm2pgsql by default. That’s when the lua subport was removed.

It is possible the script needs to be updated for the changes with version 2+, but I believe that is out of scope for this PR.

Thanks. I'll check the script.

@reneeotten reneeotten merged commit b77d6f0 into macports:master Nov 18, 2025
3 checks passed
@reneeotten
Copy link
Contributor

thanks @nilason !

@nilason
Copy link
Contributor Author

nilason commented Nov 18, 2025

thanks @nilason !

Thank you!

@nilason nilason deleted the postgis branch November 18, 2025 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer: open Affects an openmaintainer port type: update

Development

Successfully merging this pull request may close these issues.

5 participants