Opened 13 years ago

Closed 7 years ago

#30201 closed defect (fixed)

Drop +postgres83 variant from mediawiki

Reported by: chad@… Owned by: macports-tickets@…
Priority: Normal Milestone:
Component: ports Version: 1.9.2
Keywords: Cc:
Port: mediawiki

Description

Hi, I'm one of the MediaWiki developers. In 1.17.0 we dropped support for Postgres 8.3 and below. I see you already updated the current stable release to 1.17.0 (awesome, thanks!), but you still have the +postgres83 variant available which should probably be removed.

Thanks.

Change History (6)

comment:1 Changed 13 years ago by mf2k (Frank Schima)

Owner: changed from macports-tickets@… to blb@…

comment:2 Changed 13 years ago by blb@…

Note that the versioned database variants were deprecated a while ago in favor of just 'mysql' and 'postgresql' (#20854) since the specific version is picked when php5-mysql or php5-postgresql is installed. This of course means that there could be issues if an unsupported postgresql version is used for php5-postgresql, but I'm not sure of an easy way to test for that.

comment:3 in reply to:  2 Changed 13 years ago by chad@…

Replying to blb@…:

Note that the versioned database variants were deprecated a while ago in favor of just 'mysql' and 'postgresql' (#20854) since the specific version is picked when php5-mysql or php5-postgresql is installed. This of course means that there could be issues if an unsupported postgresql version is used for php5-postgresql, but I'm not sure of an easy way to test for that.

Ah didn't know it had been deprecated. It shouldn't cause a problem for us (MW installer checks SQL version for support), it was just a housekeeping thing I noticed. If there's nothing really to be done here that you can see, feel free to resolve invalid/bogus/worksforme/etc.

comment:4 Changed 13 years ago by blb@…

'port variants mediawiki' would tell you:

mediawiki has the variants:
[+]mysql
   mysql4: Deprecated variant, use just +mysql
   postgresql: Use PostGreSQL as the database backend
   postgresql83: Deprecated variant, use just +postgresql
   postgresql84: Deprecated variant, use just +postgresql

Figure I may leave this open in case there is a simple way to verify the php5-* port needed has the right variant set.

comment:5 Changed 13 years ago by blb@…

Owner: changed from blb@… to macports-tickets@…

comment:6 Changed 7 years ago by jmroot (Joshua Root)

Resolution: fixed
Status: newclosed

In 2a94adde053182f7851c2cb221af34de8c2f4ca7/macports-ports:

mediawiki: remove deprecated variants

Fixes: #30201

Note: See TracTickets for help on using tickets.