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.
Comment | File | Size | Author |
---|---|---|---|
#6 | pgsql_7_4_required.d5.patch | 681 bytes | dww |
#1 | pgsql_7_4_required.patch.txt | 673 bytes | dww |
Comments
Comment #1
dwwyeah, seems far better to just change the minimum required than to try to fix all of those problems.
Comment #2
Dries CreditAttribution: Dries commentedI'm OK with the proposed patch.
Comment #3
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD and DRUPAL-5. Thanks.
Comment #4
(not verified) CreditAttribution: commentedComment #5
vlayne CreditAttribution: vlayne commentedJust 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.
Comment #6
dwwIndeed. It seems this was never actually committed to DRUPAL-5. It is fixed in DRUPAL-6 and HEAD, however. Re-rolled for DRUPAL-5.
Comment #7
PanchoYeah, I think we should commit this to DRUPAL-5.
Comment #8
drummCommitted to 5.x.
Comment #9
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.