1) includes/database.pgsql.inc uses SHOW SERVER_VERSION to determine which postgresql version we're running, but that statement was only introduced in 7.4.
Fix: none that I know.

2) modules/system/system.install tries to define three domains with range checks (the unsigned integer types); range checks on domains were introduced in 7.4.
Fix: none that I know.

3) modules/system/system.install contains an 'ALTER SEQUENCE ... MINVALUE 2 RESTART 2' statement, which was introduced in 7.4.
Fix: use SELECT setval('...', 2). In 7.3 there is no way to set a sequence's min_value after its creation, but I don't think there are any reasons to either...

4) modules/system/system.install defines some functional indexes (indexes on functions of columns). Functional indexes support is very crude in 7.3 and does not allow functions taking other arguments than the column name, or mixing functions and simple columns as keys. 7.4 should be ok in this regard.
Fix: none that I know of.

Since I am just doing a test install of drupal none of this issues is a real showstopper to me; however I think that the minimum PostgreSQL version in the requirements page should be upgraded to 7.4.

CommentFileSizeAuthor
#6 pgsql_7_4_required.d5.patch681 bytesdww
#1 pgsql_7_4_required.patch.txt673 bytesdww
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dww’s picture

Assigned: Unassigned » dww
Status: Active » Needs review
FileSize
673 bytes

yeah, seems far better to just change the minimum required than to try to fix all of those problems.

Dries’s picture

I'm OK with the proposed patch.

Dries’s picture

Status: Needs review » Fixed

Committed to CVS HEAD and DRUPAL-5. Thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)
vlayne’s picture

Status: Closed (fixed) » Active

Just downloaded 5.7 and tried to run against PostgreSQL 7.3, and, lo and behold, this patch was apparently *not* applied. Ran into exactly this bug. Also, the system min. requirements still say PostgreSQL 7.3.

dww’s picture

Version: 5.0 » 5.x-dev
Status: Active » Reviewed & tested by the community
FileSize
681 bytes

Indeed. It seems this was never actually committed to DRUPAL-5. It is fixed in DRUPAL-6 and HEAD, however. Re-rolled for DRUPAL-5.

Pancho’s picture

Yeah, I think we should commit this to DRUPAL-5.

drumm’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 5.x.

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.