Opened 15 years ago

Closed 15 years ago

#20769 closed defect (fixed)

mcrypt, config.inc.php issues in phpmyadmin

Reported by: ora.et.labora@… Owned by: alakazam@…
Priority: Normal Milestone:
Component: ports Version: 1.7.1
Keywords: phpmyadmin Cc:
Port: phpmyadmin

Description (last modified by ryandesign (Ryan Carsten Schmidt))

There are (at least) two issues whith phpmyadmin @3.2.0.1

  1. Extension mcrypt needs to be installed/enabled in PHP
  2. File config.inc.php is not generated.

I would expect that at least a warning message is shown during the installation that one needs to make sure that 'mcrypt' is enabled and how to do that (port install php5-mcryp for example).

The second issue about that the blowfish secret needs to be set and each and every offical documentation tells us to do so by editing config.inc.php. This file would be created during an installation either manually or automatically. This port really sucks in not following the official way - this file does not exist. If this port decides to follow an alternative route, then for heavens sake, please document this proper!!

Change History (8)

comment:1 Changed 15 years ago by blb@…

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

comment:2 Changed 15 years ago by jmroot (Joshua Root)

Type: requestdefect

comment:3 Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)

Description: modified (diff)

comment:4 in reply to:  description ; Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to ora.et.labora@…:

I would expect that at least a warning message is shown during the installation that one needs to make sure that 'mcrypt' is enabled and how to do that (port install php5-mcryp for example).

The phpmyadmin port should simply declare a runtime dependency on the php5-mcrypt port, and the ports for any other modules it requires. Versions of the php5 port prior to 5.3.0_1 had mcrypt support always enabled, but it was recently moved into a separate module port along with many other modules. See #19091.

The second issue about that the blowfish secret needs to be set and each and every offical documentation tells us to do so by editing config.inc.php. This file would be created during an installation either manually or automatically. This port really sucks in not following the official way - this file does not exist. If this port decides to follow an alternative route, then for heavens sake, please document this proper!!

The port does nothing unusual. It simply copies the phpmyadmin files into the right place. It is the user's responsibility to set it up correctly, which includes creating and editing the config.inc.php. It wouldn't hurt if the port printed a message explaining to the user that this file must be created (if the user has not already done so).

comment:5 in reply to:  4 Changed 15 years ago by ryandesign (Ryan Carsten Schmidt)

Replying to ryandesign@…:

The phpmyadmin port should simply declare a runtime dependency on the php5-mcrypt port, and the ports for any other modules it requires.

The patch attached to #20818 addresses this.

comment:6 Changed 15 years ago by alakazam@…

Status: newassigned

comment:7 Changed 15 years ago by alakazam@…

r56365 fixes the mcrypt issues.

comment:8 Changed 15 years ago by alakazam@…

Resolution: fixed
Status: assignedclosed

config.inc.php issues should be fixed by r57854.

Note: See TracTickets for help on using tickets.